Skip to content
This repository has been archived by the owner on Jun 4, 2022. It is now read-only.

Pretty-print contours as well #30

Open
RoelN opened this issue Sep 5, 2016 · 5 comments
Open

Pretty-print contours as well #30

RoelN opened this issue Sep 5, 2016 · 5 comments

Comments

@RoelN
Copy link

RoelN commented Sep 5, 2016

When using --pretty it would be nice to see a newline for each contour value.

"contours": [[{"x":-16383,"y":16383,"on":true},{"x":16383,"y":16383,"on":true},{"x":16383,"y":-16383,"on":true},{"x":-16383,"y":-16383,"on":true}]]

to

"contours": [[
    {"x":-16383,"y":16383,"on":true},
    {"x":16383,"y":16383,"on":true},
    {"x":16383,"y":-16383,"on":true},
    {"x":-16383,"y":-16383,"on":true}
]]
@be5invis
Copy link
Contributor

be5invis commented Sep 5, 2016

Hmmm. There IS an option, lies in premake5.lua, removing _CARYLL_USE_PRE_SERIALIZED WILL enable formatting contours, but it will damage the performance.

@RoelN
Copy link
Author

RoelN commented Sep 5, 2016

I suppose most editors/IDEs are capable or pretty-printing JSON, or there are plugins to do that. To me it would be convenient and worth the drop in performance, as --pretty is probably used for human eyes as opposed to output meant for another script to work with, but it's a non-urgent enhancement.

@be5invis
Copy link
Contributor

be5invis commented Sep 5, 2016

@RoelN I use jq often. It has a built-in formatter, so everything is OK.

@RoelN
Copy link
Author

RoelN commented Sep 6, 2016

So are you going to leave it to and outside tool, or build it into otfcc?

@be5invis
Copy link
Contributor

be5invis commented Sep 6, 2016

@RoelN Maybe external, though there is an option to trigger.
otfcc is focused on the font part, performs serialization and optimizations.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants