[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:30:01 +0200

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> 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:30:42 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.