Re: svn commit: r1222521 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

2011-12-23 Thread Stefan Sperling
On Fri, Dec 23, 2011 at 11:25:37AM +0100, Stefan Sperling wrote: > On Fri, Dec 23, 2011 at 10:40:59AM +0400, Konstantin Kolinko wrote: > > I would say that URL and UUID should be enough. > > The repos_id is useful information for developers trying to help users > with debugging the issue. The mess

Re: svn commit: r1222521 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

2011-12-23 Thread Stefan Sperling
On Fri, Dec 23, 2011 at 10:40:59AM +0400, Konstantin Kolinko wrote: > 2011/12/23 Hyrum K Wright : > > Yikes!  I think this wins the prize for "longest function call". > > Perhaps a helper function or at least some temporary variables would > > be in order? I look into that, sure. > I would say t

Re: svn commit: r1222521 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

2011-12-22 Thread Konstantin Kolinko
2011/12/23 Hyrum K Wright : > On Thu, Dec 22, 2011 at 6:31 PM,   wrote: >> Author: stsp >> Date: Fri Dec 23 00:31:51 2011 >> New Revision: 1222521 >> >> URL: http://svn.apache.org/viewvc?rev=1222521&view=rev >> Log: >> Turn another wc-db assertion people are reporting into a normal error. >> >> * s

Re: svn commit: r1222521 - /subversion/trunk/subversion/libsvn_wc/wc_db.c

2011-12-22 Thread Hyrum K Wright
On Thu, Dec 22, 2011 at 6:31 PM, wrote: > Author: stsp > Date: Fri Dec 23 00:31:51 2011 > New Revision: 1222521 > > URL: http://svn.apache.org/viewvc?rev=1222521&view=rev > Log: > Turn another wc-db assertion people are reporting into a normal error. > > * subversion/libsvn_wc/wc_db.c >  (read_ch