Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[extras/linenoise] UTF-8 support not working properly #967

Open
github-actions bot opened this issue Jan 11, 2023 · 8 comments
Open

[extras/linenoise] UTF-8 support not working properly #967

github-actions bot opened this issue Jan 11, 2023 · 8 comments
Labels
3rd-party Issues related to 3rd-party libraries and code bug Something isn't working critical Critical, top-priority issues repl Issues related to the REPL todo Automatically generated issues from TODOs in the code

Comments

@github-actions
Copy link

[extras/linenoise] UTF-8 support not working properly

should add -DUSE_UTF8 for UTF-8 support

to all following lines

the problem is that - although it "works" - different characters are not shown at all:

e.g. accented characters, or ø - our symbol for null

# TODO(extras/linenoise) UTF-8 support not working properly

{.passC: "-I" & parentDir(currentSourcePath()) .}

# TODO(extras/linenoise) UTF-8 support not working properly
#  should add -DUSE_UTF8 for UTF-8 support
#  to all following lines
#  the problem is that - although it "works" - different characters are not shown at all:
#  e.g. accented characters, or ø - our symbol for null
#  labels: bug, 3rd-party, repl
{.compile("linenoise/linenoise.c", "-I" & parentDir(currentSourcePath())).}
{.compile("linenoise/stringbuf.c", "-I" & parentDir(currentSourcePath())).}
{.compile("linenoise/utf8.c", "-I" & parentDir(currentSourcePath())).}

#=======================================
# Types
ndex 204ec243f..571c77aa9 100644
++ b/version/build

f03bc1611ea42487492a8e8754f10ead9ea92679

@github-actions github-actions bot added 3rd-party Issues related to 3rd-party libraries and code bug Something isn't working repl Issues related to the REPL todo Automatically generated issues from TODOs in the code labels Jan 11, 2023
@stale
Copy link

stale bot commented Sep 8, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Sep 8, 2023
@drkameleon drkameleon added the critical Critical, top-priority issues label Nov 2, 2023
@stale stale bot removed the stale label Nov 2, 2023
@drkameleon
Copy link
Collaborator

drkameleon commented Nov 2, 2023

I'm marking this as critical since I basically think it is.

Arturo was built was proper unicode support in mind and apparently everything works along these lines except for the REPL which is a shame!

@RickBarretto Can you please confirm this to make sure it's not something too macOS-related?

For example, this seems to be working:
Screenshot 2023-11-02 at 11 16 06

But actually writing it, character by character, it seems like it's adding extra spaces or considers it like a "double" character (so, it seems to be adding an extra space visually - which doesn't really exist - and trying to delete the previous character leads in a mess).

Also, using anything more complicated than accented characters, leads to an even bigger mess:

Screenshot 2023-11-02 at 11 18 26

(This is the greek equivalent of "Good morning" and that's where the cursor supposedly is - while it's actually next to the closing double-quote)

This is the result of writing Chinese - or trying to - "早上好" (another way to say Good morning lol). Now, the result is even more disheartening:

Screenshot 2023-11-02 at 11 19 46

Funny enough, the above happens also when copy-pasting the text ❗

Weird, weird... 🤔

@RickBarretto
Copy link
Collaborator

I'm opening my VS right now to test it! 😉

@RickBarretto
Copy link
Collaborator

RickBarretto commented Nov 2, 2023

On VsCode:

Running from Powershell:
image

Running from bash:
image

But, not exclusive of Arturo. For Nim's secret too:

image

Copy link

stale bot commented Jun 29, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jun 29, 2024
@scifx
Copy link

scifx commented Oct 21, 2024

In REPL mode, UTF-8 always fails to work. Is there a fix in the works?

@stale stale bot removed the stale label Oct 21, 2024
@drkameleon
Copy link
Collaborator

@scifx Very valid question, but I'm afraid I don't have a very real answer.

We've been experimenting lately, although I wouldn't say the experiments have been extremely fruitful.

In any case, I can assure you it's one of the things that personally bother me a lot... so, it will be prioritized, probably one the upcoming release is finalized.

Btw: which OS are you using?

@scifx
Copy link

scifx commented Oct 24, 2024

You've been keeping an eye on it, that's good. I hope you can resolve it soon. Currently, I'm using the Windows system, but I used to use Linux, and I have also compiled it on my phone before. Therefore, this issue exists across all systems.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
3rd-party Issues related to 3rd-party libraries and code bug Something isn't working critical Critical, top-priority issues repl Issues related to the REPL todo Automatically generated issues from TODOs in the code
Projects
None yet
Development

No branches or pull requests

3 participants