-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bash] Strict special parameter highlighting #184
Comments
Another fix is adding const CharacterSet setParamStart(CharacterSet::setAlphaNum, "\'\"`{[(_*@#?-$!");
if (sc.ch == '$' && setParamStart.Contains(sc.chNext)) {
QuoteStack.Expand(sc, cmdState, stylingInside);
continue;
} so special parameter list could be configurable. |
Since the different shells have different choices, using a |
The patch, added |
Added property `lexer.bash.special.parameter` to set custom special parameters.
Uploaded another fix at #186, which added property |
Added property `lexer.bash.special.parameter` to set custom special parameters.
Current code treat all punctuation and white spaces as single character special parameter that will case bugs, like the one reported at zufuliu/notepad4#684:
![image](https://user-images.githubusercontent.com/38994291/253325478-522c9abf-6432-4dae-bf03-8788fe23edb6.png)
*
,@
,#
,?
,-
,$
,!
are supported in bash, zsh, dash, and ksh; tcsh adds%
and<
(highlight$<
seems will cause other bug).A simple fix is adding extra check after
scalar has no delimiter pair
comment:lexilla/lexers/LexBash.cxx
Lines 362 to 368 in 5f0c215
The text was updated successfully, but these errors were encountered: