Removing jsbeautifier from the project using the pure json lib which …#6
Removing jsbeautifier from the project using the pure json lib which …#6pmithrandir wants to merge 1 commit intomainfrom
Conversation
…is more efficient.
|
I'll run this and see what the output looks like tomorrow, thank you! |
|
I have run it, and have remembered that this is how I did the JSON output initially, but thought the files were very long and difficult to scan down to look for interesting things - I introduced jsbeautifier to save having to write some code to output the arrays without newlines after each character. |
|
Hi, I think it's a matter of choice. If speed is relevant for thet output generation, moving to json lib make sense... but I'm not sure it is currently. What I could suggest is to keep your system and to add a comment explaining what is the expected gain and why we do not choose to use json lib in the end. If performance are requested, the user can also disable pretty export. |
…is more efficient.
Please check if the presentation of the data is still ok for you.
Main change is that list value are now listed one per line, where before you could get many per line.
File will be longer, but values will be easier to search.