Re: recent patch to w32api/include/commctrl.h

["Robert Collins" <robert dot collins at itdomain dot com dot au>]

===
----- Original Message -----
From: "Danny Smith" <danny_r_smith_2001@yahoo.co.nz>
To: "cygwin-patches" <cygwin-patches@cygwin.com>
Sent: Wednesday, January 23, 2002 11:20 AM
Subject: recent patch to w32api/include/commctrl.h


> Robert,
> Your recent patch:
> Log message:
> 2001-12-17  Robret Collins  <rbtcollins@hotmail.com>
>
> * include/commctrl.h: New typedefs for HDLAYOUT and LPHDLAYOUT based
> on MSDN documentation for XP.
>
> added a new struct _HDITEM, which has a UNICODED field (LPTSTR
pszText) in
> it. However, some code expects to have explicit HDITEMA and HDITEMW
> structures.  Since the the new structure is essantially a replacement
> (since IE 3.0) for old HD_ITEM[AW], wouldn't following make more sense
> (second part of attached patch).  I'm also suggesting same approach
for
> NMLISTVIEW.

Wuoting from MSDN:
"typedef struct _HDITEM {
    UINT    mask;
    int     cxy;
    LPTSTR  pszText;
    HBITMAP hbm;
    int     cchTextMax;
    int     fmt;
    LPARAM  lParam;
#if (_WIN32_IE >= 0x0300)
    int     iImage;
    int     iOrder;
#endif
#if (_WIN32_IE >= 0x0500)
    UINT    type;
    LPVOID  pvFilter;
#endif
} HDITEM, *LPHDITEM;
"

I've no set opinion, as long as setup still builds :}.

Rob