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

Re: [PATCH] Improve conformance to ISO C90

From: Ivan Zhakov <ivan_at_visualsvn.com>
Date: Fri, 14 Oct 2016 16:37:41 +0200

On 14 October 2016 at 15:24, Patrick Steinhardt
<patrick.steinhardt_at_elegosoft.com> wrote:
> Hi,
>
> attached patch fixes compatibility with ISO C90. It fixes
> trailing commas in enum lists as well as one case where variadic
> macros are used, which are a feature of C99. What it does not fix
> is strings with overlength (C90 only allows for fixed strings
> with a maximum length of 509 characters).
>
> This allows us to build the code with `-Werror -pedantic
> -Wno-overlength-strings`.
>
> [[
> Improve C90 compatibility by removing trailing commas in enum
> lists as well as converting a variadic macro stub with an empty
> function.
>
> * subversion/include/svn_client.h:
> (svn_client_config_option_id_t): remove trailing comma
> * subversion/include/svn_wc.h:
> (svn_wc_notify_action_t): remove trailing comma
> * subversion/libsvn_subr/cache-membuffer.c:
> (prefix_pool_create): remove trailing comma
> * subversion/svn/svn.c:
> (svn_cl__longopt_t): remove trailing comma
> * subversion/svnmucc/svnmucc.c:
> (sub_main): remove trailing comma
> * subversion/svnmover/linenoise/linenouse.c:
> (lndebug): replace variadic macro stub with an empty variadic
> function
> ]]
Committed in 1764919 with tweaked log message.

-- 
Ivan Zhakov
Received on 2016-10-14 16:38:34 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.