hxlibc improvements to the HXCPP tool

[jgranick]

The first commit separates the HXCPP run tool into multiple files, per class. This should make it a bit easier to maintain.

The second commit performs a little bit of standardization on the code, following the current code style. This helps remove bad indenting, and other inconsistencies -- this pass was not too dramatic, I did not want to assume that certain kinds of spaces, etc, were always intended to be similar.

The third commit includes improvements made from the hxlibc tool, primarily, it includes additional logic for logging, running process and working with paths. The error messages in places are cleaned a bit, and it automatically uses ASCII coloring when available. Errors will also not throw a stack trace from the HXCPP tool, as an error when calling g++ (for example) is not relevant to the internals of the tool.

[hughsando]

I notice that this has removed the multi-threaded output reading. Was this
done for a reason?

On Tue, Mar 25, 2014 at 5:36 AM, Joshua Granick notifications@github.comwrote:

The first commit separates the HXCPP run tool into multiple files, per
class. This should make it a bit easier to maintain.

The second commit performs a little bit of standardization on the code,
following the current code style. This helps remove bad indenting, and
other inconsistencies -- this pass was not too dramatic, I did not want to
assume that certain kinds of spaces, etc, were always intended to be
similar.

The third commit includes improvements made from the hxlibc tool,
primarily, it includes additional logic for logging, running process and
working with paths. The error messages in places are cleaned a bit, and it
automatically uses ASCII coloring when available. Errors will also not
throw a stack trace from the HXCPP tool, as an error when calling g++ (for

example) is not relevant to the internals of the tool.

You can merge this Pull Request by running

git pull https://github.com/jgranick/hxcpp patch-2

Or view, comment on, or merge it at:

#37
Commit Summary

• Refactor HXCPP run tool into multiple files
• More consistent code style
• Tool improvements

File Changes

• M run.nhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-0(0)
• M tools/run/BuildTool.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-1(1991)
• A tools/run/Compiler.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-2(278)
• A tools/run/DirManager.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-3(97)
• A tools/run/File.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-4(72)
• A tools/run/FileGroup.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-5(143)
• A tools/run/HLSL.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-6(44)
• A tools/run/Linker.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-7(217)
• A tools/run/LogManager.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-8(123)
• A tools/run/PathManager.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-9(299)
• A tools/run/ProcessManager.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-10(244)
• M tools/run/Setup.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-11(552)
• A tools/run/Stripper.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-12(29)
• A tools/run/Target.hxhttps://github.com/hxlibc improvements to the HXCPP tool #37/files#diff-13(70)

Patch Links:

• https://github.com/HaxeFoundation/hxcpp/pull/37.patch
• https://github.com/HaxeFoundation/hxcpp/pull/37.diff

Reply to this email directly or view it on GitHubhttps://github.com//pull/37
.

[jgranick]

I have made fixes, so now everything appears to work properly. I had not noticed some of the other changes in how threaded builds work, using the original threaded process/output code, it's working great ;)