This pull request adds some patches on top of #767 and deprecates the original pull request.
@frlan If it looks OK to you, would you please merge it? You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/783
-- Commit Summary --
* Add file operations to Project Organizer. * Eliminate Yoda style conditions to match the style of the rest of the code * Eliminate begin/end braces around 1-line blocks to match used code style * Remove debug prints * Use locale encoding for file operations instead of utf8 * Make it more clear what's going on with "sentinel" * Remove unnecessary config writes * g_free() related cleanups * Some string literal wording changes * Make the "Delete" entry in the popup insensitive for non-empty directories * Add some TODOs
-- File Changes --
M projectorganizer/src/prjorg-project.c (11) M projectorganizer/src/prjorg-project.h (5) M projectorganizer/src/prjorg-sidebar.c (231) M projectorganizer/src/prjorg-utils.c (77) M projectorganizer/src/prjorg-utils.h (7)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/783.patch https://github.com/geany/geany-plugins/pull/783.diff
Merged #783 into master.
Since:
1. @techee is maintainer for this plugin 2. this builds for me and travis 3. enabling and disabling the plugin does not crash Geany (but not being a user thats all I can test)
merged.
Ah, I completely forgot about this pull request and actually thought it was merged a long time ago. Thanks for merging.
github-comments@lists.geany.org