Re: [CLucene-dev] 0.8.6

[<rg...@sd...>]

>>>>> "amigo" == amigo  <am...@ma...> writes:
amigo> 
amigo> Wonderful news Ben !!
amigo> Thanks for taking the time to implement MFQP, I cant wait to
amigo> try that out.
amigo> 
amigo> I still need to sort this PHP issue where results/hits dont
amigo> show and also now I noticed some lag for example if I click
amigo> Optimize in cltest.php sometimes it takes long time to execute,
amigo> or if you do Index to create a new document the first time you
amigo> do it, it's lagging for a minute or two and then fails to add
amigo> but says it did add a document?!
amigo> This did not use to happen prior to adding resources or
amigo> whatever other changes have been made after Rob did his patch
amigo> (with Rob's patch on the old version it works just fine)
amigo> 
amigo> On this side there's really nothing to php_clucene.c, I've just
amigo> added an extra long paremeter infront of most functions to
amigo> include the resource integer, and on the clucene.cpp wrapper
amigo> I've added ifdefs wherever there's a call to a _UNICODE
amigo> function, so for example instead of:
amigo> 
amigo> wchar_t* wfield = Misc::charToWide(field);
amigo> 
amigo> I just do a quick hack:
amigo> 
amigo> char_t* wfield = strdup(field);
amigo> 
amigo> Now this works fine, or at least it used to be fine in the
amigo> prior versions before the resource business, and now I'm not
amigo> sure if this is correct anymore, but it does work even with
amigo> resources (which shouldnt really make a difference)
amigo> 
amigo> Any insight would be appreciated...

I'm not sure everything that's going on here because I haven't read up
on what these resources are or looked at how we do them.

But you should note that Misc::charToWide(const char* s) allocates a
new string with new[], and the call to strdup() allocates one with
malloc(), which could cause problems later on in the processing.

--Rob