[PATCH] Fix formatting and wording of llvm-ranlib error message
Yung, Douglas via llvm-commits
llvm-commits at lists.llvm.org
Thu Apr 7 13:27:47 PDT 2016
When llvm-ranlib is run without arguments, it prints out the following error message:
llvm-ranlib.exe: llvm-ranlib.exetakes just one archive as argument
My patch fixes this up a little by adding a space between the "llvm-ranlib.exe" and "takes", as well as adds the word "an" between the words "an" and "argument". With my patch, the error message would look like this:
llvm-ranlib.exe: llvm-ranlib.exe takes just one archive as an argument
This change is NFC.
Here is the simple patch:
Index: tools/llvm-ar/llvm-ar.cpp
===================================================================
--- tools/llvm-ar/llvm-ar.cpp (revision 265713)
+++ tools/llvm-ar/llvm-ar.cpp (working copy)
@@ -760,7 +760,7 @@
static int ranlib_main() {
if (RestOfArgs.size() != 1)
- fail(ToolName + "takes just one archive as argument");
+ fail(ToolName + " takes just one archive as an argument");
ArchiveName = RestOfArgs[0];
return performOperation(CreateSymTab, nullptr);
}
Douglas Yung
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160407/92cd2eef/attachment.html>
More information about the llvm-commits
mailing list