[Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2016-12-06 Thread Thomas Lange
Bash lists with only one string item are not applying syntax highlighting.
It worked correctly with an older version of Geany, but I cannot say when this 
regression was introduced.

Example:
```
#!/bin/bash

var1=('foo')
var2=('foo: works')
var3=('foo', 'bar')
```
Expected syntax highlighting:
https://git.archlinux.org/svntogit/packages.git/tree/trunk/PKGBUILD?h=packages/audacious
See line 13, 16-20, 22.

System information:
* Arch Linux
* Geany 1.2.9

-- 
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/1308

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-04 Thread Thomas Lange
The issue persists with Version 1.30.1.

-- 
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/1308#issuecomment-299286082

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-04 Thread elextr
Highlighting is provided by the editing widget Geany uses from the Scintilla 
project.  Please report the issue 
[there](https://sourceforge.net/p/scintilla/bugs/).

Though I am not sure its a bug, being assigned to a variable, it can also 
correctly be considered an expression in parens not a command list as I read 
the bash manual, so it should not be list highlighted.  

-- 
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/1308#issuecomment-299329203

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-04 Thread Thomas Lange
Thank you, reported as https://sourceforge.net/p/scintilla/bugs/1944/. Feel 
free to close this issue.

-- 
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/1308#issuecomment-299334782

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-04 Thread elextr
Thanks for reporting back the Scintilla issue, we normally leave issues open 
until we see what Scintilla does.

-- 
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/1308#issuecomment-299335887

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-07 Thread Thomas Lange
Fixed upstream: 
https://sourceforge.net/p/scintilla/code/ci/7619719dd43233dbf9f440cfbf5e4d4a48fe552d/

Do you plan to update Scintilla with the next release?

-- 
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/1308#issuecomment-299746932

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-07 Thread elextr
If there is a Scintilla release in time it should be included.

-- 
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/1308#issuecomment-299747626

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-27 Thread Thomas Lange
Version 3.7.5 of Scintilla has been released now.

-- 
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/1308#issuecomment-304448719

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-05-27 Thread elextr
Yes, when someone has time to import the new version it will be added to Geany.

-- 
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/1308#issuecomment-304482553

Re: [Github-comments] [geany/geany] Regression: No syntax highlighting for bash lists with one item (#1308)

2017-07-24 Thread elextr
Closed #1308 via #1503.

-- 
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/1308#event-1177068382