Re: [Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

2021-09-06 Thread elextr
Merged #2862 into master. -- 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/2862#event-5262829397

Re: [Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

2021-09-06 Thread elextr
@codebrainz thanks -- 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/2862#issuecomment-913908549

Re: [Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

2021-09-06 Thread elextr
@b4n yeah, the new Julia lexer uses std::string_view which is C++17 and was causing G-P Travis failures until the distro version was updated to a modern enough one to have C++17. And of course #2867 as soon as the next release is done. -- You are receiving this because you are subscribed to t

[Github-comments] [geany/geany-plugins] emmet for geany (#1109)

2021-09-06 Thread ilianus25
emmet is a very good addon but it does not exist for this wonderfull project do you plan to add it as a plugin? thank you very much -- 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

Re: [Github-comments] [geany/geany] Update C++ compiler check to require C++17 (#2862)

2021-09-06 Thread Colomban Wendling
@b4n approved this pull request. Just tested with Debian Buster's regular g++ (8.3) which works, and ag++ wrapper hack (for it to fail): ```python3 #!/usr/bin/env python3 from os import execvp from sys import argv args = ['g++'] for arg in argv[1:]: if arg.startswith('-std='):

Re: [Github-comments] [geany/geany-plugins] Fixed a crash of geany by the git-changebar on SCI_ADDTEXT (#1106)

2021-09-06 Thread Enrico Tröger
@vlvlbel thanks! LGTM. I'm going to merge this in a few days, after CI is working again. -- 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/pull/1106#issuecomment-913864130

Re: [Github-comments] [geany/geany] Windows cross, GTK3 and x86_64 build preparations (#2590)

2021-09-06 Thread Enrico Tröger
@eht16 pushed 1 commit. f472e008b5ce3175f76db800c70005014a698205 Windows: Install development files by default with "Full" install target -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/geany/geany/pull/2590/files/5fc652454cd0f8a67271

Re: [Github-comments] [geany/geany] Fix clang warnings: Use empty array instead of NULL pointer (#2889)

2021-09-06 Thread Avinash Sonawane
@elextr I still see these warnings with `clang version 10.0.0-4ubuntu1` CFLAGS set to `-Wall -g -O2`. @b4n I'll look into this tomorrow. :) -- 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

Re: [Github-comments] [geany/geany] Feature request: allow CTRL+DEL to delete empty spaces alone, not also non-space characters after them (#2893)

2021-09-06 Thread pgp
Thanks! -- 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/2893#issuecomment-913632004

Re: [Github-comments] [geany/geany] Feature request: allow CTRL+DEL to delete empty spaces alone, not also non-space characters after them (#2893)

2021-09-06 Thread pgp
Closed #2893. -- 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/2893#event-5261040297

Re: [Github-comments] [geany/geany] Use "encoding" keyword argument for lxml's tostring() (#2885)

2021-09-06 Thread Colomban Wendling
Works as well on oldstable (Buster) here, so I guess we're safe enough. And Travis is happy as well. -- 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/2885#issuecomment-913598262

Re: [Github-comments] [geany/geany] Use "encoding" keyword argument for lxml's tostring() (#2885)

2021-09-06 Thread Colomban Wendling
Merged #2885 into master. -- 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/2885#event-5260790742

Re: [Github-comments] [geany/geany] Feature request: allow CTRL+DEL to delete empty spaces alone, not also non-space characters after them (#2893)

2021-09-06 Thread elextr
Or `Edit->Preferences->Files->Saving Files->Strip trailing spaces and tabs` and Geany will delete them automatically on saving. -- 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/2893#issue

Re: [Github-comments] [geany/geany-plugins] Fixed a crash of geany by the git-changebar on SCI_ADDTEXT (#1106)

2021-09-06 Thread Belousov Vladimir
@eht16 I've added the second commit to this pull request. I've tested Macro and Numbered Plugins- after the changes they began to work for me (before the changes they crashed Geany). I haven't tested the Lua plugin, but due to a small change there, I believe, it should work. -- You are receiv

Re: [Github-comments] [geany/geany-plugins] Fixed a crash of geany by the git-changebar on SCI_ADDTEXT (#1106)

2021-09-06 Thread Belousov Vladimir
@vlvlbel pushed 1 commit. 8c1a01ed33b0e0d6651a276835f30cc9a2b54e5b Fixed crashes in Lua, Macro and Numbered Bookmarks plugins under Win64 platform due to a different size of glong and sptr_t -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://githu

Re: [Github-comments] [geany/geany] Feature request: allow CTRL+DEL to delete empty spaces alone, not also non-space characters after them (#2893)

2021-09-06 Thread elextr
`Document->Strip trailing spaces` and do it all in one command? -- 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/2893#issuecomment-913540812

[Github-comments] [geany/geany] Feature request: allow CTRL+DEL to delete empty spaces alone, not also non-space characters after them (#2893)

2021-09-06 Thread pgp
Sometimes I deal with wrongly-formatted code files, which contain empty spaces at the row end, like in the following; ``` text here;<- (lots of empty spaces here) ``` however, when I press CTRL + DEL starting from the end of the line, the whole last word is de

Re: [Github-comments] [geany/geany] warning: explicitly defaulted move constructor is implicitly deleted (#2891)

2021-09-06 Thread elextr
Ok, so the warning should go away when #2867 is merged and in the meantime its not a problem, its a human warning, just that somebody using `Style` might get confused, but Geany does not use the C++ directly so no matter. -- You are receiving this because you are subscribed to this thread. Repl

Re: [Github-comments] [geany/geany] warning: explicitly defaulted move constructor is implicitly deleted (#2891)

2021-09-06 Thread elextr
Closed #2891. -- 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/2891#event-5260069618

Re: [Github-comments] [geany/geany] Fix clang warnings: Use empty array instead of NULL pointer (#2889)

2021-09-06 Thread elextr
@rootkea If you are using the same options as [here](https://github.com/geany/geany/issues/2891#issuecomment-913481562), especially -O0 that might explain why the compiler does not notice the protection of the null check. IIUC compilers need something like -O2 to do the data flow analysis that

Re: [Github-comments] [geany/geany] warning: explicitly defaulted move constructor is implicitly deleted (#2891)

2021-09-06 Thread Avinash Sonawane
> That code is from the Scintilla project so needs to be reported there I tried compiling latest default branch of Scintilla and I didn't see these warnings. > specify the compiler version and options you are using. While compiling `geany` I did: ``` $ ./autogen.sh --enable-binreloc --enable-vt