This is the mail archive of the cygwin-apps@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: [PATCH] Setup window.{cc,h}


Approved - Max, can you please do the honours.


On Wed, 2003-06-04 at 14:27, Gary R. Van Sickle wrote:
> > Approved on one condition - please remove the reference to folk in Hell.
> > I'm not christian, but that doesn't seem at all appropriate.
> >
> 
> Fair enough; religious humor excommunicated. ;-)
> 
> > A small suggestion (doesn't prevent it being commited) - a helper fn to
> > return an already casted value might reduce the number of casts needed.
> 
> You mean a cast operator to do what GetHWND() is doing?  Yeah, I thought of
> that; I'll probably end up doing that at some point.  The comment though is more
> of a lament that the user can't just forget about HWNDs entirely and use just
> methods that don't break encapsulation.  Ah well, it ain't MFC... yet. ;-)

I wasn't thinking of an operator per se. More just one (two?) utility
fn's.

Rob
-- 
GPG key available at: <http://members.aardvark.net.au/lifeless/keys.txt>.

Attachment: signature.asc
Description: This is a digitally signed message part


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