On 07.05.2011 13:27, Daniel Shahaf wrote:
> * apr_vformatter() doesn't support the 'll' modifier. When given %l
> followed by something other than 'd' or 'u' (for %ld or %lu, etc) ---
> in our case, followed by 'l' --- it just copies the (first) '%l' and
> discards the second 'l', to the effect of having the following error:
>
> 13:54:21 <@danielsh> subversion/libsvn_subr/svn_string.c:781: (apr_err=200004)
> 13:54:21 <@danielsh> svnadmin: E200004: Number '542543543543543543543115' is out of range '[%ld, %ld]'
>
> So, we should remove stop using %lld with apr_pvsprintf() and friends.
Depends on which version of APR you're talking about, recently there
were some changes in that area. Still, the proper thing to do is to use
the formatting macros defined by APR.
-- Brane
Received on 2011-05-08 04:50:47 CEST