MyDeveloperDay added a comment.
something @sammccall said about support `foo["name"]` made me realise that
our javascript support doesn't always support what all the options
x = {
"firstName" : "John",
"lastName" : "Smith",
"isAlive" : true,
"age" : 27,
"address" : {
"streetAddress" : "21 2nd Street",
"city" : "New York",
"state" : "NY",
"postalCode" : "10021-3100"
},
"phoneNumbers" : [
{
"type" : "home",
"number" : "212 555-1234",
"address" : "10 long address line, special road to nowhere",
},
{"type" : "office", "number" : "646 555-4567"}
],
"children" : [],
"children" : [ "foo", "bar" ],
"children" : [ "foo" ],
"children" : {},
"spouse" : null
}
with
Language: JavaScript
BasedOnStyle: LLVM
SpaceInEmptyBlock: true
SpacesInSquareBrackets: false
made me think it should have been
"children" : ["foo","bar"],
"children" : ["foo"],
"children" : { },
it seems perhaps that ensuring javascript supports these would likely given
JSON some of the style capabilities that people might want.
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
