@b4n commented on this pull request.


> @@ -265,11 +264,9 @@ void tpage_pack_widgets(gboolean tabbed)
 
        if (oldroot)
        {
-               int i = 0;
-               while (widgets[i])
+               for (int i = 0; widgets[i]; i++)
                {
                        g_object_unref(*widgets[i]);

Is cppcheck happy with that?  Actually I'm still confused on what it didn't 
like before, so maybe it's happy. It's correct anyway so whichever way's OK 
with me.

-- 
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/1017#pullrequestreview-515500152

Reply via email to