[PATCH 04 of 10] wireproto: bypass filechunkiter for small files when streaming

Mads Kiilerich mads at kiilerich.com
Thu Sep 13 19:03:32 CDT 2012


Bryan O'Sullivan wrote, On 09/14/2012 01:14 AM:
> On Thu, Sep 13, 2012 at 3:06 PM, Mads Kiilerich <mads at kiilerich.com 
> <mailto:mads at kiilerich.com>> wrote:
>
>     Failure _is_ not an option. (The test should perhaps complain harder.)
>
>
> I don't know what you mean here. The test is wrong in this particular 
> case.

The test detected that check-code reported an error - not just a warning 
which could have been whitelisted.

That error has to be dealt with in some other way.

I guess the advice given by check-code should be followed. 
opener().read() might cause problems with Python implementations that 
doesn't use reference counting, especially on Windows.

/Mads


More information about the Mercurial-devel mailing list