njames93 added a comment. I did have another plan for a different way to go about this, not sure if its really any better though. It involved building a vector of the edits the config would apply to its parent. Then after the parent is parsed just applying those edits. It results in only 1 pass over the config file however it's a little more involved to get it to work.
================ Comment at: clang/include/clang/Format/Format.h:3339 + bool AllowUnknownOptions = false, + llvm::SourceMgr::DiagHandlerTy DiagHandler = nullptr); ---------------- I know this is sufficient for the purpose of this patch, however it doesn't make sense to accept take the handler parameter without also taking a void* for its context. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D96760/new/ https://reviews.llvm.org/D96760 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
