This is an archive of the discontinued LLVM Phabricator instance.

Differential D48760

[clang-format] Support additional common functions for text proto formatting
ClosedPublic

Authored by krasimir on Jun 29 2018, 3:20 AM.

Details

Reviewers
djasper
Commits
rG61f46f224725: [clang-format] Support additional common functions for text proto formatting
rL335978: [clang-format] Support additional common functions for text proto formatting
rC335978: [clang-format] Support additional common functions for text proto formatting
Summary

This adds a few more common function names expecting a text proto argument.

Diff Detail

Repository
rC Clang

Event Timeline

krasimir created this revision.Jun 29 2018, 3:20 AM
Harbormaster completed remote builds in B19876: Diff 153459.Jun 29 2018, 3:20 AM
Herald added a subscriber: cfe-commits. · View Herald TranscriptJun 29 2018, 3:20 AM
krasimir added a reviewer: djasper.Jun 29 2018, 3:20 AM
djasper accepted this revision.Jun 29 2018, 7:15 AM
This revision is now accepted and ready to land.Jun 29 2018, 7:15 AM
Closed by commit rC335978: [clang-format] Support additional common functions for text proto formatting (authored by krasimir). · Explain WhyJun 29 2018, 7:30 AM
This revision was automatically updated to reflect the committed changes.

Revision Contents

PathSize
lib/
Format/
5 lines

Diff 153491

lib/Format/Format.cpp

Show First 20 Lines • Show All 769 Lines • ▼ Show 20 Lines GoogleStyle.RawStringFormats = {
{ {
"pb", "pb",
"PB", "PB",
"proto", "proto",
"PROTO", "PROTO",
}, },
/*EnclosingFunctionNames=*/ /*EnclosingFunctionNames=*/
{ {
"PARSE_TEXT_PROTO",
"EqualsProto", "EqualsProto",
"EquivToProto",
"PARSE_TEST_PROTO",
"PARSE_TEXT_PROTO",
"ParseTextOrDie",
}, },
/*CanonicalDelimiter=*/"", /*CanonicalDelimiter=*/"",
/*BasedOnStyle=*/"google", /*BasedOnStyle=*/"google",
}, },
}; };
GoogleStyle.SpacesBeforeTrailingComments = 2; GoogleStyle.SpacesBeforeTrailingComments = 2;
GoogleStyle.Standard = FormatStyle::LS_Auto; GoogleStyle.Standard = FormatStyle::LS_Auto;
▲ Show 20 Lines • Show All 1,455 Lines • Show Last 20 Lines