Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GeanyPy: fix Document.text_changed #526

Closed
wants to merge 1 commit into from
Closed

GeanyPy: fix Document.text_changed #526

wants to merge 1 commit into from

Conversation

vfaronov
Copy link
Contributor

@kugel- @codebrainz Mind having a look? I assume that Py_RETURN_NONE here was a typo or copy-paste error.

@codebrainz
Copy link
Member

Must be, upstream doesn't have this typo.

@kugel-
Copy link
Member

kugel- commented Feb 16, 2017

Apprently codebrainz/geanypy@b714087 (and maybe others) was never merged into g-p

@codebrainz
Copy link
Member

Git Submodules FTW!

@codebrainz
Copy link
Member

For diff -bur geanypy geany-plugins/geanypy output see here.

@codebrainz
Copy link
Member

After bringing in fixes and a few small changes to reduce diff noise:

https://gist.github.com/codebrainz/990345f4010523a49a9556b3160171f1#file-gpy-gpgpy-after-backports-diff

codebrainz added a commit to codebrainz/geany-plugins that referenced this pull request Feb 17, 2017
@frlan frlan closed this in #527 Feb 19, 2017
@vfaronov vfaronov deleted the geanypy/fix-text-changed branch February 19, 2017 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants