Scintilla will be changing the key/mouse binding for rectangular selection, as per:
https://groups.google.com/d/msg/scintilla-interest/kZcE8firI3A/dsBn0joZAwAJ
Since Geany doesn't support multiple selections, and almost every X11 window manager captures <kbd>Alt</kbd>+drag for moving a window (has saved my bacon on a number of occasions), before this change lands in Geany, I propose we change Scintilla's new default back to the existing behaviour. While it's annoying that the Windows and non-Windows behaviour would remain inconsistent, it will probably save a lot of bug reports about this feature not working, and avoid having to change any related documentation as well.
Ideally anyone interested would make this user configurable (if it is already, I couldn't find it) in the process.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/1568
New feature with new icon on toolbar & in the menu.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1471
-- Commit Summary --
* Add related icons to project
* Add to ui 'Reload all' feature
* Add shortcat key for 'Reload all' feature
* Modify reload algorithm according to new feature
* Insert some rows to doc file about new feature
-- File Changes --
M data/geany.glade (17)
M doc/geany.txt (3)
M icons/16x16/Makefile.am (3)
A icons/16x16/geany-reload-all.png (0)
M icons/24x24/Makefile.am (3)
A icons/24x24/geany-reload-all.png (0)
M icons/32x32/Makefile.am (3)
A icons/32x32/geany-reload-all.png (0)
M icons/48x48/Makefile.am (3)
A icons/48x48/geany-reload-all.png (0)
M icons/scalable/Makefile.am (3)
A icons/scalable/geany-reload-all.svg (682)
M icons/tango/16x16/Makefile.am (3)
A icons/tango/16x16/geany-reload-all.png (0)
M icons/tango/24x24/Makefile.am (3)
A icons/tango/24x24/geany-reload-all.png (0)
M icons/tango/32x32/Makefile.am (3)
A icons/tango/32x32/geany-reload-all.png (0)
M icons/tango/48x48/Makefile.am (3)
A icons/tango/48x48/geany-reload-all.png (0)
M icons/tango/scalable/Makefile.am (3)
A icons/tango/scalable/geany-reload-all.svg (682)
M src/callbacks.c (20)
M src/callbacks.h (2)
M src/keybindings.c (16)
M src/keybindings.h (3)
M src/toolbar.c (2)
M src/ui_utils.c (7)
M src/ui_utils.h (1)
-- Patch Links --
https://github.com/geany/geany/pull/1471.patchhttps://github.com/geany/geany/pull/1471.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1471
![select1](https://cloud.githubusercontent.com/assets/7548378/11047114/cc867938-8728-11e5-8ab5-c2202e02ecab.png)
![select2](https://cloud.githubusercontent.com/assets/7548378/11047115/ccb51d10-8728-11e5-85c2-8b52523e371f.png)
When I select one character, the <b>sel</b>ection count and the <b>col</b>umn number are correct, but when I extend the selection by one character, the <b>sel</b>ection increases by 4 not 1, although the <b>col</b>umn number only increases by the expected value of 1.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/745
Alright, hopefully my last Colomban-killer patchset. I tried to sync as much as possible from ctags/main here (no big parser changes at this point, only those caused by some change in main).
I'm not sure how much it's worth reviewing patch by patch (maybe except those patches that touch some Geany code or parsers) - it's much easier to diff this against universal-ctags and see the differences - there just a few of them. I made some minor changes to uctags as well so better to diff against https://github.com/techee/ctags/tree/geany_sync2 which contains those changes.
I tried to librarify ctags a bit by adding CTAGS_LIB macro by which I protect the library-specific stuff. There are however still some things which are either missing in uctags or which we should change in Geany:
* varType missing in uctags
* GRegex vs GNU regex - I kept GRegex in lregex.c (but synced all the rest) but we should think whether not to convert back to GNU regex
* isIgnoreToken() works differently than in uctags. If it simply just dropped certain tags, we could move the logic into TM but right now, it helps the C parser with parsing by suggesting whether the contents of braces should be dropped too. I'm not sure if we need to preserve this functionality (never used the ignored.tags file myself) but if we move to the new cxx parser we'd have to keep this diff unless there's some interest for this in uctags.
* lcpp contains lots of changes as it's tightly related to c.c. However, as it's used by c.c only and quite independent of the rest, if we move to the cxx parser we could keep a separate copy of lcpp for the obsolete c.c parser.
Except 829ea7d I tried to do all the work piece by piece so nothing should be lost. There wasn't much work on parsers except the mentioned patch where parsers using nestlevel had to be updated to its new API and using cork (which works so every uctags parser could be now ported to Geany).
I think it's too late for Geany 1.29 but it would be good to have it ready for early 1.30 so it can be tested during the whole release cycle.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1263
-- Commit Summary --
* Grab the complete uctags vString implementation
* Add generic pointer array
* Eliminate uses of g_stat()
* Drop some more unused functions from routines.c/h
* Remove most uses of glib calls
* Remove unused functions from options.c/h
* Grab uctags implementation of strlist
* Grab uctags keyword.c/h and add types.h with type declarations
* Make tempFile() return MIO
* Make combinePathAndFile() return char * instead of vString
* Use parse separator utilities in routines.c
* Make sure that file extension is taken from file (not preceding directory)
* Use uctags implementation of strstr()
* Don't initialize ExecutableProgram and ExecutableName
* Sync the rest of routines.c/h
* Grab uctags ctags.h and add repoinfo.c/h
* Grab args.c/h uctags implementation
* Define gcc attributes inside gcc-attr.h
* Move debug related code to debug.c/h
* Remove all unused code from main.c
* Grab all MIO changes from uctags
* Implement iFileGetLine() using gets()
* Grab ctags version of general.h and make related changes
* Grab ctags implementation of sort.c/h
* Create geany.c/h and put isIgnoreToken()
* Sync options.c/h (and introduce a lot of new garbage)
* Grab uctags kind.c/h
* Some sync of lcpp.c/h
* Move eTagFile from entry.h to entry.c
* Sync the beginning of entry.c/h
* entry: Take over uctags tag writing code
* entry: Some more or less formal changes
* entry: make makeTakEntry() implementation closer to uctags
* entry: Sync initTagEntry() code
* entry: Add the remaining code from uctags
* read: change macros into functions and move them into c
* c.c, lcpp.c: Avoid using File.mio
* read: move sInputFileInfo, sInputFile and File to read.c
* read: Add some extra data structures
* read: use makeFileTag()
* read: Some function renaming
* read: implement openBuffer() using openInputFile()
* read: Add function to initialize common parts of input/source files and use it
* read: introduce langStack for source files
* read: add lineFposMap
* read: add the remaining stuff from uctags
* parse: remove things unused by Geany for simpler merge
* parse: some simple syncs and include syncs
* parse: other small syncs
* lregex: sync includes
* lregex: replace regexBroken with regexAvailable
* lregex: some simple syncs
* lregex: remove unused functions
* lregex: Initialize lregex structures the universal-ctags way
* lregex: Add missing fields to regexPattern
* lregex: sync some function prototypes
* lregex: introduce findRegexTagsMainloop()
* lregex: some harmless syncs
* lregex: add new flag processing
* lregex: sync the remaining parts of the existing code
* lregex: Add the remaining missing functions from uctags
* Grab uctags version of parse.c/h and nestlevel.c/h plus additional fallout
* grab uctags debug.c/h
* Add typeRef to sTagEntryInfo
* read: backport patch from uctags
* Some minor syncs
* Sync error.c completely and create a custom error printer
* tmp
* Some initial work on ctags as a library
* Move ctags "API" from Geany to ctags
* Make LanguageTable variable private in parse.c
* Add the remaining missing files from uctags
* Annotate geany-specific diffs
-- File Changes --
M configure.ac (9)
M ctags/Makefile.am (42)
M ctags/main/args.c (50)
M ctags/main/args.h (6)
A ctags/main/ctags-api.c (141)
A ctags/main/ctags-api.h (56)
M ctags/main/ctags.h (21)
A ctags/main/debug.c (128)
M ctags/main/debug.h (82)
A ctags/main/dependency.c (102)
A ctags/main/dependency.h (45)
A ctags/main/e_msoft.h (71)
M ctags/main/entry.c (1233)
M ctags/main/entry.h (175)
A ctags/main/error.c (59)
A ctags/main/error.h (26)
A ctags/main/field.c (921)
A ctags/main/field.h (111)
A ctags/main/flags.c (107)
A ctags/main/flags.h (29)
A ctags/main/fmt.c (354)
A ctags/main/fmt.h (25)
A ctags/main/gcc-attr.h (30)
M ctags/main/general.h (112)
A ctags/main/htable.c (271)
A ctags/main/htable.h (47)
M ctags/main/keyword.c (22)
M ctags/main/keyword.h (4)
A ctags/main/kind.c (158)
M ctags/main/kind.h (73)
M ctags/main/lcpp.c (254)
M ctags/main/lcpp.h (42)
M ctags/main/lregex.c (806)
A ctags/main/lxcmd.c (1227)
A ctags/main/lxpath.c (217)
M ctags/main/main.c (464)
A ctags/main/mbcs.c (112)
A ctags/main/mbcs.h (22)
M ctags/main/mio.c (382)
M ctags/main/mio.h (40)
M ctags/main/nestlevel.c (61)
M ctags/main/nestlevel.h (19)
A ctags/main/objpool.c (79)
A ctags/main/objpool.h (39)
M ctags/main/options.c (3474)
M ctags/main/options.h (172)
A ctags/main/output-ctags.c (224)
A ctags/main/output-etags.c (107)
A ctags/main/output-json.c (189)
A ctags/main/output-xref.c (60)
A ctags/main/output.h (50)
M ctags/main/parse.c (2846)
M ctags/main/parse.h (262)
A ctags/main/pcoproc.c (296)
A ctags/main/pcoproc.h (29)
A ctags/main/promise.c (102)
A ctags/main/promise.h (27)
A ctags/main/ptag.c (234)
A ctags/main/ptag.h (63)
A ctags/main/ptrarray.c (152)
A ctags/main/ptrarray.h (46)
M ctags/main/read.c (962)
M ctags/main/read.h (121)
A ctags/main/repoinfo.c (12)
A ctags/main/repoinfo.h (1)
M ctags/main/routines.c (585)
M ctags/main/routines.h (71)
A ctags/main/selectors.c (382)
A ctags/main/selectors.h (35)
M ctags/main/sort.c (120)
M ctags/main/sort.h (15)
M ctags/main/strlist.c (167)
M ctags/main/strlist.h (20)
A ctags/main/types.h (29)
M ctags/main/vstring.c (214)
M ctags/main/vstring.h (29)
M ctags/parsers/abc.c (1)
M ctags/parsers/asciidoc.c (23)
M ctags/parsers/c.c (39)
M ctags/parsers/conf.c (1)
M ctags/parsers/fortran.c (14)
M ctags/parsers/lua.c (8)
M ctags/parsers/markdown.c (1)
M ctags/parsers/php.c (4)
M ctags/parsers/powershell.c (1)
M ctags/parsers/python.c (107)
M ctags/parsers/rest.c (21)
M ctags/parsers/ruby.c (38)
M ctags/parsers/txt2tags.c (59)
M src/symbols.c (2)
M src/tagmanager/Makefile.am (4)
D src/tagmanager/tm_ctags_wrappers.c (171)
D src/tagmanager/tm_ctags_wrappers.h (65)
M src/tagmanager/tm_parser.c (18)
M src/tagmanager/tm_source_file.c (74)
M src/tagmanager/tm_tag.c (2)
M src/tagmanager/tm_workspace.c (4)
M tests/ctags/bug1585745.cpp.tags (6)
M tests/ctags/bug507864.c.tags (7)
M tests/ctags/cpp_destructor.cpp.tags (2)
M tests/ctags/simple.d.tags (2)
-- Patch Links --
https://github.com/geany/geany/pull/1263.patchhttps://github.com/geany/geany/pull/1263.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1263
AppleLanguages differs based on OS X version - newer versions
return values like cs_CZ, older OS X versions return only cs. This
leads to problems in scripts executed in VTE. It seems that
AppleLocale always returns the complete value so test it first when
setting LANG to mitigate this problem.
In addition, move adding the suffix "UTF-8" to the very end of the
script because some parts of the script depend on the value
without this suffix.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany-osx/pull/9
-- Commit Summary --
* Check AppleLocale before AppleLanguages when setting LANG
-- File Changes --
M launcher-gtk2.sh (87)
M launcher-gtk3.sh (87)
-- Patch Links --
https://github.com/geany/geany-osx/pull/9.patchhttps://github.com/geany/geany-osx/pull/9.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-osx/pull/9
On startup, in Status window:
Unable to perform version check.
Error code: 6
Error message: Unacceptable TLS certificate
Pretty sure it happened with earlier versions, definitely didn't work on 1.32 as I didn't get notified of 1.33, but definitely happens on startup for 1.33.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany-plugins/issues/752
Geany version: 1.33
[gist: geany-bugs.txt ](https://gist.github.com/Tzqt/c5b34569bfb65aafeb5a8ade6bd1b7cd)
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/1876
Addresses relevant issues mentioned in #1876. I didn't care about several "dead assignments", because they are initial values for variables, or make mirrors similar logic in the same function (like successive calls moving a pointer, I left the last one storing the move even if it's not (currently) used afterward), etc.
**:warning: WARNING :warning:** The Perl change should be checked by a Perl person, I don't really know if it actually makes sense to report this differently. Also, should be upstreamed if relevant. BTW, I can move this out of this PR.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/1878
-- Commit Summary --
* Report subroutine declarations as such
* Add a sanity check on the Save As callback
* Remove an unnecessary local variable
* Remove unused parameter
-- File Changes --
M ctags/parsers/perl.c (1)
M src/dialogs.c (1)
M src/editor.c (15)
M src/editor.h (2)
M src/keybindings.c (2)
-- Patch Links --
https://github.com/geany/geany/pull/1878.patchhttps://github.com/geany/geany/pull/1878.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/1878