Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Fix sometimes TTS engine pronounces space as "percent twenty" #51

Merged
merged 1 commit into from
Apr 16, 2023
Merged

Fix sometimes TTS engine pronounces space as "percent twenty" #51

merged 1 commit into from
Apr 16, 2023

Conversation

veksha
Copy link
Contributor

@veksha veksha commented Apr 16, 2023 •
edited
Loading

Shatur
Shatur suggested changes Apr 16, 2023
View reviewed changes
Copy link
Member

@Shatur Shatur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

src/qonlinetts.cpp Outdated Show resolved Hide resolved
@Shatur Shatur changed the title fixed: Sometimes TTS engine pronounces space as "percent twenty" Fix sometimes TTS engine pronounces space as "percent twenty" Apr 16, 2023
@Shatur
Copy link
Member

Shatur commented Apr 16, 2023

Please, revert .arg formatting because we use clang-format to format all source code.

@Shatur Shatur enabled auto-merge April 16, 2023 08:59
@Shatur Shatur disabled auto-merge April 16, 2023 08:59
@Shatur Shatur enabled auto-merge (squash) April 16, 2023 08:59
@Shatur
Copy link
Member

Shatur commented Apr 16, 2023 •
edited
Loading

Sorry for bothering, but could you also apply the same change for vcpkg here?

@veksha
Copy link
Contributor Author

veksha commented Apr 16, 2023

Sorry for bothering, but could you also apply the same change for vcpkg here?

this is subproject (i'm new to subprojects). i think it's not needed here.

@Shatur
Copy link
Member

Shatur commented Apr 16, 2023

this is subproject (i'm new to subprojects). i think it's not needed here.

It contains separate vcpkg configuration and separate CI. So it's needed.

auto-merge was automatically disabled April 16, 2023 09:29

Head branch was pushed to by a user without write access

@Shatur Shatur enabled auto-merge (squash) April 16, 2023 09:30
@veksha
Copy link
Contributor Author

veksha commented Apr 16, 2023

you are right. done.

@Shatur Shatur merged commit 9a26d3c into crow-translate:master Apr 16, 2023
@veksha veksha mentioned this pull request May 26, 2023
Open
@Shatur Shatur mentioned this pull request Jul 6, 2023
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Shatur Shatur Shatur requested changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Sometimes TTS engine pronounces space as "percent twenty"
2 participants
@veksha @Shatur