Taskbar grouping in Windows 7

Jon TURNEY jon.turney@dronecode.org.uk
Tue Jul 19 15:31:00 GMT 2011

On 01/07/2011 21:38, Tobias Häußler wrote:
> On 29/06/2011 15:25, Jon TURNEY wrote:
>> On 25/06/2011 13:48, Tobias Häußler wrote:
>>> I created a small patch for XWin that adds correct grouping of taskbar
>>> icons when 'Always combine, hide labels' is set in the taskbar
>>> properties. It uses the new taskbar APIs introduced in Windows 7 to set
>>> an application id for each window. The id is based on the X11 class hints.
>>> Maybe it is useful for someone...
>> Firstly, thanks very much for this patch.
>> Getting Windows to correctly group XWin windows on the taskbar is something
>> that has needed fixing for a while, so it's great to have it done :-)
> Thanks for your suggestions! I changed the code you mentioned.

Thanks.  I've included this patch into the 1.10.3-1 package.

I noticed that there are a couple of warnings issued when building

>   CC     winmultiwindowwm.o
> In file included from winmultiwindowwm.c:69:0:
> taskbar.h:59:19: warning: ‘IID_IPropertyStore’ initialized and declared ‘extern’
> taskbar.h:67:53: warning: ‘PKEY_AppUserModel_ID’ initialized and declared ‘extern’

This seems to be related to the nonsense that is INITGUID.  I'm not sure how
to fix this warning.  Would moving all the GUIDs we use (including the DirectX
ones we use) to a separate file and compiling that with INITGUID defined be
the correct solution?

The other thing I noticed is that PropVariantClear() has been provided by
ole32.dll since at least NT4, so there's no need to dynamically link with it.
 So I'm confused as to why you are using GetProcAddress for that?

Volunteer Cygwin/X X Server maintainer

Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple
Problem reports:       http://cygwin.com/problems.html
Documentation:         http://x.cygwin.com/docs/
FAQ:                   http://x.cygwin.com/docs/faq/

More information about the Cygwin-xfree mailing list