[PATCH] Bigger Chooser Part 3 In Super 3-D: RECTPP

Max Bowsher maxb@ukf.net
Sat Apr 5 18:17:00 GMT 2003


Gary R. Van Sickle wrote:
>> Robert Collins wrote:
>>> On Sat, 2003-04-05 at 19:23, Gary R. Van Sickle wrote:
>>>> 2003-04-05  Gary R. Van Sickle  <g.r.vansickle@worldnet.att.net>
>>>>
>>>> * RECTPP.h: New file.
>>>
>>> Much better. A few remaining nits.
>>>
>>> a) The class name, as discussed before.
>>> b) RECTCC_H__F4098557_9A48_446d_AF28_2BE45D29F68D__INCLUDED_
>>>    Uhm, thats plain ugly.
>
> It's a GUID, it's supposed to by ugly.  Guaranteed to be unique, not
> going to collide with anything ever.  That's how MS's "AppWizard" does
> it, and it seems like a good idea to me.  Please Rob, if you're going to
> pick nits, pick on something that could possibly matter.  For me?  Please?

Gary, this doesn't matter much, but please no GUIDs.

I suggest you use _RECTPP_H_, as this follows the most common scheme in
setup.

Also, the file has DOS line endings. If convenient, please d2u it.
Otherwise, mention that the committer must d2u it when you resend.

Max.



More information about the Cygwin-apps mailing list