[PATCH] D68767: [clang-format] NFC - Move functionality into functions to help code structure

MyDeveloperDay via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 10 01:07:19 PDT 2019


MyDeveloperDay created this revision.
MyDeveloperDay added reviewers: klimek, mitchell-stellar, owenpan.
MyDeveloperDay added a project: clang-format.
Herald added a project: clang.

I don't think its the main()'s responsibility to be working out how to dump the config, or the format()'s  responsibility to determine what is and isn't a valid BOM

move those functions out to their own functions, this gives clear roles and responsibilities and it makes it easier to add new additional capabilities.


Repository:
  rC Clang

https://reviews.llvm.org/D68767

Files:
  clang/tools/clang-format/ClangFormat.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D68767.224272.patch
Type: text/x-patch
Size: 5393 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191010/ec76bf81/attachment.bin>


More information about the cfe-commits mailing list