Plethora777 / mcpe_viz

Minecraft Pocket Edition (MCPE) World Visualization & Reporting Tool with Web App
GNU General Public License v2.0
167 stars 39 forks source link

Don't intermix tabs/spaces for indention #24

Closed shellbj closed 7 years ago

shellbj commented 7 years ago

Pick one please!

kylesutton commented 7 years ago

+1 for tabs, though there are a number of places where the current styling wouldn't fit well with their use (definition of _do_chunk_v3 function, for instance).

shellbj commented 7 years ago

I think spaces would be the least impactful; most indention is spaces until you get to a natural 8 character tab stop.

Plethora777 commented 7 years ago

I use emacs, and well... I don't think this is a big deal :)

shellbj commented 7 years ago

Then do everyone a favor and add this to your settings (setq-default indent-tabs-mode nil)

Plethora777 commented 7 years ago

Thanks very much for the tip -- I will check it out!

I didn't mean to be dismissive, I just did not know a reasonable fix for it without changing my editor, which would be a drag. I appreciate it.

Plethora777 commented 7 years ago

Updating to add a .dir-locals.el file with the command suggested by @shellbj Thanks.