Page MenuHomePhabricator

[clang-format] Support text proto extensions

Authored by krasimir on Feb 12 2018, 5:18 AM.


Diff Detail

rC Clang

Event Timeline

krasimir created this revision.Feb 12 2018, 5:18 AM
djasper added inline comments.Feb 12 2018, 7:05 AM

It might be useful to attach a test case for what happens if the "[...]" does not fit on one line. I don't even know how I would format that, but it seems important to know and not accidentally modify the behavior.

krasimir updated this revision to Diff 133872.Feb 12 2018, 8:44 AM
  • Add tests with long [...]
krasimir marked an inline comment as done.Feb 12 2018, 8:45 AM
krasimir added inline comments.

I added some tests. The current behavior seems reasonable.

krasimir marked an inline comment as done.Feb 12 2018, 11:33 AM
djasper accepted this revision.Feb 12 2018, 11:28 PM

Cool, thanks.

This revision is now accepted and ready to land.Feb 12 2018, 11:28 PM
This revision was automatically updated to reflect the committed changes.