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

Unicode command line string support on Windows. #1410

Closed
ghost opened this issue Oct 21, 2024 · 5 comments
Closed

Unicode command line string support on Windows. #1410

ghost opened this issue Oct 21, 2024 · 5 comments

Comments

@ghost
Copy link

ghost commented Oct 21, 2024

Related discussion

#880 (comment)

Is your feature request related to a problem?

When using the Fluidsynth on windows :
Now, the command line supports ANSI strings, but not non-ANSI strings.
This can sometimes cause problems in multilingual environments.

To be precise, this is not a bug, is a limitation of the ANSI command line interface.
Windows users generally only use ANSI strings, so they will have less trouble using Fluidsynth.
The maintainers seem to be very busy with more important issues.
So this update may not be a high priority.

Describe the solution you'd like

If the command line supported UNICODE strings, the Fluidsynth users would be able to use UNICODE strings in filenames or devicenames, and then more freeing from character encoding issues.

Describe alternatives you've considered

Updating the command line interface from ANSI to UNICODE.
The libfluidsynth functions already support utf-8 strings, so there aren't many updates.

Additional context

The code below need some changes:

  • include/fluidsynth/misc.h
  • src/bindings/fluid_filerenderer.c
  • src/fluidsynth.c

new_codes.zip

@pedrolcl
Copy link
Contributor

Now, the command line supports ANSI strings, but not non-ANSI strings.

That is not true as an absolute sentence.

This can sometimes cause problems in multilingual environments.

This is a vague sentence and cannot be verified as-is. We need a concrete example that can be reproduced.

Some issues have already been reported related to the use of Unicode arguments: #128, #983, #1322, #1388 and then supposedly fixed by #718, #1325, and #1390. What is exactly not working?

@ghost
Copy link
Author

ghost commented Oct 27, 2024

All the answers to your questions can be found by looking at the Fluidsynth source code.
I can't explain it more precisely than that.
Thank you.

@pedrolcl
Copy link
Contributor

All the answers to your questions can be found by looking at the Fluidsynth source code. I can't explain it more precisely than that. Thank you.

@stardusteyes : This answer is quite arrogant on your part. I would understand if you say that you are not proficient in English, but in PR #1411 you wrote something that would be more fitting here on this issue report:

Tested with non-ANSI characters.
for example:

    ♬.sf2
    ♬.mid
    ♬.wav

In this case, an error will occur when running with Fluidsynth 2.3.7.
And the file names are not displayed correctly.

    Parameter '?.sf2' not a SoundFont or MIDI file or error occurred identifying it.
    Parameter '?.mid' not a SoundFont or MIDI file or error occurred identifying it.
    Rendering audio to file '?.wav'..
    fluidsynth: error: Failed to open audio file '?.wav' for writing

@derselbst I would label this report as invalid.

Also: in the light of the recent xz backdoor disaster I would be very careful with contributions from random anonymous actors.

@ghost
Copy link
Author

ghost commented Oct 27, 2024

Yes, you're right!
I don't understand English at all.
I'm just pasting the results of Google Translate into this sentence, and I can't answer complex questions correctly.

By the way, are you suggesting to close this issue?
I would like #1411 to be merged.
But if you think it shouldn't be merged, please ask the maintainer to close this issue.
I think the pull request will be closed automatically.

See you.

@ghost
Copy link
Author

ghost commented Oct 28, 2024

I'm closing this issue with the comment that this is not important.

@ghost ghost closed this as completed Oct 28, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant