Thread (126 messages) flat view 126 messages, 3 authors, 2017-01-09

Re: [PATCH 02/17] dir: convert create_simplify to use the pathspec struct interface

From: Duy Nguyen <hidden>
Date: 2016-12-07 10:18:18

On Wed, Dec 7, 2016 at 4:51 AM, Brandon Williams [off-list ref] wrote:
Convert 'create_simplify()' to use the pathspec struct interface from
using the '_raw' entry in the pathspec.
It would be even better to kill this create_simplify() and let
simplify_away() handle struct pathspec directly.

There is a bug in this code, that might have been found if we
simpify_away() handled pathspec directly: the memcmp() in
simplify_away() will not play well with :(icase) magic. My bad. If
:(icase) is used, the easiest/safe way is simplify nothing. Later on
maybe we can teach simplify_away() to do strncasecmp instead. We could
ignore exclude patterns there too (although not excluding is not a
bug).
-- 
Duy
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help