Bert Huijben wrote:
> I haven't reviewed all of this, but I'm wondering why in your patch
> [[
> def parse_path(self):
> - path = self.parse_line('Node-path: (.+)$', required=False)
> - if not path and self.lines[self.current] == 'Node-path: \n':
> - self.current += 1
> - path = ''
> + path = self.parse_line('Node-path: (.*)$', required=False)
> ]]
>
> This replaces '.+' with '.*'
>
> I was wondering why this was necessary?
That's just a bug fix. The empty path '' is a valid path. (A path may start
with a slash or not.) This part of the parser was simply wrong, before. It was
only used in a few tests and hadn't been used on a root path yet.
Thanks for looking.
- Julian
