Martin Sebor wrote:
[...]
Hmm. There are two possible interpretations of the third argument
to rw_match():
1. it gives the maximum number of characters to compare (i.e.,
after all directives have been expanded)
2. it denotes the number of characters in the first argument
(i.e., before the expansion of any directives).
It looks like currently we do (2). I missed this and made changes
to implement (1). Needless to say, even though my changes are "good",
they cause many failures in the test suite. I need to go back and
implement (1). It shouldn't be too hard but it means that I won't
be able to commit my changes tonight (what I have is attached so you
can see where I am).
OTOH, (1) seems to make more sense to me. What do you think? If
you agree, how difficult do you think it would be to switch the
string tests to using it instead of (2)?
Martin
