This pull request is heavily based on #645 from which it borrows the majority of the changes. The main improvement is that this one doesn't break compatibility with GTK2 (there are macros which select the correct code path accordingly). I also removed some cosmetic-only changes to maintain the differences as small as possible and provided a corrected code for console font selection (which was just commented out in the original pull request).
This was successfully tested on Centos 7.5 with stock development packages. You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-plugins/pull/790
-- Commit Summary --
* Ported the 'Debugger' plugin to GTK3 while maintaining compatibility with GTK2.
-- File Changes --
M build/debugger.m4 (5) M debugger/src/btnpanel.c (25) M debugger/src/cell_renderers/cellrendererbreakicon.c (73) M debugger/src/cell_renderers/cellrendererbreakicon.h (11) M debugger/src/cell_renderers/cellrendererframeicon.c (62) M debugger/src/cell_renderers/cellrendererframeicon.h (7) M debugger/src/cell_renderers/cellrenderertoggle.c (9) M debugger/src/dconfig.c (68) M debugger/src/debug.c (49) M debugger/src/dpaned.c (6) M debugger/src/gui.c (4) M debugger/src/plugin.c (4) M debugger/src/stree.c (4) M debugger/src/tpage.c (84)
-- Patch Links --
https://github.com/geany/geany-plugins/pull/790.patch https://github.com/geany/geany-plugins/pull/790.diff
Closed #790.
Why was this closed and #791 opened? If it's because you wanted to make changes, you can just push them to existing branch and the pull request will update.
Yes I'm aware, buy I somehow managed to completely break my forked repository, and it was then a mess to sort out, with tons of changes from the master branch... Sorry for the inconvenience.
No worries, I mostly wanted to link the two issues and point out (because a lot of people don't realize) that you can just force push your changes to the existing PR (even if you messed it up badly).
github-comments@lists.geany.org