Re: [EasyHack] #44681 port to CLucene from java/Lucene

2012-02-21 Thread G.H.M.Valkenhoef, van
Great to hear that it works & that it will make it into master :-). Any thoughts on this one:  3) When creating the CLucene FileReader (HelpIndexer.cxx), the path is converted to plain ASCII, that's probably dangerous. There is probably a way to work around this, but I haven't gotten around to

License for my contributions

2012-02-20 Thread G.H.M.Valkenhoef, van
Hi all, I hereby declare that my past and future contributions to LibreOffice are licensed under the LGPL v3+ and the MPL v1.1. Best wishes, Gert van Valkenhoef ___ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.or

Re: [EasyHack] #44681 port to CLucene from java/Lucene

2012-02-15 Thread G.H.M.Valkenhoef, van
Great, thanks. That looks like it fills in most of the missing pieces from my patch. On 15-02-12, Caolán McNamara wrote: > On Tue, 2012-02-14 at 12:01 +, Caolán McNamara wrote: > > On Tue, 2012-02-14 at 09:45 +0100, G.H.M.Valkenhoef, van wrote: > > > > > > &g

Re: [EasyHack] #44681 port to CLucene from java/Lucene

2012-02-14 Thread G.H.M.Valkenhoef, van
On 14-02-12, Andras Timar wrote: > 2012/2/14 Norbert Thiebaud : > > Just a word of encouragement: thanks for working on that, I'm looking > > forward to see the impact on the build with LANG=all :-) > > Me too, and also I wonder, if it fixes > https://bugs.freedesktop.org/show_bug.cgi?id=40665

Re: [EasyHack] #44681 port to CLucene from java/Lucene

2012-02-14 Thread G.H.M.Valkenhoef, van
Thanks for the answers, responses below. On 13-02-12, Caolán McNamara wrote: > The xmlhelp/source/com/sun/star/help/*.java route is the one that sets > the "bExtensionMode". I think this one is which third party extensions > can insert their help into our help system. The cxxhelp is the > straig

Re: [EasyHack] #44681 port to CLucene from java/Lucene

2012-02-14 Thread G.H.M.Valkenhoef, van
On 13-02-12, Rene Engelhard wrote: > Hi, > > first of all: thanks for this effort, which should have been done when this > lucene dependency was introduced in the first place in OOo times. > > But... > > On Mon, Feb 13, 2012 at 04:17:49PM +0100, Radek Doulik wrote: > > >   * This implementatio