search

mzijlstra / flowcharts

Flowcharts: Computational Thinking Tool
Apache License 2.0
2 stars 0 forks source link

Clean up: JSON.stringify() -> wr.stringify() #26

Closed mzijlstra closed 7 years ago

mzijlstra commented 9 years ago

I use JSON.stringify() throught the code -- some places have exceptions on them to make it look better, some don't... And really for arrays I would like to have spaces after every comma.

It would be good to make a global helper function: wr.stringify() that does all of this, and then have all the code just use this helper (keeping things clean and clear).

mzijlstra commented 7 years ago

Don't think I care anymore...

mzijlstra commented 7 years ago

Long ago me knew what he was talking about, and was absolutely right, a stringify() function is critical. More recent me did not know what he was talking about when he said that he didn't care anymore.

Due to re-discovering the technical necessity of this issue I have now implemented it. Although not as a global wr.stringify() function, but rather as a private stringify() function inside the exec.js file