Skip to content

Julia issue 97#98

Closed
getzze wants to merge 4 commits intoScintillaOrg:masterfrom
getzze:julia-issue-97
Closed

Julia issue 97#98
getzze wants to merge 4 commits intoScintillaOrg:masterfrom
getzze:julia-issue-97

Conversation

@getzze
Copy link
Copy Markdown
Contributor

@getzze getzze commented Aug 3, 2022

closes #97

EDIT: now a correct "bad before; good after" example.

table[]!='#'

is now parsed correctly as table identifier, [] brackets, != operator and '#' character.
Instead of table identifier, [] brackets, !=' operator and #' comment (because the first ' was incorrectly parsed as adjoint)

Also update unicode characters allowed as identifiers.

@nyamatongwe
Copy link
Copy Markdown
Member

Committed the 7eaf635 change set with examples in x.jl as 84f9a6c. Pull requests should include short test cases that show 'bad before; good after' so they can be checked and will remain fixed.

The above example doesn't parse as stated since ! is part of the table! identifier.

Lexing result both before and after e267314

{9}table!{7}={6}'#'{0}

However, it does fix the originating problem and this simplified example:

table[d]!='#' for d=[(1,0),(-1,0),(0,1),(0,-1)]

@getzze
Copy link
Copy Markdown
Contributor Author

getzze commented Aug 5, 2022

Sorry, I put a bad example (I edited the first message).

I added a commit with a test case.

nyamatongwe pushed a commit that referenced this pull request Aug 6, 2022
@nyamatongwe nyamatongwe added julia Caused by the julia lexer committed Issue fixed in repository but not in release labels Aug 6, 2022
@nyamatongwe
Copy link
Copy Markdown
Member

Committed.

@nyamatongwe nyamatongwe mentioned this pull request Aug 6, 2022
@nyamatongwe
Copy link
Copy Markdown
Member

Included in 5.1.9.

@nyamatongwe nyamatongwe removed the committed Issue fixed in repository but not in release label Dec 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

julia Caused by the julia lexer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Julia

2 participants