search

FlixelCommunity / Flixel-Power-Tools

The Flixel Power Tools are a package of classes designed to provide extra functionality to your Flixel games. Originally created by Richard Davey (Photon Storm), this is the Flixel Community fork of the library, striving to keep the library living and up to date with the latest version of Flixel.
http://www.photonstorm.com/flixel-power-tools
Other
8 stars 3 forks source link

Removed all tabs and replaced them with 4 spaces. #21

Closed VestOfHolding closed 3 years ago

VestOfHolding commented 10 years ago

Did my best to correct all alignment issues caused by this. Couldn't find a way to do this in FlashDevelop, so I did it in Notepad++.

IQAndreas commented 10 years ago

Nice work, however, I actually believe it is better to use tabs instead of spaces. That way, if one developer likes two spaces and another prefers four, each developer can individually adjust the amount of whitespace per tabs in their IDE's settings.

VestOfHolding commented 10 years ago

While that's true, you still end up with the case that since tabs are read differently by the various programs you can use to look at the code. Sometimes to the point of having a negative effect on the readability. At the very least I would recommend two things:

  1. Pick a setting and enforce it on the entire project. I haven't seen anything like that last time I looked. I seem to remember there being a section of the documentation devoted to conditions for contributing somewhere.
  2. Put in that same section of the documentation how to tell FlashDevelop and maybe other IDEs how to automatically convert tabs to spaces.

At the end of the day, it was an issue that was easy to address, so I did. Just forgot to link to the issue. Issue #20