Page MenuHomePhabricator

[clang-format] Support python-style comments in text protos

Authored by krasimir on Nov 8 2017, 8:34 AM.

Diff Detail


Event Timeline

krasimir created this revision.Nov 8 2017, 8:34 AM
krasimir updated this revision to Diff 122213.Nov 9 2017, 2:34 AM
  • Add more tests
krasimir added a subscriber: cfe-commits.
bkramer added a subscriber: bkramer.Nov 9 2017, 7:38 AM
bkramer added inline comments.
48 ↗(On Diff #122213)

These should stay arrays. You can use ArrayRef to get a convenient reference to them.

djasper accepted this revision.Nov 10 2017, 12:53 AM
djasper added inline comments.
344 ↗(On Diff #122213)

There should be a linebreak here.

59 ↗(On Diff #122213)

Maybe it's now better to do:

return && !FormatTok.TokenText.startsWith("/*");


This revision is now accepted and ready to land.Nov 10 2017, 12:53 AM
krasimir updated this revision to Diff 122420.Nov 10 2017, 4:46 AM
krasimir marked 3 inline comments as done.
  • [clang-format] Address review comments
This revision was automatically updated to reflect the committed changes.