klimek added a comment. In D54881#1462893 <https://reviews.llvm.org/D54881#1462893>, @MyDeveloperDay wrote:
> > I agree and would be happy with the change if it would only change the > > line-filtered workflow, but this afaict (unless I'm missing something :) > > will also affect the workflow where the provided range is 0-length range, > > which has an implicit "format stuff around this" request from the user > > inside it. I'd be happy with a patch that differentiates these two sides. > > This use case needs capturing in a unit test..it isn't obvious Agreed! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D54881/new/ https://reviews.llvm.org/D54881 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
