r314484 - [NFC] Replace 'arguments' with 'parameters' in comments relating to lexing a macro definition.

Faisal Vali via cfe-commits cfe-commits at lists.llvm.org
Thu Sep 28 19:43:23 PDT 2017


Author: faisalv
Date: Thu Sep 28 19:43:22 2017
New Revision: 314484

URL: http://llvm.org/viewvc/llvm-project?rev=314484&view=rev
Log:
[NFC] Replace 'arguments' with 'parameters' in comments relating to lexing a macro definition.

Modified:
    cfe/trunk/lib/Lex/PPDirectives.cpp

Modified: cfe/trunk/lib/Lex/PPDirectives.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Lex/PPDirectives.cpp?rev=314484&r1=314483&r2=314484&view=diff
==============================================================================
--- cfe/trunk/lib/Lex/PPDirectives.cpp (original)
+++ cfe/trunk/lib/Lex/PPDirectives.cpp Thu Sep 28 19:43:22 2017
@@ -2132,10 +2132,10 @@ void Preprocessor::HandleIncludeMacrosDi
 // Preprocessor Macro Directive Handling.
 //===----------------------------------------------------------------------===//
 
-/// ReadMacroParameterList - The ( starting an argument list of a macro
-/// definition has just been read.  Lex the rest of the arguments and the
+/// ReadMacroParameterList - The ( starting a parameter list of a macro
+/// definition has just been read.  Lex the rest of the parameters and the
 /// closing ), updating MI with what we learn.  Return true if an error occurs
-/// parsing the arg list.
+/// parsing the param list.
 bool Preprocessor::ReadMacroParameterList(MacroInfo *MI, Token &Tok) {
   SmallVector<IdentifierInfo*, 32> Parameters;
 
@@ -2143,7 +2143,7 @@ bool Preprocessor::ReadMacroParameterLis
     LexUnexpandedToken(Tok);
     switch (Tok.getKind()) {
     case tok::r_paren:
-      // Found the end of the argument list.
+      // Found the end of the parameter list.
       if (Parameters.empty())  // #define FOO()
         return false;
       // Otherwise we have #define FOO(A,)
@@ -2167,7 +2167,7 @@ bool Preprocessor::ReadMacroParameterLis
         Diag(Tok, diag::err_pp_missing_rparen_in_macro_def);
         return true;
       }
-      // Add the __VA_ARGS__ identifier as an argument.
+      // Add the __VA_ARGS__ identifier as a parameter.
       Parameters.push_back(Ident__VA_ARGS__);
       MI->setIsC99Varargs();
       MI->setParameterList(Parameters, BP);
@@ -2185,7 +2185,7 @@ bool Preprocessor::ReadMacroParameterLis
         return true;
       }
 
-      // If this is already used as an argument, it is used multiple times (e.g.
+      // If this is already used as a parameter, it is used multiple times (e.g.
       // #define X(A,A.
       if (std::find(Parameters.begin(), Parameters.end(), II) !=
           Parameters.end()) {  // C99 6.10.3p6
@@ -2193,7 +2193,7 @@ bool Preprocessor::ReadMacroParameterLis
         return true;
       }
 
-      // Add the argument to the macro info.
+      // Add the parameter to the macro info.
       Parameters.push_back(II);
 
       // Lex the token after the identifier.




More information about the cfe-commits mailing list