You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Greg Stein <gs...@gmail.com> on 2009/06/23 20:35:40 UTC

Re: svn commit: r37937 - trunk/subversion/libsvn_wc

On Fri, Jun 5, 2009 at 14:21, Hyrum K. Wright<hy...@hyrumwright.org> wrote:
>...
> +++ trunk/subversion/libsvn_wc/wc_db.c  Fri Jun  5 05:21:51 2009        (r37937)
>...
> @@ -231,7 +231,7 @@ static const char * const statements[] =
>   "select id from wcroot where local_abspath is null;",
>
>   /* STMT_SELECT_REPOSITORY */
> -  "select id, root from repository where uuid = ?1;",
> +  "select id from repository where uuid = ?1 and root = ?2;",

The root is unique. No need to include the uuid in the WHERE clause.

Cheers,
-g

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2364684


Re: svn commit: r37937 - trunk/subversion/libsvn_wc

Posted by "Hyrum K. Wright" <hy...@hyrumwright.org>.
On Jun 23, 2009, at 3:35 PM, Greg Stein wrote:

> On Fri, Jun 5, 2009 at 14:21, Hyrum K. Wright<hy...@hyrumwright.org>  
> wrote:
>> ...
>> +++ trunk/subversion/libsvn_wc/wc_db.c  Fri Jun  5 05:21:51  
>> 2009        (r37937)
>> ...
>> @@ -231,7 +231,7 @@ static const char * const statements[] =
>>   "select id from wcroot where local_abspath is null;",
>>
>>   /* STMT_SELECT_REPOSITORY */
>> -  "select id, root from repository where uuid = ?1;",
>> +  "select id from repository where uuid = ?1 and root = ?2;",
>
> The root is unique. No need to include the uuid in the WHERE clause.

r38176.

-Hyrum

------------------------------------------------------
http://subversion.tigris.org/ds/viewMessage.do?dsForumId=462&dsMessageId=2364707