Enrico Tröger

> Okay, I understand. I just propose my suggestion, because it would be very convenient if it could autosave every second. So we don't need to manually save the file...

@elextr @techee @b4n @kugel- review is welcome and don't be scared even there is much Windows and mingw in the title and description, the main part is just bash code...

I tend to agree with @nixuser58 and @kugel-. With proper highlighting of the new keywords, the code looks much better. The drawback is the mentioned "re.match" function which is now...

Sure, that would be great. Please note the https://github.com/geany/geany/blob/master/scripts/update-scintilla.sh script which makes the update quite easy.

Good guess! It was as easy as this. Thanks. Still not sure why is this necessary now but it is easy enough, I guess. Now, cppcheck finds new issues. The...

``` make[3]: Entering directory '/home/enrico/projects/geany-plugins/geanyctags/src' /usr/bin/cppcheck \ -q --template gcc --error-exitcode=0 \ --library=gtk \ -I/home/enrico/apps/include/geany \ \ . geanyctags.c:165:2: warning: Mismatching allocation and deallocation: argv [mismatchAllocDealloc] g_strfreev(argv); ^ geanyctags.c:125:9: note:...

There are many more warnings :(. I fixed fhe most obvious ones but there are some left where I'm not sure how to fix them or whether this might be...

> To eliminate the warning, the `argv` variable could be only declared/freed in the `#ifndef G_OS_WIN32` blocks because it's unused on linux. I tried with the following changes: ```diff diff...

I thought `argv` is *only* used on Linux? Anyway, I tried also with `ifdef`s and it's the same result. Either my testing is wrong or cppcheck ignores these macros.

Actually it doesn't make much different if the new plugin would compile or lint the code from a technical point of view because linting as well as compiling in Geany...