Christopher Lenz wrote:
>
> 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?
One downside, over time the trailing white spaces can build up.
On editors that support the end key taking you to the real end of
the line, sometimes it can be 20+ characters beyond the real end of
the line. I've seen it happen.
As you mention, some editors add trailing white space for no good
reason. Sometimes they don't know when to stop. This happens
especially often in editors that auto-indent the next line when
you hit return.
In any case, if everyone had our editors set to remove needless
trailing whitespace, then it also wouldn't be an issue. :)
I personally don't care either way, I just thought I'd mention
the downside since you asked. Even though I understand that it
was probably a rhetorical question. ;)
-Paul
> 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]>
--
To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
