PatternMatchingEventHandler behavior

[BoboTiG]

I was trying to ignore several patterns using the PatternMatchingEventHandler class. In the dispatch method, we have that behavior:

...
if match_any_paths(paths, **kwargs):
    self.on_any_event(event)
   ...

It means that if I want to ignore .tmp files and if I rename a file from test.txt to test.tmp, then the event handler will still take into account that change. That is because we filter on all paths and if one is not ignored, then it considers the event as a one.

I think, if I understand correctly, the this is not the desired behavior.

On the other hand, if we check the dispatch method of RegexMatchingEventHandler:

...
if any(r.match(p) for r in self.ignore_regexes for p in paths):
    return

if any(r.match(p) for r in self.regexes for p in paths):
    self.on_any_event(event)
   ...

Clearly, here, if one of the paths is ignored, then we stop now. That is the wanted behavior.

So, should I send a PR to fix PatternMatchingEventHandler behavior? Perhaps I misunderstand how to use that class or its purpose, let me know.