clucene-developers — CLucene development-related discussion.

Re: [CLucene-dev] NearSpansUnordered bug fix

[Kostka B. <ko...@to...>]

HighlighterFix and FSDirectory_fix are ready to merge, I think
Borek

> -----Original Message-----
> From: Itamar Syn-Hershko [mailto:it...@co...]
> Sent: Saturday, November 06, 2010 4:55 PM
> To: clu...@li...
> Subject: Re: [CLucene-dev] NearSpansUnordered bug fix
> 
> Where are we with the smart_pointers branchs?
> 
> 
> And which of all the fix branches is ready to be merged to master?
> 
> 
> I also recall some of you guys should have removed some branches after
> verifying a merge (I think it was Jiri and Borek). Was it ever done?
> 
> 
> Itamar.
> 
> 
> On 3/11/2010 9:23 PM, Veit Jahns wrote:
> 
> > 2010/11/2 Itamar Syn-Hershko<it...@co...>:
> >> I'm waiting on some feedback from Veit.
> > Ups! It seems, that I've lost completely track of something. Can you
> > give me a clue, Itamar?
> >
> > Veit
> >
> > ------------------------------------------------------------------------------
> > Achieve Improved Network Security with IP and DNS Reputation.
> > Defend against bad network traffic, including botnets, malware,
> > phishing sites, and compromised hosts - saving your company time,
> > money, and embarrassment.   Learn More!
> > http://p.sf.net/sfu/hpdev2dev-nov
> > _______________________________________________
> > CLucene-developers mailing list
> > CLu...@li...
> > https://lists.sourceforge.net/lists/listinfo/clucene-developers
> >
> >
> 
> ------------------------------------------------------------------------------
> The Next 800 Companies to Lead America's Growth: New Video Whitepaper
> David G. Thomson, author of the best-selling book "Blueprint to a
> Billion" shares his insights and actions to help propel your
> business during the next growth cycle. Listen Now!
> http://p.sf.net/sfu/SAP-dev2dev
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers

[CLucene-dev] get a Snippet around a searched word

[muhammad i. <m.i...@gm...>]

Hello every body,
is it possible to retrieve a snippet around a searched word in clucence?

-- 
Sincerely
--------------------
Mohammad Ismael
Software Developer
Mobile:+20114753575

Re: [CLucene-dev] NearSpansUnordered bug fix

[Itamar Syn-H. <it...@co...>]

Where are we with the smart_pointers branchs?


And which of all the fix branches is ready to be merged to master?


I also recall some of you guys should have removed some branches after 
verifying a merge (I think it was Jiri and Borek). Was it ever done?


Itamar.


On 3/11/2010 9:23 PM, Veit Jahns wrote:

> 2010/11/2 Itamar Syn-Hershko<it...@co...>:
>> I'm waiting on some feedback from Veit.
> Ups! It seems, that I've lost completely track of something. Can you
> give me a clue, Itamar?
>
> Veit
>
> ------------------------------------------------------------------------------
> Achieve Improved Network Security with IP and DNS Reputation.
> Defend against bad network traffic, including botnets, malware,
> phishing sites, and compromised hosts - saving your company time,
> money, and embarrassment.   Learn More!
> http://p.sf.net/sfu/hpdev2dev-nov
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers
>
>

Re: [CLucene-dev] GermanStemmer.cpp compilation problem

[Veit J. <nun...@go...>]

2010/11/3 Veit Jahns <nun...@go...>:
> 2010/11/3 Kostka Bořivoj <ko...@to...>:
>> I got set of following (strange) warnings:
>>
>> GermanStemmer.cpp(118) : warning C4066: characters beyond first in wide-character constant ignored
>> GermanStemmer.cpp(121) : warning C4066: characters beyond first in wide-character constant ignored
>> GermanStemmer.cpp(124) : warning C4066: characters beyond first in wide-character constant ignored
>> GermanStemmer.cpp(128) : warning C4066: characters beyond first in wide-character constant ignored
>> GermanStemmer.cpp(144) : warning C4066: characters beyond first in wide-character constant ignored
>> GermanStemmer.cpp(183) : warning C4066: characters beyond first in wide-character constant ignored

Fixed the remaining warnings in the german_analyzer branch.

Veit

Re: [CLucene-dev] NearSpansUnordered bug fix

[Veit J. <nun...@go...>]

