This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Support repeated field lists in protos
ClosedPublic

Authored by krasimir on Feb 14 2018, 10:04 AM.

Diff Detail

Repository
rL LLVM

Event Timeline

krasimir created this revision.Feb 14 2018, 10:04 AM
djasper accepted this revision.Feb 15 2018, 5:34 AM

Looks good.

lib/Format/TokenAnnotator.cpp
2355 ↗(On Diff #134254)

This is almost a duplicate of the one above. Can we pull out a function or lambda?

This revision is now accepted and ready to land.Feb 15 2018, 5:34 AM
krasimir updated this revision to Diff 134427.Feb 15 2018, 7:21 AM
  • Pull out check in lambda
  • Address review comments
This revision was automatically updated to reflect the committed changes.