readAll() is problematic, why to potentially fill all the ram completely, thus removing cache?<br>Such approaches can degrade overall plasma's performance. <br><br>Posted from a phone. <br><br>On Sunday, 26 June 2016, Elvis Angelaccio <<a href="mailto:elvis.angelaccio@kdemail.net">elvis.angelaccio@kdemail.net</a>> wrote:<br>> This is an automatically generated e-mail. To reply, visit: <a href="https://git.reviewboard.kde.org/r/128283/">https://git.reviewboard.kde.org/r/128283/</a><br>> Review request for KDE Frameworks, KDE Usability and David Faure.<br>> By Elvis Angelaccio.<br>><br>> Updated June 26, 2016, 10:34 a.m.<br>><br>> Changes<br>><br>> Implementation of the new layout. Checksums are cached to avoid duplicated computations.<br>><br>> Repository: kio<br>><br>> Description<br>><br>> This patch adds a Checksums tab in the properties dialog, where the user can retrieve and verify the most popular checksum algorithms (md5, sha1 and sha256).<br>><br>> To simplify the implementation, the checksums are computed as soon as the user opens the dialog. This can take a while if the file is huge (in particular with sha256), but the computation happens in another thread and in practice this should not be a performance problem.<br>><br>> The tab is available only for readable local files (no simlinks) and only when there is a single selection.<br>><br>> Please note that some of the labels in the screenshots are clipped due to a bug in breeze: <a href="https://bugs.kde.org/show_bug.cgi?id=364426">https://bugs.kde.org/show_bug.cgi?id=364426</a><br>><br>> Testing<br>><br>> Check whether the computed values match the values from md5sum, sha1sum, sha256sum.<br>> Check whether the line edits get a green background if the computed and expected values match.<br>> Check whether the line edits get a red background if the computed and expected values differ.<br>><br>> Diffs (updated)<br>><br>> src/widgets/CMakeLists.txt (f906577)<br>> src/widgets/checksumswidget.ui (PRE-CREATION)<br>> src/widgets/kpropertiesdialog.cpp (d0a2faa)<br>> src/widgets/kpropertiesdialog_p.h (c01554e)<br>><br>> View Diff<br>><br>> File Attachments (updated)<br>><br>> Default dialog<br>> Expected checksum is ok<br>> Expected checksum doesn't match<br>> MD5 ready to be shared<br><br>-- <br>regards, Jaroslaw Staniek<br><br>KDE:<br>: A world-wide network of software engineers, artists, writers, translators<br>: and facilitators committed to Free Software development - <a href="http://kde.org" target="_blank">http://kde.org</a><br>Calligra Suite:<br>: A graphic art and office suite - <a href="http://calligra.org" target="_blank">http://calligra.org</a><br>Kexi:<br>: A visual database apps builder - <a href="http://calligra.org/kexi" target="_blank">http://calligra.org/kexi</a><br>Qt Certified Specialist:<br>: <a href="http://www.linkedin.com/in/jstaniek" target="_blank">http://www.linkedin.com/in/jstaniek</a><br>