Review Request: Save declensions gender specific

[Frederik Gladhorn]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/103325/#review8757
-----------------------------------------------------------

Ship it!


Hm, for one thing, the lib does support versioning. It's also possible to accept the old syntax. That is done to keep some compatibility already (I don't remember where without diving into the code right now), though I think it just read the old format without using the version information.
But in this case I'm actually for simply changing the lib and not supporting the old implementation for the simple reason that there was no UI that could edit this before. The only way to actually get the declension info there was by hand. No program made use of this. So let's keep it clean and make it proper.
I think the patch is good. I did not read/test anything really though, sorry.

- Frederik Gladhorn


On Dec. 4, 2011, 1:50 p.m., Jan Gerrit Marker wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/103325/
> -----------------------------------------------------------
> 
> (Updated Dec. 4, 2011, 1:50 p.m.)
> 
> 
> Review request for KDE Edu.
> 
> 
> Description
> -------
> 
> Some languages (Latin, for example) have declensions for adjectives which do not only depend on the number and case but on the gender, too. libkdeedu has an implementation for declensions which does not honor this. This patch corrects that wrong behavior. The change adds an incompatibility to files which were created using the old library. I don't think that there are a lot of files with declensions out there, as for example parley does not have the feature, yet, but if needed I'd try parse old files, too.
> 
> 
> Diffs
> -----
> 
>   keduvocdocument/keduvocdeclension.cpp 327c3ed 
> 
> Diff: http://git.reviewboard.kde.org/r/103325/diff/diff
> 
> 
> Testing
> -------
> 
> Works with parley, after finishing the implementation of the declension widget (I'll create another review request in order to get this in).
> 
> 
> Thanks,
> 
> Jan Gerrit Marker
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20111206/1b6d93f4/attachment.html>