2010/11/2 Itamar Syn-Hershko <it...@co...>:
> I'm waiting on some feedback from Veit.

Ups! It seems, that I've lost completely track of something. Can you
give me a clue, Itamar?

Veit

Re: [CLucene-dev] GermanStemmer.cpp compilation problem

[Veit J. <nun...@go...>]

2010/11/3 Kostka Bořivoj <ko...@to...>:
> I got set of following (strange) warnings:
>
> GermanStemmer.cpp(118) : warning C4066: characters beyond first in wide-character constant ignored
> GermanStemmer.cpp(121) : warning C4066: characters beyond first in wide-character constant ignored
> GermanStemmer.cpp(124) : warning C4066: characters beyond first in wide-character constant ignored
> GermanStemmer.cpp(128) : warning C4066: characters beyond first in wide-character constant ignored
> GermanStemmer.cpp(144) : warning C4066: characters beyond first in wide-character constant ignored
> GermanStemmer.cpp(183) : warning C4066: characters beyond first in wide-character constant ignored
>
> during compile on Win XP with MS VS 2005
>
> All are related to lines, where german characters are used as constants in source
>
> Any idea what (if anything) could be wrong?

It is an encoding issuee. But did you compiled the GermanStemmer.cpp
from the master branch? A more recent version of GermanStemmer is in
the german_analyzer branch (I replaced the umlauts and the ß-ligature
with their corresponding code number). Howevver, with this version I
get warnings for the lines with the section sign (§), which I thought
is an ASCII sign, but isn't. I must somehow overlooked these warnings.
I will take care of this warning. Thanks for the hint.

Veit

[CLucene-dev] GermanStemmer.cpp compilation problem

[Kostka B. <ko...@to...>]

I got set of following (strange) warnings:

GermanStemmer.cpp(118) : warning C4066: characters beyond first in wide-character constant ignored
GermanStemmer.cpp(121) : warning C4066: characters beyond first in wide-character constant ignored
GermanStemmer.cpp(124) : warning C4066: characters beyond first in wide-character constant ignored
GermanStemmer.cpp(128) : warning C4066: characters beyond first in wide-character constant ignored
GermanStemmer.cpp(144) : warning C4066: characters beyond first in wide-character constant ignored
GermanStemmer.cpp(183) : warning C4066: characters beyond first in wide-character constant ignored

during compile on Win XP with MS VS 2005

All are related to lines, where german characters are used as constants in source

Any idea what (if anything) could be wrong?

Borek

Re: [CLucene-dev] NearSpansUnordered bug fix

[Itamar Syn-H. <it...@co...>]

Hi Jiri,

Unfortunately I'm having some trouble finding time to dedicate to 
clucene at the moment, so all the checks and merges I planned on doing 
are on hold. I hope to get to them in a week or two.

I looked on your branch - it contains one trivial fix. Please merge it 
to master yourself. As for the other branches, they contain more 
intrusive changes, and I'd like to asses their impact first. Also, I'm 
waiting on some feedback from Veit.

Itamar.

On 2/11/2010 6:03 PM, Šplíchal Jiří wrote:
>
> Hello,
>
> I have just added a new branch called SpanQuery_fix with a bug fix for 
> this class.
>
> The problem we found was:
>
> -if you run a SpanNearQuery with two SpanTermQueries as subqueries
>
> and the first of these SpanTermQueries did not match any document, then
>
> the global query still returned matching documents based on the second 
> subquery.
>
> I went once more through the java-code and I realized that I made a 
> mistake while porting this
>
> class.
>
> Itamar, could you please merge this branch with master and remove it 
> afterwards?
>
> Jiri
>
> PS: what about the memory_leaks branch?
>
> --
>
> Jiří Šplíchal
>
> TOVEK, spol. s r.o.
>
> spl...@to... <mailto:spl...@to...>
>
> +420 606671930
>
>
> ------------------------------------------------------------------------------
> Nokia and AT&T present the 2010 Calling All Innovators-North America contest
> Create new apps&  games for the Nokia N8 for consumers in  U.S. and Canada
> $10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
> Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
> http://p.sf.net/sfu/nokia-dev2dev
>
>
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers

[CLucene-dev] NearSpansUnordered bug fix

[Šplíchal J. <spl...@to...>]

Hello,

 

I have just added a new branch called SpanQuery_fix with a bug fix for this class.

The problem we found was: 

-          if you run a SpanNearQuery with two SpanTermQueries as subqueries 

