Add 'detailed-ast' output as an alias for 'dump'

[steveire]

Summary

Future development can then dump other content than AST.

[steveire]

Format

[steveire]

Update tests

[aaron.ballman]

LGTM with some fixes to the wording.

clang-query/Query.cpp

57

Add full stop at the end of the sentence.

59

Typo: deatiled -> detailed

clang-query/QueryParser.cpp

119

This is reasonable, but as you add new output modes, hopefully this list won't need to be duplicated from above.

clang-query/Query.h

 #include "clang/ASTMatchers/Dynamic/VariantValue.h"
 #include "llvm/ADT/IntrusiveRefCntPtr.h"
 #include "llvm/ADT/Optional.h"
 #include <string>
 
 namespace clang {
 namespace query {
 
-enum OutputKind { OK_Diag, OK_Print, OK_Dump };
+enum OutputKind { OK_Diag, OK_Print, OK_DetailedAST };
 
 enum QueryKind {
   QK_Invalid,
   QK_NoOp,
   QK_Help,
   QK_Let,
   QK_Match,
   QK_SetBool,

clang-query/Query.cpp

OS << "Available commands:\n\n"
         "  quit, q                           "
         "Terminates the query session.\n\n"
         "Several commands accept a <feature> parameter. The available features "
         "are:\n\n"
         "  print                             "
         "pretty-print bound nodes\n"
         "  diag                              "
         "diagnostic location for bound nodes\n"
+        "  detailed-ast                      "
+        "Detailed AST output for bound nodes\n"
         "  dump                              "
-        "Detailed AST output for bound nodes\n\n";
+        "Detailed AST output for bound nodes (alias of deatiled-ast)\n\n";
   return true;
 }
 
 bool QuitQuery::run(llvm::raw_ostream &OS, QuerySession &QS) const {
   QS.Terminate = true;
   return true;
 }
 
for (auto MI = Matches.begin(), ME = Matches.end(); MI != ME; ++MI) {
           break;
         }
         case OK_Print: {
           OS << "Binding for \"" << BI->first << "\":\n";
           BI->second.print(OS, AST->getASTContext().getPrintingPolicy());
           OS << "\n";
           break;
         }
-        case OK_Dump: {
+        case OK_DetailedAST: {
           OS << "Binding for \"" << BI->first << "\":\n";
           BI->second.dump(OS, AST->getSourceManager());
           OS << "\n";
           break;
         }
         }
       }
 

clang-query/QueryParser.cpp

QueryRef QueryParser::parseSetBool(bool QuerySession::*Var) {
   return new SetQuery<bool>(Var, Value);
 }
 
 QueryRef QueryParser::parseSetOutputKind() {
   StringRef ValStr;
   unsigned OutKind = LexOrCompleteWord<unsigned>(this, ValStr)
                          .Case("diag", OK_Diag)
                          .Case("print", OK_Print)
-                         .Case("dump", OK_Dump)
+                         .Case("detailed-ast", OK_DetailedAST)
+                         .Case("dump", OK_DetailedAST)
                          .Default(~0u);
   if (OutKind == ~0u) {
-    return new InvalidQuery("expected 'diag', 'print' or 'dump', got '" +
-                            ValStr + "'");
+    return new InvalidQuery(
+        "expected 'diag', 'print', 'detailed-ast' or 'dump', got '" + ValStr +
+        "'");
   }
   return new SetQuery<OutputKind>(&QuerySession::OutKind, OutputKind(OutKind));
 }
 
 QueryRef QueryParser::endQuery(QueryRef Q) {
   const char *Extra = Begin;
   if (!lexWord().empty())
     return new InvalidQuery("unexpected extra input: '" +