[Github-comments] [geany/geany-plugins] Debugger: Move debug output accessing after check whether err != Null (#511)

[Colomban Wendling]

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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20161218/d960824f/attachment.html>