Re: rw_match() bug

[Martin Sebor <se...@roguewave.com>]

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