RFR(XS): 7901298: jcheck should check that every file ends with exactly one newline

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Mon May 25 10:13:06 UTC 2015


On 2015-05-13 11:37, Chris Hegarty wrote:
> On 13/05/15 08:35, Volker Simonis wrote:
>> Ping...
>>
>> Can anybody please have a look at the new webrev?
>
> I am not a Reviewer on this project, but I quickly skimmed the webrev 
> and it seems ok.
>
>> I think the proposed solution is not so bad :)
>
> I agree. If we want to evolve jcheck in a manner that enables more 
> strict checking, then such an approach seems reasonable.

I seems that this approach will lead to each improvement requiring a new 
line in the configuration. I think that's not ideal. But then again, we 
need *some* way of being able to evolve jcheck. And if the requirement 
is to be backward compatible with all previous commits, then this might 
be the only way.

So I'm in favour for Volker's fix, at least until someone finds a better 
way to evolve jcheck.

/Magnus


More information about the hg-tools-dev mailing list