Unity-Technologies / qstat

New official qstat repository
Artistic License 2.0
121 stars 33 forks source link

Split the display code, fix #28 #23

Closed illwieckz closed 7 years ago

illwieckz commented 9 years ago

The code is just unmaintainable, both standard, raw, xml and json display logic is in qstat.c which is wrong.

This PR split standard, raw, xml, and json display logic in dedicated c-files, create needed h-files for them, and create a h-file for templating display too to avoid redeclarations (the template code was just copy pasting some qstat.h lines, and qstat.h were copypasting some template definitions).

must be merged right after #13 and before #25 and others

stevenh commented 7 years ago

Not fully used, but the requirement to split out JSON was included in #53