[PATCH] D28081: Make GetStyle return Expected<FormatStyle> instead of FormatStyle
Alexander Shaposhnikov via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jan 9 22:06:33 PST 2017
alexshap added inline comments.
================
Comment at: lib/Format/Format.cpp:424
+llvm::Error make_string_error(const llvm::Twine &Message) {
+ return llvm::make_error<llvm::StringError>(Message,
----------------
amaiorano wrote:
> ioeric wrote:
> > Maybe make this `inline`?
> Yes.
regarding this function and some other helper functions in this file:
1. why aren't they static ?
(if this stuff is not used outside of this .cpp - but yeah, i assume i'm missing smth)
2. http://llvm.org/docs/CodingStandards.html naming conventions are
"Function names should be verb phrases (as they represent actions), and command-like function should be imperative. The name should be camel case, and start with a lower case letter (e.g. openFile() or isFoo())"
https://reviews.llvm.org/D28081
More information about the cfe-commits
mailing list