Review Request 122181: [KUnitConversion] UnitCategory convert: Call UnitCategoryPrivate instead of duplicating it

[Vishesh Handa]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/122181/
-----------------------------------------------------------

(Updated Jan. 23, 2015, 12:10 p.m.)


Status
------

This change has been marked as submitted.


Review request for KDE Frameworks.


Repository: kunitconversion


Description
-------


  This is probably a mistake when implementing the private class. Both
  UnitCategoryPrivate::convert and UnitCategory::convert essentially
  the same thing. However, all sub categories modify the
  UnitCategoryPrivate virtual method, so we should be calling that one.
    
    
  This was caught while trying to debug the currency converter. It has its
  own custom convert function.


Diffs
-----

  src/unitcategory.cpp c34217e 

Diff: https://git.reviewboard.kde.org/r/122181/diff/


Testing
-------

Fixes a test in another patch.


Thanks,

Vishesh Handa

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20150123/87ab576b/attachment.html>