kwk added a comment. In D61440#1488450 <https://reviews.llvm.org/D61440#1488450>, @teemperor wrote:
> FYI, I still think there are some style issues there (e.g. the > AppleObjCRuntimeV2.cpp change makes it longer than 80 chars) but I can just > clang-format this before committing. > > Otherwise, unless someone objects to this change I'll merge this tomorrow. > Thanks for the patch @kkleine! @teemperor I've formatted the AppleObjCRuntimeV2.cpp using `git clang-format` and it worked. Thank you for pointing me this handy tool. But as you probably can see now in commit 0b4b24065a12678c773f8c3f0fc722e50405fc85, not all the changes were related to the `override` keyword. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D61440/new/ https://reviews.llvm.org/D61440 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
