Re: Is it woth to have SVN_REVNUM_T_FMT?

C.A.T.Magic wrote:

> Branko Cibej wrote:
>
>> -- Always use %ld (or %lu), and cast the values. Not good, because
>> we then truncate 64-bit values on 32-bit platforms. Unfortunately
>> there is no portable format string for 64-bit integers.
>>
>> -- Define our own formats (e.g., %R for revision numbers) and write
>> our own printf format scanner that converts them to the defined
>> macros (urgh!)
>
>
> why doesn't
> apr_file_printf
> do this by itself?
> this way you could get rid of all these weird
> apr_file_printf(out, "%" APR_SIZE_T_FMT " bytes were read\n",
> total_bytes);
> constructs at the same time.
> I think APR could even be modified for this without creating
> incompatibilities with existing applications,
> because "%" APR_SIZE_T_FMT " bytes" would simply resolve to
> something like "%Z bytes" for all applications.
> but i don't know the design decisions for this in APR.

These could of course be added to APR, and that would be the nicest
solution -- *if* we use APR's vformatter functions exclusively. But
that's something we can't require from all clients.

Or can we? Personally I'd be quite happy whith this restriction.

-- 
Brane Čibej   <brane_at_xbc.nu>   http://www.xbc.nu/brane/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org