04.03.2002 14:48:05, "Pill, Juergen" <[EMAIL PROTECTED]> wrote: >Hello, > >I have my editor (CodeGuide) configured in following way: > >Tab size = 4, indent size = 4 >Remove trailing white space when saving = true
I don't see that as being helpful. Tabs to spaces is good, but how does trailing white space hurt anyone? There's still lots of them in the source, and editors like the Eclipse java editor by default *add* trailing white space in some situations, like JavaDoc comments (IIRC). So IMHO, that option should be disabled. The rest looks good to me, thanks for checking. -chris >Use windows line separators = true >Convert tabs to spaces when saving = true > >We discussed those settings some time ago (if I remember everything >right). Is this setting still agreed? Unfortunately my windiff >command suppressed whitespace changes. > >Best regards, > >Juergen > > > > > -----Original Message----- >From: Remy Maucherat [mailto:[EMAIL PROTECTED]] >Sent: Saturday, March 02, 2002 20.06 PM >To: Slide Developers Mailing List >Subject: Re: cvs commit: >jakarta-slide/src/share/org/apache/slide/authenticate CredentialsToken.java > >> 26.02.2002 09:49:32, [EMAIL PROTECTED] wrote: >> >juergen 02/02/26 00:49:32 >> > >> > Modified: src/share/org/apache/slide/authenticate >> > CredentialsToken.java >> > Log: >> > cache principal instance and added getPrincipal method. Because I >> > need the principal in the stores (Eckehard) >> >> [snip] >> > + /** >> > + * Returns the current principal >> > + * >> > + * @return java.security.Principal >> > + */ >> > + public Principal getPrinipal() { >> > + return principal; >> > + } >> >> Ermmm, that should be getPrincipal() instead of getPrinipal(), I'm >> sure (focus on the missing 'c'). >> :o) >> >> > /** >> > * Returns the public creddentials. >> > - * >> > - * @return String >> > + * >> > + * @return String >> > */ >> > public String getPublicCredentials() { >> > return credentials; >> >> Why is it that the commits coming from you guys always report plenty >> of 'invisible' changes, like removed whitespace at end of line etc. >> It makes looking at the diffs really cumbersome. Would it be possible >> to turn that off (assuming it's being done automatically by your >> favorite editor) ? > >My original sources did contain tabs. Apparently Juergen's editor converts >the tabs to spaces (hopefully, it's not the other way around ;-)), which is >good in some ways, but isn't too good an idea. > >If we want to get rid of tabs, we should do it all at once, IMO. Otherwise, >it makes the diffs really annoying to deal with, as Christopher pointed out. > >Remy > > >-- >To unsubscribe, e-mail: <mailto:slide-dev- [EMAIL PROTECTED]> >For additional commands, e-mail: <mailto:slide-dev- [EMAIL PROTECTED]> > >-- >To unsubscribe, e-mail: <mailto:slide-dev- [EMAIL PROTECTED]> >For additional commands, e-mail: <mailto:slide-dev- [EMAIL PROTECTED]> > > -chris ________________________________________________________________ cmlenz at gmx.de -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
