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

Jump to bottom

Please make g_language_dependent_spec really global #115

Closed
GoogleCodeExporter opened this issue Apr 22, 2015 · 3 comments
Closed

Please make g_language_dependent_spec really global #115

GoogleCodeExporter opened this issue Apr 22, 2015 · 3 comments
Labels
auto-migrated Priority-Medium Type-Defect

Comments

@GoogleCodeExporter
Copy link 

What steps will reproduce the problem?
1. Run mozc_tool with QT_IM_MODULE=uim when uim-mozc is installed and used as 
default method
2. mozc_tool will crash


What is the expected output? What do you see instead?
not crash

What version of the product are you using? On what operating system?
Fedora 15, Mozc r87

Please provide any additional information below.
libuim-mozc.so is loaded by dlopen() within mozc_tool.  Because 
g_language_dependent_spec is not global, mozc_tool returns NULL with 
language::GlobalLanguageSpec::GetLanguageDependentSpec() in some timings even 
libuim-mozc.so set the lang_spec beforehand.

Removing anonymous namespace of g_language_dependet_spec and extern it solve 
the problem, I think.

Original issue reported on code.google.com by ek.k...@gmail.com on 12 Jan 2012 at 2:45
@GoogleCodeExporter
Copy link
Author 

Thank you for reporting,

I'm sorry we can not follow-up uim-mozc package, but we have plan to refactor 
language spec module in future(maybe GlobalLanguageSpec will be deprecated).

Regards.

Original comment by n...@google.com on 12 Jan 2012 at 5:31

  • Changed state: Accepted

@GoogleCodeExporter
Copy link
Author 

I also thought the language spec looked tentative one.  Thanks for your 
explanation.  Looking forward to your upcoming release.

Original comment by ek.k...@gmail.com on 12 Jan 2012 at 6:03

@GoogleCodeExporter
Copy link
Author 

Fixed in r105.
Thank you again for taking your time to report this issue.

Original comment by yukawa@google.com on 26 Mar 2012 at 10:23

  • Changed state: Fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
auto-migrated Priority-Medium Type-Defect
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@GoogleCodeExporter