Richard Heck wrote:

    By the way, are we sure we want the "search as you go" functionality
    currently in the citation dialog? There is notable slowness on my
    machine (Athlon 2400+, 1GB), as I've got 500+ BibTeX keys in the dialog
    and they all have to be searched on every keypress.
In svn, the search is restrained to the already searched list so only
the first keypress search through the full list. Maybe this has
changed with you patch.
No, I didn't change that. I did mis-state this, though. So consider:
Suppose I do a case-insensitive search and type an "s". Then the search
will narrow the list down to everything that contains "s" somewhere in
the record. That is not likely to narrow it down very much. (Remember
that it doesn't just search the keys. Maybe it should.)

No because some users complained that they didn't care much about the keys.

So now I type an
"a". That narrows it down to everything that contains "sa". Again,
little progress. Now I type "m". That will weed out a few things, but
possibly not very many, especially because I have a lot of annote,
abstract, and review fields in the database. (Note here that the current
BibTeX parser loads all fields and searches all fields.) So there are
going to be three pretty much full DB searches, and the fourth one will
likely be quite large as well. Of course, I've chosen a bad case. If I'd
type "fre", I'd have made more progress. But still it is slow.

I see. I just want to say that I tested this extensively at the time I implemented it (that was before the new parser) and the speed was OK even with a _lot_ of entries. My test case was to load all Bibtex files that come with Miktex.

If the speed is now a problem, I guess a check box "Find as you type" which default to yes is necessary.

Abdel.

Reply via email to