[Dnsmasq-discuss] [PATCH] Inotify: Ignore backup files created by editors

Andy Hawkins andy at gently.org.uk
Thu Feb 15 17:55:33 GMT 2018


Hi,

In article <slrnp8bfuo.vn.andy at xcp-mailnews.gently.org.uk>,
           Andy Hawkins<andy at gently.org.uk> wrote:
> I'll take a look at your version of the code in git and see how it looks
> compared to mine.

Ok, things are a bit clearer after looking at the actual commit.

My only comment (and it's a fairly personal one) is that I generally dislike
seeing variable assignments inside if statements. However, I can see the
reason for doing it in this case, as if you carried out the strlen outside
the 'if', then you'd need to put that in an 'if' of its own, checking that
in->len wasn't zero.

Thanks for including the fix however, appreciated. What are the chances of
this ending up in a package in the current debian stable?

Thanks again

Andy




More information about the Dnsmasq-discuss mailing list