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

fix unicode #117

Closed
wants to merge 3 commits into from
Closed

Conversation

readmemyrights
Copy link
Contributor

  • dtk-speak, emacspeak-speak: change when unicode is handled
  • dectalk-voices, outloud-voices: enable unicode handling for these engines

This introduces a new variable, dtk-handle-unicode, and the various
character translation functions like dtk-unicode-replace-chars only get
triggered if it's set to t. Only a minority of synthesizers today need
this functionality so the variable defaults to nil.

* only do the various transformations when dtk-handle unicode is set
* dtk-handle-unicode can be set by the user but in the next commit the
  various engines will set it automatically that need this.
engines

mac, espeak, and most other modern ttses can handle unicode in some way,
these are the only ones that really need help.
@tvraman
Copy link
Owner

tvraman commented Mar 12, 2024

PR looks good, but please update it after pulling from Master since other files in the repo have changed, and I dont want to accidentally merge unrelated files.

Hopefully this won't cause even more git breakage.
@readmemyrights
Copy link
Contributor Author

I've just pulled in all the newest changes. The two commits I made only
contain the changes I wanted (confirmed with git show).

I'll post a git patch on the mailing list incase this doesn't work either.

@tvraman
Copy link
Owner

tvraman commented Mar 13, 2024

Merged; thanks for your contribution

@tvraman tvraman closed this Mar 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants