This is the mail archive of the cygwin-patches@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: src/winsup/w32api ChangeLog include/wingdi.h


> Robert. I know your intentions were good, but please there is 
> no need to
> submit to the patch tracker page at mingw SourceForge site as 
> well. That is
> for submission of new patches needing review.  Unless you 
> make clear that
> you have comitted this patch to winsup CVS, it may lead to 
> someone else
> (like myself) checking in your patch to the SourceForge CVS (perhaps
> modified) leading to conflicts at merging.

Ok. So does this sound right:
If I commit to cygwin, don't submit to sourceforge, if I don't checkin
myself, post to sourceforge?
 
> Also, in future, keeping to the general layout of existing 
> w32api headers
> (defines, then typedefs, then prototypes) would be good.  
> This makes it
> easier to protect typedefs and prototypes against RC_INVOKED, 
> while leaving
> the constants visible to windres.

Ok. I was following the MS header arrangement - I wasnt' aware of a
particular layout in the headers for w32api. Will follow in the future.

Rob


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]