Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-11-20 Thread LarsGit223
I suggest that you open a new issue for that with a specific example code which reproduces the problem. -- 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/1549#issuecomment-345842384

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-11-20 Thread AdamDanischewski
Note: This issue is fixed, yet the colorization is still wrong for HERE-DOC's. It is broken in Scintilla there is an open bug yet it's a low priority, it should probably get "auto-fixed" via Scintilla at some point. -- You are receiving this because you are subscribed to this thread. Reply to

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-11-20 Thread Thomas Martitz
Closed #1549. -- 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/1549#event-1350516068

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-11-20 Thread LarsGit223
I downloaded and built release 1.32 and re-tested this. Looks good, I think this can be closed. -- 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/1549#issuecomment-345792753

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-21 Thread AdamDanischewski
Thanks, when the next release is out it should be easy to quickly test and then we can close this. In the meantime if anyone is frustrated about this a workaround is to temporarily comment out the problematic array element assignments - that will put the highlighting back to normal and you can d

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-21 Thread LarsGit223
:thumbsup: Great! -- 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/1549#issuecomment-316945291

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-21 Thread elextr
@LarsGit223 thanks for testing #1503, it should be included in the next Geany release, so hopefully this will be fixed then. -- 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/1549#issuecom

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-21 Thread LarsGit223
I cloned, compiled, installed and tested with b4n's branch scintilla/update-375. With that version this problem is solved. So I can confirm that PR #1503 would fix this. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-20 Thread AdamDanischewski
Thanks, I hope so. I'll let them close it themselves if it has, I don't have the environment to test. -- 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/1549#issuecomment-316772331

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-20 Thread LarsGit223
I assume it was fixed with this ticket: https://sourceforge.net/p/scintilla/bugs/1944/ -- 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/1549#issuecomment-316770945

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-20 Thread LarsGit223
Update: it looks good in SciTe 3.7.5: ![testbashinscite375](https://user-images.githubusercontent.com/9009011/28430006-229f62d8-6d7f-11e7-98ad-4875bc7f9d79.png) So it seems like it has been fixed already . -- You are receiving this because you are subscribed to this thread. Reply to this email d

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-20 Thread AdamDanischewski
It looks like someone broke Scintilla, I've launched a bug ticket on Scintilla: https://sourceforge.net/p/scintilla/bugs/1963/ -- 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/1549#issuec

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-20 Thread LarsGit223
I downloaded and built SciTe 3.7.4, the file looks like this after opening: ![testbashinscite374](https://user-images.githubusercontent.com/9009011/28427586-f2cc43da-6d76-11e7-9c25-3b7dd44db165.png) SciTe shows the following version numbers in the about dialog: Version 3.7.4 compiled for GTK+ 2.24

Re: [Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-19 Thread elextr
Confirmed with 1.31, problem does not appear with Scite 3.6.0 (whats in my distro), can anybody test with Scite 3.7.4 which has the same Scintilla as Geany 1.31? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://githu

[Github-comments] [geany/geany] Bash - Shell Script Colorizer is Broken (#1549)

2017-07-19 Thread AdamDanischewski
I've got a Bash script and it is using arrays (something the other shells don't have, maybe that's why this isn't more thoroughly tested?). It cannot handle nested parenthesis, I'm not sure if this problem is actually Scintilla. Here is how to reproduce, enter the following into a file t.bsh: