More review comments

Author: vgvassilev

clang/lib/AST/DeclPrinter.cpp

@@ -258,6 +258,7 @@ void DeclPrinter::prettyPrintAttributes(const Decl *D,
     return;
 
   if (D->hasAttrs()) {
+    assert(Pos != AttrPosAsWritten::Unknown && "Use Default");
     const AttrVec &Attrs = D->getAttrs();
     for (auto *A : Attrs) {
       if (A->isInherited() || A->isImplicit())
@@ -272,10 +273,10 @@ void DeclPrinter::prettyPrintAttributes(const Decl *D,
         assert(APos != AttrPosAsWritten::Unknown && "Implicit attribute!");
         if (Pos == AttrPosAsWritten::Default || Pos == APos) {
           if (Pos != AttrPosAsWritten::Left)
-            Out << " ";
+            Out << ' ';
           A->printPretty(Out, Policy);
           if (Pos == AttrPosAsWritten::Left)
-            Out << " ";
+            Out << ' ';
         }
         break;
       }

clang/lib/AST/StmtPrinter.cpp

@@ -296,7 +296,7 @@ void StmtPrinter::VisitAttributedStmt(AttributedStmt *Node) {
   for (const auto *Attr : Attrs) {
     Attr->printPretty(OS, Policy);
     if (Attr != Attrs.back())
-      OS << " ";
+      OS << ' ';
   }
 
   PrintStmt(Node->getSubStmt(), 0);