malcolm.parsons added a comment.

In, @jdemeule wrote:

> Ok, I understand the problem.
>  Previously, during the deduplication step, replacements per file where 
> sorted and it is not the case anymore.
>  That means now, clang-apply-replacement is highly dependant of reading file 
> order during error reporting.
>  I mainly see 2 options (I can miss others), rewrite the test (e.g merging 
> yaml file together) or sort the replacements per file to fix the order of 
> application.
> Can I ask you what solution you prefer?

I'd prefer sorting, so that results will be consistent when the tool is used on 
real data.

cfe-commits mailing list

Reply via email to