hel-ableton added a comment.

> The problem as I see it was that the original bug, highly constrained the 
> cases where "CurrentState.LastSpace = State.Column;" to one particular style 
> (which if it happens to be your style great but not if its not.

You mean the original bugfix was already unnecessarily constrained, and now my 
proposed fix is only opening it up for one my case? That may be so. In any case 
this might really not be the ideal fix, and I'm open to any other, better way 
of fixing it.


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

https://reviews.llvm.org/D136154

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

Reply via email to