Timeout for a simple case about comby HOT 4 CLOSED

The pattern above is trying to match the whole file before the first = __builtin_annot_intval probably ?

Yes :) Others have run into this before so you're not alone. I think the right move might be to emit a warning for this sort of pattern. Tracking in #146. I added a little note on this in the docs:

You almost never want to start a template with :[hole], since it matches everything including newlines up to its suffix. This can make things slow. :[hole] is typically useful inside balanced delimiters.

I guess it should still find something ?

It probably would, if the timeout is higher (set with -timeout 10 for 10 seconds). There is a performance issue related to this, which would make this operation faster, but matching everything up to the = is still probably not the intended operation. Hope that helps.

from comby.

It does help ! I fixed my pattern, happy to hear a warning will be emitted and the doc improving.
Thank you for your reactivity, Comby's a great project !

from comby.

Addressed in #147.

from comby.

Wait it's just me being dumb isn't it ? The pattern should be

:[[x]] = __builtin_annot_intval("symb_int", :[[x]]);

The pattern above is trying to match the whole file before the first = __builtin_annot_intval probably ?

I don't know if the fact that it times out is really an issue then. I guess it should still find something ?
Sorry for submitting an issue if it is not one, I'll let you decide if that should be closed.

from comby.