This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Recognize "hxx" as a C++ header in clang-format-diff.py
ClosedPublic

Authored by vmiklos on Sep 18 2020, 12:28 PM.

Details

Summary

And shift "proto" to the next line to avoid a too long line.

Diff Detail

Event Timeline

vmiklos created this revision.Sep 18 2020, 12:28 PM
Herald added a project: Restricted Project. · View Herald TranscriptSep 18 2020, 12:28 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
vmiklos requested review of this revision.Sep 18 2020, 12:28 PM

Mydeveloperday, Sam: could you please take a look? Hopefully a trivial one. Thanks :-)

There are some larger codebases like LibreOffice that use .hxx as their header file extension, it's a bit inconsistent that .cxx is recognized but .hxx is not. This is meant to fix the issue.

This revision is now accepted and ready to land.Sep 18 2020, 12:37 PM
This revision was landed with ongoing or failed builds.Sep 18 2020, 12:43 PM
This revision was automatically updated to reflect the committed changes.

Thanks for the quick feedback!