<table><tr><td style="">knauss 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/D3203" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>This roundtrip may makes a difference if you have non UTF-8 encoded files- did you check against those? I know that I added those roundtrips at other places, because otherwise the string weren't formatted correctly.</p>

<p>Removing the trimmed function means, that for files with only spaces, you do not return true, but will push this to fromRawString function. Is this function aware of those files? Do we will get the same result aka true?</p>

<p>It would be nice if you add tests for this, to make sure nothing is changed.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>rKCALENDARCORE KCalendar Core</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3203" rel="noreferrer">https://phabricator.kde.org/D3203</a></div></div><br /><div><strong>EMAIL PREFERENCES</strong><div><a href="https://phabricator.kde.org/settings/panel/emailpreferences/" rel="noreferrer">https://phabricator.kde.org/settings/panel/emailpreferences/</a></div></div><br /><div><strong>To: </strong>dfaure, djarvie, smartins<br /><strong>Cc: </strong>knauss, kde-pim, KDE PIM, spencerb, dvasin, winterz, vkrause, mlaurent, dvratil<br /></div>