[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r24612 - trunk/subversion/bindings/javahl/native

From: Blair Zajac <blair_at_orcaware.com>
Date: 2007-04-17 22:47:37 CEST

Hyrum K. Wright wrote:
> blair@tigris.org wrote:
>> Author: blair
>> Date: Tue Apr 17 13:32:49 2007
>> New Revision: 24612
>>
>> Log:
>> * subversion/bindings/javahl/native/*.{h,cpp}:
>> Formatting clean up:
>> Start comment sentences with capital letters.
>> Do not use inline comments.
>> Wrap to 80 characters.
>> Add vertical whitespace between functions and methods,
>> preprocessor macros.
>> Remove comments on constructor and destructors where the comment
>> was "this is a constructor" or "this is a destructor".
>
>
> Blair,
> Thanks for the much needed cleanups. It seems that lots of comments that
> were already under 80 characters per line were wrapped even farther. Is
> there a reason for this?

Hyrum,

No specific reason. Some of them had minor changes in it, so while I'm
in XEmacs, I just do a reformat of the paragraph. I think XEmacs wraps
it down to 72 characters, and I'm just used to seeing them that short
and sometimes is wraps nicer.

This is purely bike-sheddy stuff :)

Regards,
Blair

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Apr 17 22:47:50 2007

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.