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

geanydoc: Current word as initial text in interactive mode #457

Merged
merged 1 commit into from
Jan 3, 2017
Merged

geanydoc: Current word as initial text in interactive mode #457

merged 1 commit into from
Jan 3, 2017

Conversation

vfaronov
Copy link
Contributor

@vfaronov vfaronov commented Jul 9, 2016

This makes the plugin more useful for namespaced languages like Python, where you often want to tweak the words a bit (e.g. xyzzysomelib.xyzzy) before looking them up.

dialog = create_Interactive();
dialog = create_Interactive(default_word);
if (default_word)
g_free(default_word);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

g_free() (like free()) is NULL-safe, no need to guard it

This makes the plugin more useful for namespaced languages
like Python, where you often want to tweak the words a bit
(e.g. "xyzzy" -> "somelib.xyzzy") before looking them up.
@vfaronov
Copy link
Contributor Author

vfaronov commented Jul 9, 2016

@b4n Fixed, thanks.

@b4n
Copy link
Member

b4n commented Jul 9, 2016

Code LGBI, but I don't know the plugin at all.

@vfaronov
Copy link
Contributor Author

vfaronov commented Jul 9, 2016

@b4n I’m not sure what’s the workflow on this project. Should I mail Yura Siamashka, who is listed in MAINTAINERS, and ask him to have a look?

@b4n
Copy link
Member

b4n commented Jul 9, 2016

@vfaronov ideally yes he'd have a look at it

@vfaronov
Copy link
Contributor Author

@b4n I mailed Yura a couple weeks ago, but he has not responded.

@frlan frlan merged commit 3f4de15 into geany:master Jan 3, 2017
@vfaronov vfaronov deleted the geanydoc/interactive-current-word branch February 15, 2017 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants