extract.js - pretty mode should sort properties alphabetically about heroprotocoljs HOT 3 CLOSED

I disagree. extract.js is included to prove the library works, and it works as such.

These 'value-add' features are 'feature-creep' and should be for another project.

I feel this is a back-end library and should be for the protocol and documentation only.

from heroprotocoljs.

In order for people working on other projects to leverage the library fully, it needs to make deciphering the replays content as easy as possible. The documentation is part of that, but reading extracted files manually to get a feel of them is another.

If someone just wants to dump data to be exploited elsewhere, not using the --pretty option outputs a one-line file with a direct JSON representation of the data.

If someone wants to have a manual look at the inside of a file, they can add the --pretty option to add new lines and indent. The original library also sorted the properties alphabetically, I suppose for readability reasons. As the --pretty option is not intended for optimized code but for debugging and exploration in the first place, it has all it's place here.

from heroprotocoljs.

Done 8204114.

from heroprotocoljs.