[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: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: 2007-04-17 22:59:46 CEST

Blair Zajac wrote:
> 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 :)

Oh, I agree. :) I just wanted to know if there was something I needed
to be aware of in the future.

-Hyrum

Received on Tue Apr 17 22:59:33 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.