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

Add composite dictionary support (gsoc2013cdict branch) #34

Closed
rrthomas opened this issue Dec 31, 2016 · 2 comments · Fixed by #383
Closed

Add composite dictionary support (gsoc2013cdict branch) #34

rrthomas opened this issue Dec 31, 2016 · 2 comments · Fixed by #383

Comments

@rrthomas
Copy link
Contributor

This would be nice to have in a release.

@rrthomas
Copy link
Contributor Author

I've some some work on this at https://github.com/rrthomas/enchant/tree/compdict

Unfortunately, the code in the gsoc2013cdict branch appears to be incomplete: the logic for setting up a composite dictionary provider is there, but the methods to deal with checking multiple dictionaries for words remains stubbed out. I have emailed the author to see if they can help.

@rrthomas
Copy link
Contributor Author

See #349 for motivation for this work.

It would be nice if composite dictionaries were handled transparently, with no need to change the API, simply by recognizing composite language tags in the parser. This would then allow the CLI front-end to support multiple dictionaries with no changes. One question is whether the more natural comma separator could be used instead of the current colon; I see no reason why not.

rrthomas added a commit to rrthomas/enchant that referenced this issue May 26, 2024
This completes the implementation of composite dictionaries.
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 a pull request may close this issue.

1 participant