-
Notifications
You must be signed in to change notification settings - Fork 27
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
update deformatter (apertium-destxt) escape chars #93
base: master
Are you sure you want to change the base?
Conversation
(i'm fine with us switching to GH actions but it should really be a separate PR, imo) |
I noticed travis builds weren't working so I added the workflow file. I'll make a separate PR for GH acions |
This reverts commit 6eb1eeb.
@Vaydheesh can you also include ['*', '#', '+', '~'] in escape_chars list, please? |
@eagad The escape_chars is used to wrap |
@sushain97 any thoughts on this? |
They are escaped when expanding: https://github.com/apertium/lttoolbox/blob/master/lttoolbox/expander.cc#L116 But de/re-formatters must not touch |
fix: #92
Update deformatter to handle all the escape characters https://github.com/apertium/apertium/blob/c1b6b446b25bb22d74fc579ebb48b7140aa09b85/apertium/txt-format.xml#L8