sandsmark created this revision.
sandsmark added reviewers: ngraham, Frameworks, broulik.
sandsmark added a project: Frameworks.
Herald added a subscriber: kde-frameworks-devel.
sandsmark requested review of this revision.

REVISION SUMMARY
  Not entirely sure about this approach (and the stuff to get the test 
working), but I use kunitconversion in mangonel and the blocking behavior 
annoyed me a lot.
  
  A simpler approach would be to just try to check and download the cache 
immediately, but still force wait in the foreground, but the code (at least 
behavior) gets a lot more complex, especially with threading.

TEST PLAN
  The tests pass, it gets the currencies fairly quickly but doesn't block the 
UI thread.

REPOSITORY
  R292 KUnitConversion

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

AFFECTED FILES
  autotests/CMakeLists.txt
  autotests/convertertest.cpp
  autotests/convertertest.h
  autotests/valuetest.cpp
  autotests/valuetest.h
  src/currency.cpp
  src/currency_p.h

To: sandsmark, ngraham, #frameworks, broulik
Cc: broulik, ngraham, kde-frameworks-devel, #frameworks, LeGast00n, cblack, 
michaelh, bruns

Reply via email to