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 #239 ] Reset oldChiSymbolCursor in chewing_handle_Default #264

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Billy4195
Copy link
Contributor

@Billy4195 Billy4195 commented May 23, 2017

Fix #239
This issue is because of the chiSymbolCursor be reset to incorrect oldChiSymbolCursor.
This commit can prevent the chiSymbolCursor be reset to incorrect oldChiSymbolCursor value happened in #239, because the oldChiSymbolCursor will be reset as the current chiSymbolCursor after chewing_handle_Default().

Fix chewing#239
This issue is because of the chiSymbolCursor be reset to incorrect
oldChiSymbolCursor.
This commit can prevent the chiSymbolCursor be reset to incorrect
oldChiSymbolCursor value happened in chewing#239, because the
oldChiSymbolCursor will be reset as the current chiSymbolCursor
after chewing_handle_Default().
@Billy4195 Billy4195 changed the title [Fix #239] Reset oldChiSymbolCursor in chewing_handle_Default [ Fix #239 ] Reset oldChiSymbolCursor in chewing_handle_Default May 23, 2017
@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 90.552% when pulling 743d819 on Billy4195:Fix#239 into cad4a6e on chewing:master.

@kanru kanru added the rewrite in rust Might be fixed by the rust rewrite label Dec 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rewrite in rust Might be fixed by the rust rewrite
Projects
None yet
Development

Successfully merging this pull request may close these issues.

assertion fail in ChoiceEndChoice()
3 participants