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

Vishesh Handa me at vhanda.in
Wed Jan 21 14:47:13 UTC 2015


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

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/20150121/3200eed1/attachment.html>


More information about the Kde-frameworks-devel mailing list