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

Re: svn commit: r37185 - trunk/subversion/libsvn_subr

From: Christian Boos <cboos_at_neuf.fr>
Date: Tue, 14 Apr 2009 20:42:30 +0200

Hello,

Greg Stein wrote:
> On Sun, Apr 12, 2009 at 17:14, Arfrever Frehtes Taifersar Arahesis
> <Arfrever.FTA_at_gmail.com> wrote:
>
>> ...
>> +++ trunk/subversion/libsvn_subr/sqlite.c Sun Apr 12 08:14:29
2009 (r37185)
>> @@ -597,8 +597,12 @@ init_sqlite(apr_pool_t *pool)
>> "thread-safe mode"));
>> #endif
>> #if SQLITE_VERSION_AT_LEAST(3,6,0)
>> - SQLITE_ERR_MSG(sqlite3_config(SQLITE_CONFIG_MULTITHREAD),
>> - "Could not configure SQLite");
>> + /* If SQLite has been already initialized, sqlite3_config() returns
>> + SQLITE_MISUSE. */
>> + int err = sqlite3_config(SQLITE_CONFIG_MULTITHREAD);
>> + if (err != SQLITE_OK && err != SQLITE_MISUSE)
>> + return svn_error_create(SQLITE_ERROR_CODE(err), NULL,
>> + "Could not configure SQLite");
>> SQLITE_ERR_MSG(sqlite3_initialize(), "Could not initialize SQLite");
>> #endif
>>
>
> You'll need some braces in there for that variable declaration (can't
> (portably) declare the variable mid-block).
>

That fix is also needed for Trac, see http://trac.edgewall.org/ticket/8199
Would you please consider porting that change to the 1.6.x branch? Thanks!

-- Christian

PS: sorry Greg for the double post, as I didn't do a "Reply All" at first...

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=1714565
Received on 2009-04-14 20:43:07 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.