[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: svn commit: r1480406 - in /subversion/trunk: ./ subversion/include/ subversion/include/private/ subversion/libsvn_subr/ subversion/libsvn_subr/utf8proc/ subversion/tests/ subversion/tests/libsvn_subr/

From: Branko Čibej <brane_at_wandisco.com>
Date: Tue, 14 May 2013 06:33:00 +0200

On 14.05.2013 06:30, Branko Čibej wrote:
> On 14.05.2013 05:04, Daniel Shahaf wrote:
>> Joe Swatosh wrote on Mon, May 13, 2013 at 18:53:35 -0700:
>>> Is SIZE_MAX c89? Should it be APR_SIZE_MAX in utf8proc.h (along with
>>> including svn_dep_compat.h)?
>> subversion/libsvn_subr/utf8proc/utf8proc.h is a vendor branch. So
>> perhaps we should make sure to include a header file that does
>> #ifndef SIZE_MAX \n #define SIZE_MAX APR_SIZE_MAX \n #endif
>> before including subversion/libsvn_subr/utf8proc/utf8proc.h ?
> It turns out that utf8proc.h includes <stdbool.h> and <stdint.h>

Eh, that's <inttypes.h> not <stdint.h>.

> on
> platforms other than (predictably) Windows. So if we want to maintain
> C90 compatibility it'll take more than just a hack; we'll have to modify
> the header itself.
> Which I don't find to be a particularly big pain since we already
> localize messages in the .c file, for example. It's what vendor branches
> are for after all.
> -- Brane

Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com
Received on 2013-05-14 06:33:36 CEST

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.