On Fri, 2 Mar 2012 15:58:39 +0100, Justus Winter <4winter at
informatik.uni-hamburg.de> wrote:
> Formerly the xapian database object was deleted and closed in its
> destructor once the object was garbage collected. Explicitly call
> close() so that the database and the associated lock is
On Fri, 2 Mar 2012 15:58:39 +0100, Justus Winter
4win...@informatik.uni-hamburg.de wrote:
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated lock is released
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated lock is released
immediately.
The comment is a courtesy of Austin Clements.
Signed-off-by: Justus Winter <4winter at
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated lock is released
immediately.
The comment is a courtesy of Austin Clements.
Signed-off-by: Justus Winter
On Thu, Mar 01, 2012 at 07:59:30AM +0100, Justus Winter wrote:
> Olly wrote:
> >It is hard to say if calling close() is actually useful here from just
> >seeing the patch.
>
> Huh? I provided a test case...
I only saw the part of the patch Austin quoted in the mail he cc-ed to
me.
> Quoting
Quoth Justus Winter on Mar 01 at 7:59 am:
> Quoting Austin Clements (2012-02-29 23:17:54)
> >Quoth Olly Betts on Feb 29 at 9:19 pm:
> >> On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
> >> > Quoth Justus Winter on Feb 29 at 10:19 am:
> >> > > Formerly the xapian database object
Hi :)
Olly wrote:
>It is hard to say if calling close() is actually useful here from just
>seeing the patch.
Huh? I provided a test case...
Quoting Austin Clements (2012-02-29 23:17:54)
>Quoth Olly Betts on Feb 29 at 9:19 pm:
>> On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
Quoth Justus Winter on Mar 01 at 7:59 am:
Quoting Austin Clements (2012-02-29 23:17:54)
Quoth Olly Betts on Feb 29 at 9:19 pm:
On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
Quoth Justus Winter on Feb 29 at 10:19 am:
Formerly the xapian database object was deleted
On Thu, Mar 01, 2012 at 07:59:30AM +0100, Justus Winter wrote:
Olly wrote:
It is hard to say if calling close() is actually useful here from just
seeing the patch.
Huh? I provided a test case...
I only saw the part of the patch Austin quoted in the mail he cc-ed to
me.
Quoting Austin
On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
> Quoth Justus Winter on Feb 29 at 10:19 am:
> > Formerly the xapian database object was deleted and closed in its
> > destructor once the object was garbage collected. Explicitly call
> > close() so that the database and the
Quoth Olly Betts on Feb 29 at 9:19 pm:
> On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
> > Quoth Justus Winter on Feb 29 at 10:19 am:
> > > Formerly the xapian database object was deleted and closed in its
> > > destructor once the object was garbage collected. Explicitly call
Quoth Justus Winter on Feb 29 at 10:19 am:
> Formerly the xapian database object was deleted and closed in its
> destructor once the object was garbage collected. Explicitly call
> close() so that the database and the associated lock is released
> immediately.
Interesting. Is this a bug in
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated lock is released
immediately.
Signed-off-by: Justus Winter <4winter at informatik.uni-hamburg.de>
---
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated lock is released
immediately.
Signed-off-by: Justus Winter 4win...@informatik.uni-hamburg.de
---
lib/database.cc |
On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
Quoth Justus Winter on Feb 29 at 10:19 am:
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
close() so that the database and the associated
Quoth Olly Betts on Feb 29 at 9:19 pm:
On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
Quoth Justus Winter on Feb 29 at 10:19 am:
Formerly the xapian database object was deleted and closed in its
destructor once the object was garbage collected. Explicitly call
Hi :)
Olly wrote:
It is hard to say if calling close() is actually useful here from just
seeing the patch.
Huh? I provided a test case...
Quoting Austin Clements (2012-02-29 23:17:54)
Quoth Olly Betts on Feb 29 at 9:19 pm:
On Wed, Feb 29, 2012 at 10:48:33AM -0500, Austin Clements wrote:
17 matches
Mail list logo