D20181: Add Binary Data units (bits, kilobytes, kibibytes ... yottabytes)

[Aleix Pol Gonzalez]

apol added a comment.


  In D20181#441918 <https://phabricator.kde.org/D20181#441918>, @JJRcop wrote:
  
  > In D20181#441887 <https://phabricator.kde.org/D20181#441887>, @aacid wrote:
  >
  > > Can we use https://api.kde.org/frameworks/kcoreaddons/html/classKFormat.html instead of doing this again?
  >
  >
  > I think that's out of scope for this patch. It should be done in another patch so we can refactor Frequency <https://phabricator.kde.org/source/kunitconversion/browse/master/src/frequency.cpp;99cfd52570b747e46d7657e1e11d2605b9a09b81$33> and Length <https://phabricator.kde.org/source/kunitconversion/browse/master/src/length.cpp;99cfd52570b747e46d7657e1e11d2605b9a09b81$33> to use KFormat as well.
  
  
  This can't be out of scope since as soon as you add it here we'll have to maintain API, so it can't actually be done as an iteration.
  
  Aleix

REVISION DETAIL
  https://phabricator.kde.org/D20181

To: JJRcop, broulik, #plasma, ngraham
Cc: apol, aacid, ngraham, kde-frameworks-devel, michaelh, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190401/69a1f8fc/attachment.html>