and the first of these SpanTermQueries did not match any document, then 

the global query still returned matching documents based on the second subquery.

 

I went once more through the java-code and I realized that I made a mistake while porting this

class. 

 

Itamar, could you please merge this branch with master and remove it afterwards?

 

Jiri

 

PS: what about the memory_leaks branch?

 

--

Jiří Šplíchal

TOVEK, spol. s r.o.

spl...@to... <mailto:spl...@to...> 

+420 606671930

 

[CLucene-dev] strange behavior during building

[theorist <eg...@ma...>]

hello,

http://stackoverflow.com/questions/3952699/

any thoughts?

Re: [CLucene-dev] How to submit patch?

[Veit J. <nun...@go...>]

2010/10/25 Pankaj Jangid <pan...@gm...>:
> I have opened a ticket
> https://sourceforge.net/tracker/?func=detail&aid=3094661&group_id=80013&atid=558446

Thanks!

Ben/Itamar, if you want, you can assign this ticket to me. I will look at it.

Veit

[CLucene-dev] Terms are out of order precondition failing

[Matt R. <mr...@mr...>]

Hi,

I'm just getting started with CLucene and I'm running into a precondition that is failing. I grabbed the latest snapshot from Git (just a day or two ago), here is the error I'm getting:

file:/Users/mronge/Projects/clucene/src/core/CLucene/index/TermInfosWriter.cpp line:161
Terms are out of order: field=body (number 0) lastField=body (number 0) text= lastText=\377x

Here is the precondition:

    CND_PRECONDITION(compareToLastTerm(fieldNumber, termText, termTextLength) < 0 ||
      (isIndex && termTextLength == 0 && lastTermTextLength == 0),
      (string("Terms are out of order: field=") + Misc::toString(fieldInfos->fieldName(fieldNumber)) +
      " (number " + Misc::toString(fieldNumber) + ")" +
      " lastField=" + Misc::toString(fieldInfos->fieldName(lastFieldNumber)) +
      " (number " + Misc::toString(lastFieldNumber) + ")" +
      " text=" + Misc::toString(termText, termTextLength) +
      " lastText=" + Misc::toString(lastTermText.values, lastTermTextLength)
      ).c_str() );

From looking at compareToLastTerm it appears it returns 1 if the length termText is < length of lastTermText, so it fails here when termText is empty. Is this correct? Shouldn't it just return -1 when when termText is empty, since "empty" could be ordered in front of everything else?

Thanks!
Matt

Re: [CLucene-dev] How to submit patch?

[Pankaj J. <pan...@gm...>]

Thanks Veit.

I have opened a ticket
https://sourceforge.net/tracker/?func=detail&aid=3094661&group_id=80013&atid=558446

Attached the patch file there.

--
Thanks & Regards
Pankaj

On Fri, Oct 22, 2010 at 11:22 PM, Veit Jahns <nun...@go...>wrote:

> Hi Pankaj!
>
> > I want to submit those changes back to the clucene project. How can I do
> > that?
>
> Great! There two ways: You open a ticket in the bug tracker [1] of
> CLucene (where you can attach the patches) or you just send them to
> the mailing list. Itamar, I or someone other will take a look at it,
> review it and submit it to the repository.
>
> >  Are there any guidelines written?
>
> No, there are no guidelines. In my oppinion, it would be useful, if
> you add a short explaination, what caused the bug is and why do your
> patch fixed it. This will help to undertstand yor patch.
>
> Kind regards,
>
> Veit
>
> [1] http://sourceforge.net/tracker/?group_id=80013&atid=558446
>
>
> ------------------------------------------------------------------------------
> Nokia and AT&T present the 2010 Calling All Innovators-North America
> contest
> Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
> $10 million total in prizes - $4M cash, 500 devices, nearly $6M in
> marketing
> Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
> http://p.sf.net/sfu/nokia-dev2dev
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers
>

Re: [CLucene-dev] CLUCENE_INLINE_EXPORT and How to test contribs

[Veit J. <nun...@go...>]

2010/9/13 Itamar Syn-Hershko <it...@co...>:
> 2. I don't see why cl_test can't be linked to test the contribs as well.
> If you could update the CMake scripts to make this optional (the linkage
> to contribs, and the contrib tests), that would be ideal.

