[Github-comments] [geany/geany-plugins] Sync with latest upstream (#527)

Matthew Brush notifications at xxxxx
Fri Feb 17 05:47:42 UTC 2017


codebrainz commented on this pull request.



> @@ -62,7 +62,7 @@ Editor_get_property(Editor *self, const gchar *prop_name)
 		PyObject *py_doc;
 		py_doc = (PyObject *) Document_create_new_from_geany_document(
 									self->editor->document);
-		if (py_doc && py_doc != Py_None)
+		if (!py_doc || py_doc == Py_None)

I didn't look into this difference, I just picked the kind of condition that made the branch execute when it seemed plausible to 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/527#pullrequestreview-22426532
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.geany.org/pipermail/github-comments/attachments/20170216/80051e1d/attachment.html>


More information about the Github-comments mailing list