Marc Slemko wrote:
[...]
>and note how the code a bit later on uses either r->uri or test_location
>very carefully depending on if it is locationmatch or location. This is
>this way on purpose, and is documented beavior.
>
>It is annoying and perhaps could have been done better if starting from
>ground zero, but there are no easy answers to dealing with such things...
>
I agree; the current semantics aren't what I'd choose if starting over,
but it's better to not break the documented behavior. I think I can
eliminate the string-copying in a way that preserves the current
behavior, though. If it works, I'll post a new patch later in the
week.
--Brian
