New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

codeStyle app folder #2046

Open
wants to merge 1 commit into
base: development
from

Conversation

Projects
None yet
4 participants
@John3
Contributor

John3 commented Jul 13, 2017

Add code style, fix spaces, tab, 3 spaces, brackets and indentation.

relate to:

This PR just have the app folder, later I will continue with the whole project.

Forum: http://forums.torque3d.org/viewtopic.php?f=2&t=1042&start=10

@dottools ? :D

@John3

This comment has been minimized.

Show comment
Hide comment
@John3

John3 Jul 13, 2017

Contributor

oh wait! should be 3 or 4 spaces? lol

Contributor

John3 commented Jul 13, 2017

oh wait! should be 3 or 4 spaces? lol

@dottools

This comment has been minimized.

Show comment
Hide comment
@dottools

dottools Jul 13, 2017

Oh my... Well the syntax corrections look correct, but it's going to be a nightmare for people to diff against their repositories. ;)

Edit: It's 3 spaces (yuck I know) indention traditionally with Torque code base.

dottools commented Jul 13, 2017

Oh my... Well the syntax corrections look correct, but it's going to be a nightmare for people to diff against their repositories. ;)

Edit: It's 3 spaces (yuck I know) indention traditionally with Torque code base.

@John3

This comment has been minimized.

Show comment
Hide comment
@John3

John3 Jul 13, 2017

Contributor

Oh my... Well the syntax corrections look correct, but it's going to be a nightmare for people to diff against their repositories. ;)

aaaah T_T we can't never beat you with the code style nathan :( you are a heartless person lol :D

Edit: maybe with the t3d v4 release can be include and use the 4 spaces instead.

Contributor

John3 commented Jul 13, 2017

Oh my... Well the syntax corrections look correct, but it's going to be a nightmare for people to diff against their repositories. ;)

aaaah T_T we can't never beat you with the code style nathan :( you are a heartless person lol :D

Edit: maybe with the t3d v4 release can be include and use the 4 spaces instead.

@Azaezel

This comment has been minimized.

Show comment
Hide comment
@Azaezel

Azaezel Jul 14, 2017

Contributor

Fars it goes, I don't thiiiiiiiiink there's anything being touched this release that would conflict with alterations to the app directory, so cleanup on that one shouldn't cause any headaches with wipwork...

Contributor

Azaezel commented Jul 14, 2017

Fars it goes, I don't thiiiiiiiiink there's anything being touched this release that would conflict with alterations to the app directory, so cleanup on that one shouldn't cause any headaches with wipwork...

@Areloch Areloch added the Improvement label Jul 25, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment