code review

Author: MythreyaK

clang/lib/Sema/SemaInit.cpp

@@ -3095,7 +3095,7 @@ InitListChecker::CheckDesignatedInitializer(const InitializedEntity &Entity,
       }
 
       const auto GenerateDesignatedInitReorderingFixit =
-          [&](SemaBase::SemaDiagnosticBuilder &Diags) {
+          [&](SemaBase::SemaDiagnosticBuilder &Diag) {
             struct ReorderInfo {
               int Pos{};
               const Expr *InitExpr{};
@@ -3107,11 +3107,10 @@ InitListChecker::CheckDesignatedInitializer(const InitializedEntity &Entity,
             const auto *CxxRecord =
                 IList->getSemanticForm()->getType()->getAsCXXRecordDecl();
 
-            for (const auto &Field : CxxRecord->fields()) {
+            for (const FieldDecl *Field : CxxRecord->fields())
               MemberNameInx[Field->getIdentifier()] = Field->getFieldIndex();
-            }
 
-            for (const auto *Init : IList->inits()) {
+            for (const Expr *Init : IList->inits()) {
               if (const auto *DI =
                       dyn_cast_if_present<DesignatedInitExpr>(Init)) {
                 // We expect only one Designator
@@ -3148,8 +3147,8 @@ InitListChecker::CheckDesignatedInitializer(const InitializedEntity &Entity,
               const auto InitText =
                   Lexer::getSourceText(CharRange, SM, LangOpts);
 
-              Diags << FixItHint::CreateReplacement(OrigExpr->getSourceRange(),
-                                                    InitText.str());
+              Diag << FixItHint::CreateReplacement(OrigExpr->getSourceRange(),
+                                                   InitText.str());
             }
           };
 
@@ -3162,10 +3161,10 @@ InitListChecker::CheckDesignatedInitializer(const InitializedEntity &Entity,
         unsigned OldIndex = StructuredIndex - 1;
         if (StructuredList && OldIndex <= StructuredList->getNumInits()) {
           if (Expr *PrevInit = StructuredList->getInit(OldIndex)) {
-            auto Diags = SemaRef.Diag(PrevInit->getBeginLoc(),
-                                      diag::note_previous_field_init)
-                         << PrevField << PrevInit->getSourceRange();
-            GenerateDesignatedInitReorderingFixit(Diags);
+            auto Diag = SemaRef.Diag(PrevInit->getBeginLoc(),
+                                     diag::note_previous_field_init)
+                        << PrevField << PrevInit->getSourceRange();
+            GenerateDesignatedInitReorderingFixit(Diag);
           }
         }
       }