I ported the GermanAnalyzer and the respective test from JLucene. It
can be found in the branch german_analyer. The test for the
GermanAnalyzer will be linked to cl_test, if building  contrib-libs is
activated in CMake. Let me know, if you have comments on this.

Veit

Re: [CLucene-dev] How to submit patch?

[Veit J. <nun...@go...>]

Hi Pankaj!

> I want to submit those changes back to the clucene project. How can I do
> that?

Great! There two ways: You open a ticket in the bug tracker [1] of
CLucene (where you can attach the patches) or you just send them to
the mailing list. Itamar, I or someone other will take a look at it,
review it and submit it to the repository.

>  Are there any guidelines written?

No, there are no guidelines. In my oppinion, it would be useful, if
you add a short explaination, what caused the bug is and why do your
patch fixed it. This will help to undertstand yor patch.

Kind regards,

Veit

[1] http://sourceforge.net/tracker/?group_id=80013&atid=558446

[CLucene-dev] New memory leak found and fixed

[Kostka B. <ko...@to...>]

BufferedNorms pointer removed from norms array if empty but object not deleted
Fix pushed to memory_leaks branch
Borek

[CLucene-dev] Potential bug in SegmentTermEnum.cpp

[Rustem A. <ar...@gm...>]

Hi,

src/core/CLucene/index/SegmentTermEnum.cpp : line 377

[code]
        //Instantiate the new buffer + 1 is needed for terminator '\0'
        if ( buffer == NULL )
            buffer = (TCHAR*)malloc(sizeof(TCHAR) * (bufferLength+1));
        else
            buffer = (TCHAR*)realloc(buffer, sizeof(TCHAR) *
(bufferLength+1));

        if ( copy || force_copy){
            //Copy the text of term into buffer
            _tcsncpy(buffer,_term->text(),bufferLength);
        }
[/code]

If malloc / realloc return NULL?

[CLucene-dev] Similarity with constant idf

[Šplíchal J. <spl...@to...>]

Hi,

 

I have to change clucene scoring in the way that the number of documents that contain a certain term

does not influence the final score when querying documents using this term. 

In other words I have to make the score  of a document independent of content of the index - it should

be influenced only by the document itself and the query.

 

I understood that to achieve this it is enough to overwrite the Similarity and make the inverse document

frequency - idf constant. This seems to work fine. But I noticed that the base class Similarity contains two other

idf() functions which are not virtual - especially the float_t Similarity::idf(Term* term, Searcher* searcher) 

which retrieves the document frequency and calls the virtual idf() function to compute the result.

 

I was wondering whether we could make those two other functions also virtual so that I could save retrieving

the document frequency at all. I could just return the constant value. 

 

Jiri

 

 

[CLucene-dev] _Term.h included in CLucene/search/Query.h

[Pankaj J. <pan...@gm...>]

The _*.h files are not supposed to be shipped in the release package. But
_Term.h file is included in CLucene/search/Query.h to make use of class
Term_UnorderedCompare. Either we should move Term_UnorderedCompare to Term.h
or we should have an alternative in Query.h.

Clients using "clucene" get error "include file _Term.h not found" when they
include Query.h.


--
Regards
Pankaj

Re: [CLucene-dev] Fwd: Highlighter. UPD

[Kostka B. <ko...@to...>]

Ok

 

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Thursday, October 21, 2010 12:04 PM
To: clu...@li...
Subject: Re: [CLucene-dev] Fwd: Highlighter. UPD

 

Hi, yes and trailing quote

Test text - word1 word2 ("word3")
Search - word3
After getBestFragments: word1 word2 ("<b>word3</b> - miising ")

2010/10/21 Kostka Bořivoj <ko...@to...>

Hi,

 

I'm still not sure, what is the purpose of added lines. Adding trailing parenthesis to the end of fragment?

 

Borek

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Tuesday, October 12, 2010 4:57 PM
To: clu...@li...
Subject: Re: [CLucene-dev] Fwd: Highlighter. UPD

 

Sorry... monday....

            while (text[lastEndOffset + trail] == _T(')') || text[lastEndOffset + trail] == _T('"'))

or a previous version...

2010/10/12 Kostka Bořivoj <ko...@to...>

I think while is an infinite cycle in this version...

Borek

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Monday, October 11, 2010 5:25 PM


To: clu...@li...

Subject: Re: [CLucene-dev] Fwd: Highlighter. UPD

 

