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

Pasting an En dash (–) crashes #112

Closed
FluXy opened this issue Mar 17, 2022 · 4 comments
Closed

Pasting an En dash (–) crashes #112

FluXy opened this issue Mar 17, 2022 · 4 comments
Assignees
Labels
bug Something isn't working

Comments

@FluXy
Copy link
Contributor

FluXy commented Mar 17, 2022

Pasting only tested with Label and StaticText.


For loading:

  • When loading a layout file where it's (–) already in the "Text" property an exception happens:

CEGUI::UnicodeStringException in function 'void __cdecl CEGUI::String::checkUtf8CodePointSizeForValidity(const unsigned int,const unsigned int)' (C:\Personal\Repos\cegui\cegui\src\String.cpp:298) : Invalid initial byte detected for a code point in the string

Which seems correct but it would be nice if it only ignores that character and not the whole .layout file

  • When loading a layout file where it's as XML (–) already in the "Text" property it works.

CEED-v1.1.1-Win-x86

The en dash is like a hyphen (-) but slightly longer (–).
https://en.wikipedia.org/wiki/Dash#En_dash

@niello niello added the bug Something isn't working label Mar 19, 2022
@niello niello self-assigned this Mar 19, 2022
@niello niello closed this as completed in 5b29bcc Mar 19, 2022
@niello
Copy link
Collaborator

niello commented Mar 19, 2022

@FluXy can you build CEED with this fix? I wouldn't prepare a separate release for it unless absolutely inevitable.

@niello
Copy link
Collaborator

niello commented Mar 19, 2022

ceed.exe.txt

Here is an alternative. Simply remove ".txt"

@FluXy
Copy link
Contributor Author

FluXy commented Mar 19, 2022

Thanks for the quick fix! Works fine now

@niello
Copy link
Collaborator

niello commented Mar 20, 2022

I had overcome my laziness and released 1.1.2 with this fix and a couple of additional improvements. Enjoy and feel free to report problems)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants