<table><tr><td style="">dfaure added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D13584">View Revision</a></tr></table><br /><div><div><p>I don't really know this code, but one thing that could be done to ensure that the change is safe, is to increase unittest coverage (which is already not too bad, but more would be safer)</p>

<p><a href="https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.10/100/cobertura/src_lib_util/kformatprivate_cpp/" class="remarkup-link" target="_blank" rel="noreferrer">https://build.kde.org/job/Frameworks%20kcoreaddons%20kf5-qt5%20SUSEQt5.10/100/cobertura/src_lib_util/kformatprivate_cpp/</a></p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R244 KCoreAddons</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D13584">https://phabricator.kde.org/D13584</a></div></div><br /><div><strong>To: </strong>bruns, Frameworks<br /><strong>Cc: </strong>dfaure, astippich, kde-frameworks-devel, michaelh, ngraham, bruns<br /></div>