|
From: | Paolo Bonzini |
Subject: | bug#16481: dfa.c and Rational Range Interpretation |
Date: | Tue, 11 Feb 2014 22:44:09 +0100 |
User-agent: | Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.2.0 |
Il 11/02/2014 22:42, Paul Eggert ha scritto:
Ouch, I wasn't aware of this. Clearly the patch I put in was wrong -- at least for the documentation that got put into NEWS.
Yeah, sorry for not spelling it out entirely. I worked on grep in bursts, and as a result I tend to take too many things for granted.
Perhaps you're right, and the best thing to do for now is to revert the patch while we can think about a better solution. This should be done soon, since Jim wants to do a grep release. Please let me think about it for a day or two. I would like to fix the bug, anyway, even if that patch wasn't the right way to do it. Longer term, it'd be better to simplify the code (perhaps along the lines that you suggested) as it's too full of gotchas now.
I 100% agree with this. If I don't hear from you I'll revert the patch next Friday.
Paolo
[Prev in Thread] | Current Thread | [Next in Thread] |