angelgarcia added a comment. I implemented this, with the following addition: if several errors share the same interval, I can still apply the biggest one that was not discarded during the sweep. This way, the first example would work and in the two other examples it would just apply a random one, as you suggested. But I found a different case that also fails: A: [a, b)[c,d) B: [e, f) such that e < a, b < f < c.
In this case it just discards A. This is not incorrect (as long as we don't apply both, everything is OK), but it gets away from the idea of "if two fixes overlap, discard both", which is what Manuel said when I started this. I don't think that changing that idea is a problem, but right now the behavior of the tool is so diffuse that maintaining and testing it would be a bit painful. Also, if we want to allow applying one of the fixes when they overlap, we have a different problem, it might be better just to start from scratch and think about how to solve that problem. http://reviews.llvm.org/D13516 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
