Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
List-Subscribe: <mailto:cygwin-subscribe AT cygwin DOT com>
List-Archive: <http://sourceware.org/ml/cygwin/>
List-Post: <mailto:cygwin AT cygwin DOT com>
List-Help: <mailto:cygwin-help AT cygwin DOT com>, <http://sourceware.org/ml/#faqs>
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Delivered-To: mailing list cygwin AT cygwin DOT com
Message-ID: <43752488.60E7014E@dessent.net>
Date: Fri, 11 Nov 2005 15:08:56 -0800
From: Brian Dessent <brian AT dessent DOT net>
MIME-Version: 1.0
To: cygwin AT cygwin DOT com
Subject: Re: Error in GetICMProfile() declaration
References: <Pine DOT LNX DOT 4 DOT 63 DOT 0511111327200 DOT 1443 AT vgfl DOT cnhywyhpnf DOT bet>
Content-Type: text/plain; charset=us-ascii
Content-Transfer-Encoding: 7bit
X-IsSubscribed: yes
Reply-To: cygwin AT cygwin DOT com

"Paul J. Lucas" wrote:

> So this really looks like a bug in the declaration.

It looks like you're right.  I believe that the w32api headers are
maintained by the mingw project so the best thing to do would probably
be submit a patch here
<http://sourceforge.net/tracker/?group_id=2435&atid=302435> or a message
here <http://sourceforge.net/forum/forum.php?forum_id=286641> alerting
them of the issue.  Note that for w32api you have to be careful not to
submit patches that are based on code found in e.g. the platform SDK or
MS sample code, so that there isn't any question of taint.  I don't
think a patch that just changes DWORD to LPDWORD would have any issues
though.

Brian

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