owenca wrote:

> As per @efriedma-quic's comment I'll go ahead and revert the commit again to 
> prevent the immediate breakage.

@PiJoules, I don't know if @efriedma-quic's comment was an endorsement of your 
original hasty (and incomplete) revert, but it didn't address the first part of 
the policy I quoted above. Please stop forcing your incomplete revert in 
without first consulting the clang-format team @mydeveloperday, 
@HazardyKnusperkeks, etc. If we reach the consensus to fix #160513 by reverting 
instead of by fixing forward, you need to identify the proper "set of patches" 
to revert and open a pull request for approval by the clang-format team. That 
set must include both #111115 and #164048 and most likely one or more commits 
in between. cc @JohnC32

https://github.com/llvm/llvm-project/pull/164670
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to