On 10.08.2014 21:17, stef...@apache.org wrote:
> Author: stefan2
> Date: Sun Aug 10 19:17:53 2014
> New Revision: 1617148
[...]
> @@ -973,7 +869,11 @@ add_complex_matches(lookup_state_t *stat
> for (i = 0; i < patterns->nelts; ++i)
> {
> node_t *node = APR_ARRAY_IDX(patterns, i, node_t *);
> - if (match_pattern(segment->data, node->segment.data))
> +
> + /* There should be not slashes and periods should be treated as
> + * literals. */
> + if (APR_SUCCESS == apr_fnmatch(node->segment.data, segment->data,
> + APR_FNM_PATHNAME | APR_FNM_PERIOD))
> add_next_node(state, node);
I assume there's no way to get slashes in a segment in any case, right?
So APR_FNM_PATHNAME is redundant. I wonder however why you think that we
need APR_FNM_PERIOD?
Oh, and BTW: please don't compare to APR_SUCCESS. We don't use that to
check the results of strcmp, either.
-- Brane
--
Branko Čibej | Director of Subversion
WANdisco | Realising the impossibilities of Big Data
e. br...@wandisco.com
