Hello, I have my editor (CodeGuide) configured in following way:
Tab size = 4, indent size = 4 Remove trailing white space when saving = true 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:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
