I checked in a small patch to your branch, can you test to see if it breaks anything?
Itamar On 21/8/2010 1:24 AM, Kostka Bořivoj wrote: > I checked Jlucene and exception is thrown here only in case of error, so I'll > fix function this way. > > Borek > > >> -----Original Message----- >> From: Kostka Bořivoj [mailto:kos...@tovek.cz] >> Sent: Friday, August 20, 2010 9:43 PM >> To: clucene-developers@lists.sourceforge.net >> Subject: [CLucene-dev] TermVectorsWriter::close strange code >> >> In AddIndexesNoOptimize test I got an exception during >> IndexWriter::addIndexesNoOptimize call. >> >> I discovered the problem is in >> >> void TermVectorsWriter::close(CLuceneError* err) >> >> where last block of code >> >> if ( err != NULL ) >> err->set(keep.number(), keep.what()); >> else >> throw keep; >> >> is always called, but it should be called only if some error occurs, I think >> >> Could anybody check, if I didn't miss something? >> >> Borek >> >> ------------------------------------------------------------------------------ >> This SF.net email is sponsored by >> >> Make an app they can't live without >> Enter the BlackBerry Developer Challenge >> http://p.sf.net/sfu/RIM-dev2dev >> _______________________________________________ >> CLucene-developers mailing list >> CLucene-developers@lists.sourceforge.net >> https://lists.sourceforge.net/lists/listinfo/clucene-developers >> > ------------------------------------------------------------------------------ > This SF.net email is sponsored by > > Make an app they can't live without > Enter the BlackBerry Developer Challenge > http://p.sf.net/sfu/RIM-dev2dev > _______________________________________________ > CLucene-developers mailing list > CLucene-developers@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/clucene-developers > > > ------------------------------------------------------------------------------ This SF.net email is sponsored by Make an app they can't live without Enter the BlackBerry Developer Challenge http://p.sf.net/sfu/RIM-dev2dev _______________________________________________ CLucene-developers mailing list CLucene-developers@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/clucene-developers
