Empty tabs and spaces left over from earlier code edits are getting forcibly removed from code on saving. While this can be a nice feature to keep things clean, there seems to be no way of disabling the thing and is being a major inconvenience for github collaboration as it's introducing unintended changes in commits that in worse cases hide the few intentional line changes beneath an avalanche of unintentional whitespace changes throughout the entire file.
Numbered Steps to Reproduce Problem:
1. Edit some older code in a little fix attempt that is not supposed to touch the other parts of the code.
2. Hit save.
3. Commit your branch and get yelled at by the management for unrelated changes that aren't supposed to be there. (Or alternatively just open the file again and see all the unconsented whitespace removal.)
4. Face the humiliation of having to manually add useless void whitespace back into the code with a text editor just to preserve consistency.
Code Snippet (if applicable) to Reproduce Problem:
/*
Hi.
Here comes some bad formatting
There's some empty tabs between there and here's a space at the end.
You are not supposed to touch this part of the code by the way. It's illegal t: management
*/
Expected Results:
Expected for DM to only save the actual changed changes.
Actual Results:
DM proceeded to "clean" things up without asking. Even adding those back and hitting save to get them back lossed them.
Does the problem occur:
Every time? Or how often? Every time.
In other games? Only in DM editing.
In other user accounts? Needs confirmation.
On other computers? Needs confirmation.
When does the problem NOT occur?
In the past before some updates.
Did the problem NOT occur in any earlier versions? If so, what was the last version that worked?
Will have to crapshoot later when I have time to try out all of them.
Seems that now even the current stable version is doing that.
(Visit http://www.byond.com/download/build to download old versions for testing.)
Workarounds:
Redoing all your changes with other external text editors.
Anywho, I agree that should be disableable, at least the empty line one (maybe not the trailing space one) purely because it can make it mildly annoying to insert code later when empty lines reset to the base tab level, and what it provides doesn't outweigh that (minor) inconvenience.