Skip to content
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

Edge case of parenthesis #266

Open
wysohn opened this issue Mar 11, 2020 · 0 comments
Open

Edge case of parenthesis #266

wysohn opened this issue Mar 11, 2020 · 0 comments
Assignees
Labels
domain:suggestion A suggestion, i.e: new ideas, optimization, etc. package:core Related to share domain nor script interpreter priority:low

Comments

@wysohn
Copy link
Member

wysohn commented Mar 11, 2020

When lexer performs lexical analysis, there can be two case of parenthesis:

  1. The regular parenthesis that can be typed using keyboard

  2. Parenthesis that can be copied directly from texts, yet it's not the same as the typed parenthesis (discord chat for example)

The second case is not accepted and throws "Unrecognizable character" Exception, yet since it's like the only case where ambiguous parenthesis can be used, it would be convenient to support such parenthesis.

Possible candidates can be found here: https://stackoverflow.com/questions/13535172/list-of-all-unicodes-open-close-brackets

Implementing this, copying the code from text will be less troublesome.


Note: Though, we only support the unicode characters which are part of UTF-8, and if it's not part of UTF-8, throw the exception as before.

@wysohn wysohn added the domain:suggestion A suggestion, i.e: new ideas, optimization, etc. label Mar 11, 2020
@Sayakie Sayakie added priority:low package:core Related to share domain nor script interpreter labels Jul 20, 2023
@Sayakie Sayakie self-assigned this Jul 20, 2023
@Sayakie Sayakie changed the title Egde case of parenthesis Edge case of parenthesis Jul 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
domain:suggestion A suggestion, i.e: new ideas, optimization, etc. package:core Related to share domain nor script interpreter priority:low
Projects
None yet
Development

No branches or pull requests

2 participants