Re: [PATCH v2 4/8] scripts: add glimpse.sh for indexing the kernel

From: Julia Lawall
Date: Sat Jun 18 2016 - 06:38:14 EST


On Sat, 18 Jun 2016, Julia Lawall wrote:

> Overall, idutils seems to be a good choice. As compared to a grep based
> solution, it knows what is code, so it doesn't report on files where the
> words of interest only occur in comments. As compared to glimpse, it
> knows that foo_bar is a single word. Indexing is faster than with
> glimpse, and looking things up in the index is also slightly faster, even
> though Coccinelle needs to make multiple calls because it doesn't support
> complex formulas. It does support regexps, which could perhaps be even
> faster, but since the running time currently is mostly under 1 second and
> often under .1 seconds, it probably doesn't matter.

I'm not suggesting that idutils should be the default. Only that someone
who wants to go to the trouble of indexing could find that idutils is a
good choice.

julia