* Code Formatting vs Trailing Whitespaces
@ 2008-06-23 18:15 Florian Köberle
2008-06-23 20:31 ` Robin Rosenberg
0 siblings, 1 reply; 4+ messages in thread
From: Florian Köberle @ 2008-06-23 18:15 UTC (permalink / raw)
To: git; +Cc: Shawn O. Pearce, Robin Rosenberg, Marek Zawirski
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi
I noticed that Robin removed trailing whitespaces from my patch(thanks).
It appears that if you enable code formatting and the removement of
trailing whitespaces in the save actions option that the formatting
option will win.
It will add a space at the second line of a javadoc comment:
/**
~ * <----
I think this is a bug in eclipse, did any of you create a bug report for
that? Or a feature request for an "no whitespace after javadoc" option
in the code formatter page?
I want to have both options active, because I don't want to commit wrong
formatted patches.
Best regards,
Florian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFIX+hW59ca4mzhfxMRApU4AKCbYzwRFqVcBCr11zc9lk6jqick2ACeMBhW
nXOoN4nIsTjzWWhQzDOxiz4=
=bm+i
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Code Formatting vs Trailing Whitespaces
2008-06-23 18:15 Code Formatting vs Trailing Whitespaces Florian Köberle
@ 2008-06-23 20:31 ` Robin Rosenberg
2008-06-23 21:17 ` Florian Köberle
0 siblings, 1 reply; 4+ messages in thread
From: Robin Rosenberg @ 2008-06-23 20:31 UTC (permalink / raw)
To: Florian Köberle; +Cc: git, Shawn O. Pearce, Marek Zawirski
måndagen den 23 juni 2008 20.15.50 skrev Florian Köberle:
> Hi
>
> I noticed that Robin removed trailing whitespaces from my patch(thanks).
> It appears that if you enable code formatting and the removement of
> trailing whitespaces in the save actions option that the formatting
> option will win.
> It will add a space at the second line of a javadoc comment:
> /**
> ~ * <----
>
> I think this is a bug in eclipse, did any of you create a bug report for
> that? Or a feature request for an "no whitespace after javadoc" option
> in the code formatter page?
This one matches: https://bugs.eclipse.org/bugs/show_bug.cgi?id=49619
> I want to have both options active, because I don't want to commit wrong
> formatted patches.
When I have both options active, it adds the whitespace when formatting, but
removes it when saving in (version 3.3.2).
-- robin
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Code Formatting vs Trailing Whitespaces
2008-06-23 20:31 ` Robin Rosenberg
@ 2008-06-23 21:17 ` Florian Köberle
2008-06-24 6:01 ` Robin Rosenberg
0 siblings, 1 reply; 4+ messages in thread
From: Florian Köberle @ 2008-06-23 21:17 UTC (permalink / raw)
To: Robin Rosenberg; +Cc: git, Shawn O. Pearce, Marek Zawirski
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
Hi
| This one matches: https://bugs.eclipse.org/bugs/show_bug.cgi?id=49619
Thank you. I voted for that feature, maybe it helps to draw attention to
this feature wish.
| When I have both options active, it adds the whitespace when
formatting, but
| removes it when saving in (version 3.3.2).
Strange, I use version 3.3.2 too. I configured it in the workspace settings.
Maybe you did in the project settings?
Best regards,
Florian
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org
iD8DBQFIYBLy59ca4mzhfxMRAs7vAJ4la1ABhh5Q6pwvhGdx9bZHJvQvWwCfS8nr
2oakNrgbvGSVlGkLzwlJYbw=
=rZFh
-----END PGP SIGNATURE-----
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: Code Formatting vs Trailing Whitespaces
2008-06-23 21:17 ` Florian Köberle
@ 2008-06-24 6:01 ` Robin Rosenberg
0 siblings, 0 replies; 4+ messages in thread
From: Robin Rosenberg @ 2008-06-24 6:01 UTC (permalink / raw)
To: Florian Köberle; +Cc: git, Shawn O. Pearce, Marek Zawirski
måndagen den 23 juni 2008 23.17.38 skrev Florian Köberle:
> Hi
>
> | This one matches: https://bugs.eclipse.org/bugs/show_bug.cgi?id=49619
> Thank you. I voted for that feature, maybe it helps to draw attention to
> this feature wish.
>
> | When I have both options active, it adds the whitespace when
> formatting, but
> | removes it when saving in (version 3.3.2).
> Strange, I use version 3.3.2 too. I configured it in the workspace settings.
> Maybe you did in the project settings?
Indeed.
-- robin
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2008-06-24 6:06 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-06-23 18:15 Code Formatting vs Trailing Whitespaces Florian Köberle
2008-06-23 20:31 ` Robin Rosenberg
2008-06-23 21:17 ` Florian Köberle
2008-06-24 6:01 ` Robin Rosenberg
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).