MyDeveloperDay marked an inline comment as done.
MyDeveloperDay added inline comments.


================
Comment at: clang/docs/ReleaseNotes.rst:284
 
+- Basic Support has been adding for Formatting .json files (with very limited 
options)
+
----------------
curdeius wrote:
> Maybe instead of putting "with very limited options", you may add a link to 
> the doc describing limitations?
This new strategy actually gives us lots of pre-existing options, for example:

using `TabWidth` and `UseTab`  we can tabify/de-tabify the json

using `SpacesInContainerLiterals` we can control if the json is `["foo"]` or `[ 
"foo" ]`

Any option that can be used to format the javascript:

`x =  { ... } `

can be used to format the JSON, I believe this gives us the best of both 
worlds, and some flexibility for the future if people want to format JSON in a 
specific BraceWrapping style.





CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93528/new/

https://reviews.llvm.org/D93528

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to