[clang] [clang-format] Fix crashes in AlignArrayOfStructures (PR #72520)
    Björn Schäpers via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Fri Nov 17 00:02:50 PST 2023
    
    
  
================
@@ -1316,6 +1316,8 @@ void WhitespaceManager::alignArrayInitializersRightJustified(
         auto Offset = std::distance(Cells.begin(), CellIter);
         for (const auto *Next = CellIter->NextColumnElement; Next;
              Next = Next->NextColumnElement) {
+          if (RowCount >= CellDescs.CellCounts.size())
+            break;
----------------
HazardyKnusperkeks wrote:
I would change the other lines too. I think it's better to see the condition in the `for`, but it's your call.
https://github.com/llvm/llvm-project/pull/72520
    
    
More information about the cfe-commits
mailing list