New version :)

            // append trailing ')' and '"' to snippet
            size_t trail = 0;
            while (text[lastEndOffset] == _T(')') || text[lastEndOffset] == _T('"'))
                ++trail;
            if (trail)
            {
                writeTo->append(text + lastEndOffset, trail);
                lastEndOffset += trail;
            }

2010/10/11 Kostka Bořivoj <ko...@to...>

OK, I'll look at it 

Borek

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Monday, October 11, 2010 11:17 AM
To: clu...@li...
Subject: [CLucene-dev] Fwd: Highlighter. UPD

 

Hi!


Please, append next block to code

src/contribs-lib/CLucene/highlighter/Highlighter.cpp

Highlighter::getBestTextFragments
Line: 315-317

            // append trailing ')' and '"' to snippet
            while (text[lastEndOffset] == _T(')') || text[lastEndOffset] == _T('"'))
            {
                writeTo->append(text + lastEndOffset, 1);
                ++lastEndOffset;
            }

            // append text after end of last token
            //if (lastEndOffset < (int32_t)_tcslen(text))
            //newText->append(text+lastEndOffset);

 


------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
CLucene-developers mailing list
CLu...@li...
https://lists.sourceforge.net/lists/listinfo/clucene-developers

 


------------------------------------------------------------------------------
Beautiful is writing same markup. Internet Explorer 9 supports
standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
Spend less time writing and  rewriting code and more time creating great
experiences on the web. Be a part of the beta today.
http://p.sf.net/sfu/beautyoftheweb
_______________________________________________
CLucene-developers mailing list
CLu...@li...
https://lists.sourceforge.net/lists/listinfo/clucene-developers

 


------------------------------------------------------------------------------
Nokia and AT&T present the 2010 Calling All Innovators-North America contest
Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
$10 million total in prizes - $4M cash, 500 devices, nearly $6M in marketing
Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
http://p.sf.net/sfu/nokia-dev2dev
_______________________________________________
CLucene-developers mailing list
CLu...@li...
https://lists.sourceforge.net/lists/listinfo/clucene-developers

 

