jbushmaster007 / anyedittools

Automatically exported from code.google.com/p/anyedittools
Other
1 stars 0 forks source link

'Remove trailing whitespace' should be off by default #86

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Find yourself in a need of a cool plugin, like AnyEdit Tools
2. Install the plugin
3. Use one function of the plugin (I used: Convert To Unicode notation)

What is the expected output? What do you see instead?

Expected:
Function works.
Nothing else should be changed changed.

Observed:
Function works.
I have to MANUALLY revert tons of changes in my file before committing. 
I have to FIND OUT why all this happened (maybe because of Eclipse setting?), 
FIND OUT it was AnyEdit, FIND the checkbox and disable it.
Time wasted. 

What version of the product are you using? On what operating system?
2.4.4.201307082032, Linux

Please provide any additional information below.

I understand 'Remove trailing whitespace' is a super cool feature, but for me 
it is also redundant. If a code I'm editing is properly formatted, I use 
Eclipse for reformat it after editing, so no redundant spaces are left (except 
for the javadoc lines..).

If, however, the code I'm editing is NOT properly formatted, I CANNOT use 
formatter on the whole file, or any feature like 'Remove trailing whitespace', 
as this produces completely unreadable commit, where important changes are lost 
in an overall noise and, BEWARE, any future merge with other branches will most 
likely fail.

Original issue reported on code.google.com by piotr.fi...@gmail.com on 31 Oct 2013 at 8:05

GoogleCodeExporter commented 8 years ago
I know about Issue 80. That one is related, but those are not duplicates.

I strongly believe there is a user-unfriendly default setting. With issue 80 
completed, there could be a user-friendly default setting which preserves 
'Remove trailing whitespace' functionality.

Original comment by piotr.fi...@gmail.com on 31 Oct 2013 at 8:07

GoogleCodeExporter commented 8 years ago
There are some thousands of anyedit users and everyone has different opinions 
which settings make sense and which not. 

Unfortunately for you, your personal preference does not match the preference 
of almost all other anyedit users (otherwise there would be much more bugs in 
the tracker), so you have to do a hard work and select the checkbox each time 
you create a new workspace (or not use anyedit or use workspace mechanics 
plugin to maintain you preferred settings over all workspaces).

Closing as wont fix.

Original comment by iloveeclipse on 31 Oct 2013 at 8:38

GoogleCodeExporter commented 8 years ago
Thank you for explanation.
I will have to advise against installing AnyEdit within my company. People are 
lazy and tend not to do 'hard work and select the checkbox' :)

Original comment by piotr.fi...@gmail.com on 31 Oct 2013 at 9:50