gedare added inline comments.

================
Comment at: clang/lib/Format/TokenAnnotator.cpp:5007
       return false;
     }
     const FormatToken *Previous = Right.MatchingParen->Previous;
----------------
owenpan wrote:
> Can we simply add the above and leave the original `return` statement on line 
> 5009 unchanged?
Yes this seems to work. I'll push a diff shortly along with reverting the 
EXPECT_EQ outside of my new tests.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D137762/new/

https://reviews.llvm.org/D137762

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to