Re: [CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuerytoitself

[Šplíchal J. <spl...@to...>]

I pushed the bug fix into the memory_leaks branch.

Itamar, do you plan to merge this branch with master?

Jiri

 

From: Šplíchal Jiří [mailto:spl...@to...] 
Sent: Thursday, October 21, 2010 12:41 PM
To: clu...@li...
Subject: Re: [CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuerytoitself

 

Hello, 

 

yes this is definitely a bug, thank you. I will correct it.

 

Jiri

 

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Thursday, October 21, 2010 12:14 PM
To: clu...@li...
Subject: [CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuery toitself

 

Hi,

src/core/CLucene/search/SpanQueryFilter.cpp : line 24

<code>
SpanQueryFilter::SpanQueryFilter( CL_NS2(search,spans)::SpanQuery * query, bool bDeleteQuery )
{
    this->query = query;
    bDeleteQuery = bDeleteQuery;
}
</code>

bDeleteQuery = bDeleteQuery;
Hmmm... this->bDeleteQuery = bDeleteQuery; ?

Bad style for variable names

Re: [CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuery toitself

[Šplíchal J. <spl...@to...>]

Hello, 

 

yes this is definitely a bug, thank you. I will correct it.

 

Jiri

 

 

From: Rustem Alimov [mailto:ar...@gm...] 
Sent: Thursday, October 21, 2010 12:14 PM
To: clu...@li...
Subject: [CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuery toitself

 

Hi,

src/core/CLucene/search/SpanQueryFilter.cpp : line 24

<code>
SpanQueryFilter::SpanQueryFilter( CL_NS2(search,spans)::SpanQuery * query, bool bDeleteQuery )
{
    this->query = query;
    bDeleteQuery = bDeleteQuery;
}
</code>

bDeleteQuery = bDeleteQuery;
Hmmm... this->bDeleteQuery = bDeleteQuery; ?

Bad style for variable names

[CLucene-dev] How to submit patch?

[Pankaj J. <pan...@gm...>]

Hi,

I use clucene for one project. For that we have to port it on various
platforms. To compile it properly on AIX, Linux, Solaris (x86 and sparc) I
had to make few small changes to the clucene code from HEAD.

I want to submit those changes back to the clucene project. How can I do
that? Are there any guidelines written?

--
Regards
Pankaj

[CLucene-dev] SpanQueryFilter.cpp - assignment of bDeleteQuery to itself

[Rustem A. <ar...@gm...>]

Hi,

src/core/CLucene/search/SpanQueryFilter.cpp : line 24

<code>
SpanQueryFilter::SpanQueryFilter( CL_NS2(search,spans)::SpanQuery * query,
bool bDeleteQuery )
{
    this->query = query;
    bDeleteQuery = bDeleteQuery;
}
</code>

bDeleteQuery = bDeleteQuery;
Hmmm... this->bDeleteQuery = bDeleteQuery; ?

Bad style for variable names

Re: [CLucene-dev] Fwd: Highlighter. UPD

[Rustem A. <ar...@gm...>]

Hi, yes and trailing quote

Test text - word1 word2 ("word3")
Search - word3
After getBestFragments: word1 word2 ("<b>word3</b> - miising ")

2010/10/21 Kostka Bořivoj <ko...@to...>

>  Hi,
>
>
>
> I'm still not sure, what is the purpose of added lines. Adding trailing
> parenthesis to the end of fragment?
>
>
>
> Borek
>
>
>
> *From:* Rustem Alimov [mailto:ar...@gm...]
> *Sent:* Tuesday, October 12, 2010 4:57 PM
> *To:* clu...@li...
> *Subject:* Re: [CLucene-dev] Fwd: Highlighter. UPD
>
>
>
> Sorry... monday....
>
>             while (text[lastEndOffset + trail] == _T(')') ||
> text[lastEndOffset + trail] == _T('"'))
>
> or a previous version...
>
>  2010/10/12 Kostka Bořivoj <ko...@to...>
>
> I think while is an infinite cycle in this version...
>
> Borek
>
>
>
> *From:* Rustem Alimov [mailto:ar...@gm...]
> *Sent:* Monday, October 11, 2010 5:25 PM
>
>
> *To:* clu...@li...
>
> *Subject:* Re: [CLucene-dev] Fwd: Highlighter. UPD
>
>
>
> New version :)
>
>             // append trailing ')' and '"' to snippet
>             size_t trail = 0;
>             while (text[lastEndOffset] == _T(')') || text[lastEndOffset] ==
> _T('"'))
>                 ++trail;
>             if (trail)
>             {
>                 writeTo->append(text + lastEndOffset, trail);
>                 lastEndOffset += trail;
>             }
>
> 2010/10/11 Kostka Bořivoj <ko...@to...>
>
> OK, I'll look at it
>
> Borek
>
>
>
> *From:* Rustem Alimov [mailto:ar...@gm...]
> *Sent:* Monday, October 11, 2010 11:17 AM
> *To:* clu...@li...
> *Subject:* [CLucene-dev] Fwd: Highlighter. UPD
>
>
>
> Hi!
>
>
> Please, append next block to code
>
> src/contribs-lib/CLucene/highlighter/Highlighter.cpp
>
> Highlighter::getBestTextFragments
> Line: 315-317
>
>             // append trailing ')' and '"' to snippet
>             while (text[lastEndOffset] == _T(')') || text[lastEndOffset] ==
> _T('"'))
>             {
>                 writeTo->append(text + lastEndOffset, 1);
>                 ++lastEndOffset;
>             }
>
>             // append text after end of last token
>             //if (lastEndOffset < (int32_t)_tcslen(text))
>             //newText->append(text+lastEndOffset);
>
>
>
>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
> Spend less time writing and  rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers
>
>
>
>
>
> ------------------------------------------------------------------------------
> Beautiful is writing same markup. Internet Explorer 9 supports
> standards for HTML5, CSS3, SVG 1.1,  ECMAScript5, and DOM L2 & L3.
> Spend less time writing and  rewriting code and more time creating great
> experiences on the web. Be a part of the beta today.
> http://p.sf.net/sfu/beautyoftheweb
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers
>
>
>
>
> ------------------------------------------------------------------------------
> Nokia and AT&T present the 2010 Calling All Innovators-North America
> contest
> Create new apps & games for the Nokia N8 for consumers in  U.S. and Canada
> $10 million total in prizes - $4M cash, 500 devices, nearly $6M in
> marketing
> Develop with Nokia Qt SDK, Web Runtime, or Java and Publish to Ovi Store
> http://p.sf.net/sfu/nokia-dev2dev
> _______________________________________________
> CLucene-developers mailing list
> CLu...@li...
> https://lists.sourceforge.net/lists/listinfo/clucene-developers
>
>