[prev in list] [next in list] [prev in thread] [next in thread] 

List:       subversion-dev
Subject:    Re: svn commit: r1816069 - /subversion/branches/1.8.x-issue4707/subversion/svnrdump/dump_editor.c
From:       Julian Foad <julianfoad () apache ! org>
Date:       2017-11-23 9:47:18
Message-ID: fe0413f8-cfb1-d9bf-b108-54ca6dc44d16 () apache ! org
[Download RAW message or body]

Julian Foad wrote:
> Evgeny Kotkov wrote:
>> I think that using APR_SIZE_T_FMT would still lead to the same issue with
>> large file sizes in the 32-bit environment (where size_t is also 32 bit).
>>
>> Perhaps, the code should be using APR_OFF_T_FMT as the format specifier?
>>
>> (The apr_file_info_t.size value is an apr_off_t)
> 
> Thank you! I will fix it.

Fixed in r1816111 on '1.8.x-issue4707' branch. 'STATUS' nomination updated.

- Julian
[prev in list] [next in list] [prev in thread] [next in thread]