[PATCH] D46508: [llvm-rc] Allow optional commas between the string table index and value

[Zachary Turner via Phabricator via llvm-commits]

zturner accepted this revision.
zturner added inline comments.
This revision is now accepted and ready to land.


================
Comment at: test/tools/llvm-rc/Inputs/tag-stringtable-basic.rc:9
+  1, "b"
+  16, "bb"
 }
----------------
Can you put a comment above this that says "commas are optional, so we make sure to test both cases"?

This way someone editing this test in the future knows to preserve the commas.

Alternatively, you could break this out into a separate test called `tag-stringtable-commas.rc`


Repository:
  rL LLVM

https://reviews.llvm.org/D46508