From: Hazen B. <hba...@ma...> - 2008-04-15 23:47:14
|
On Apr 15, 2008, at 5:25 PM, Andrew Ross wrote: > On Tue, Apr 15, 2008 at 01:43:05PM -0400, Hezekiah M. Carty wrote: >> On Tue, Apr 15, 2008 at 4:52 AM, Andrew Ross >> <and...@us...> wrote: >>> >>> Hi Hez, >>> I've commited these changes with just a couple of modifications. >>> Note >>> that in C source files we use C style comments like /* */ and not >>> C++ comments like //. Some examples of the latter have crept >>> into the >>> source, but it is best avoided if possible. There was a time >>> when not >>> all C compilers supported C++ comments, but I suspect those days >>> are >>> gone. >>> >>> Andrew >> >> Andrew, >> >> Thanks for the response and for applying the fix. My apologies on >> the >> commenting issue - I will stick to C style comments in the future. >> >> Along those lines - should I update the patch I submitted for >> plimagefr coordinate transforms accordingly? I used C++ // comments >> for most of my comments in that code. >> >> Thank you again for taking the time to review and check this in. >> >> Hez > > No problem. Don't worry about the comments. I've been through and > fixed > the comments in the main src directory. There were a couple of other > instances as well, which no-one has complained about. This probably > backs up my statement that modern compilers don't care. My apologies as well. I'll clean up the comments in the cairo, svg and aqt driver code. -Hazen |