Skip to content
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

Begin to clean style, spacing and indentation, ref #14 #24

Closed
wants to merge 5 commits into from

Conversation

illwieckz
Copy link
Contributor

Hi, this PR clean some style, spacing and indentation ins some files. ref #14

This PR needs #13 and must be merged after #23 and before #24 and others.

@illwieckz illwieckz changed the title Begin to cleaning style, spacing an indentation, ref #14 Begin to clean style, spacing an indentation, ref #14 Jul 6, 2015
@illwieckz illwieckz changed the title Begin to clean style, spacing an indentation, ref #14 Begin to clean style, spacing and indentation, ref #14 Jul 6, 2015
illwieckz added 2 commits July 8, 2015 17:10
- add json support to games added since Steve Teuber's patches
- use xform_names instead of name_xforms since Steven Hartland's refactoring
- use xform_printf() instead of fprintf() for json output
- remove commented out code
- use %s for a const switch seems a bit wasteful, fixing it
- do not use inline tests
- explicitely test pointers with NULL
- escape lone backslash character too
- RFC 4627 says: “JSON text SHALL be encoded in Unicode. The default encoding is UTF-8.”
- reindent display_template.c
- reindent qstat.h
- fix spacing, bracing for display_*.c and qstat.c
@illwieckz
Copy link
Contributor Author

I will redo this PR with a better one, I'm working on crustify and I will be able to produce a PR for the whole repository. This PR will just be noise if we merge another PR that does reindentation on the same files in the future. So I'm closing it.

@illwieckz illwieckz closed this Jul 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant