b4n commented on this pull request.
Fix looks sensible :)
I'm not fond of the whitespace changes in the same commit, and there's a slight
style "issue" (see comment), but I don't mind too much either, so I'll leave
the call to you.
> if (err)
+ {
+ #ifdef DEBUG_OUTPUT
style: the preproc directives used not to be indented
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/pull/511#pullrequestreview-13460822
