[PATCH] D40103: [libFuzzer] Encapsulate commands in a class

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 16 12:41:05 PST 2017


morehouse added inline comments.


================
Comment at: lib/fuzzer/FuzzerCommand.cpp:38
+
+Command::Command(const Command &other)
+    : Args(other.Args), CombinedOutAndErr(other.CombinedOutAndErr),
----------------
LLVM style prefers first letter of parameters and locals to be capitalized.  (here and below)

The one common exception I've seen is for one-letter loop indexes to be lowercase (though some people capitalize those too).


https://reviews.llvm.org/D40103





More information about the llvm-commits mailing list