The SpellChecker class:
lingucomponent/source/spell/sspellimp.hxx / .cxx
Hunspell ** m_aDicts;
Locale * m_aDLocs;
Which looks like a mess to me; we should use std::vector<> for these instead; and nice iterators.
We should prolly use a std::unique_ptr<> as well for the Hunspell objects - but please do add some debug / check that the Hunspell objects are not created and destroyed more frequently than before the re-factor.
Thanks ! =)
I would like to take this task up. May I proceed ? Anything that I should keep in mind relating to the last line in the description : "destroyed more frequently than before the re-factor." ?
I'd like to work on this, if its up for taking?
Sahil - can you confirm you're not making progress here ? I don't see it assigned to you. If so just assign it to yourself - abstatic & go for it ! =) looking forward to your patch.
abstatic committed a patch related to this issue.
It has been pushed to "master":
tdf#112373 Using vector of struct to manage hunspell dicts
It will be available in 6.1.0.
The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
Affected users are encouraged to test the fix and report feedback.
A polite ping, still working on this bug?
I think this bug was fixed, with my patch.
Indeed - thanks abstatic ! =) closing.