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

Rename GeanyLipsum to just Lipsum #472

Merged
merged 6 commits into from
Aug 10, 2016

Conversation

frlan
Copy link
Member

@frlan frlan commented Aug 6, 2016

There is no need for Geany prefix on plugins.

config_file = g_strconcat(geany->app->configdir,
G_DIR_SEPARATOR_S, "plugins", G_DIR_SEPARATOR_S,
"geanylipsum", G_DIR_SEPARATOR_S, "lipsum.conf", NULL);

#ifndef G_OS_WIN32
/* We try only to move if we are on not Windows platform */
config_dir_old = g_strconcat(geany->app->configdir, G_DIR_SEPARATOR_S,
Copy link
Member

Choose a reason for hiding this comment

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

could use g_build_filename() instead for these.

Copy link
Member

Choose a reason for hiding this comment

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

yep, would probably be neater

@frlan frlan force-pushed the feature/rename_geanylipsum branch from e565d8a to a129fc4 Compare August 7, 2016 20:05
@@ -1,12 +1,12 @@
GeanyLipsum
Lipsum
-----------
Copy link
Member

Choose a reason for hiding this comment

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

could change the underline length too (not necessary though)

@b4n
Copy link
Member

b4n commented Aug 7, 2016

Otherwise LGBI

if (dialogs_show_question(
_("Renamed plugin detected!\n"
"\n"
"GeanyLipsum has been renamed to Lipsum -- you surely have "
Copy link
Member

Choose a reason for hiding this comment

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

as you will have already recognised

@frlan frlan force-pushed the feature/rename_geanylipsum branch from a129fc4 to 7f33eb6 Compare August 10, 2016 18:33
@@ -0,0 +1,9 @@
AC_DEFUN([GP_CHECK_GEANYLIPSUM],
Copy link
Member

Choose a reason for hiding this comment

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

you could want to rename that (and in the configure.ac too).

@b4n
Copy link
Member

b4n commented Aug 10, 2016

Otherwise LGBI

@frlan frlan force-pushed the feature/rename_geanylipsum branch from 7f33eb6 to 9abd912 Compare August 10, 2016 20:31
@frlan frlan merged commit d4ada5c into geany:master Aug 10, 2016
@frlan frlan deleted the feature/rename_geanylipsum branch January 5, 2017 09:34
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