<table><tr><td style="">elvisangelaccio updated the summary for this revision. <a href="https://phabricator.kde.org/transactions/detail/PHID-XACT-DREV-3kp776ahmglqpwf/" rel="noreferrer">(Show Details)</a>
</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/D3272" rel="noreferrer">View Revision</a></tr></table><br /><div><strong>EDIT DETAILS</strong><div><div style="white-space: pre-wrap; color: #74777D;">Static objects should be avoided if possible. The only reason we needed this QMap as global static was because of its usage in ArchiveModelSorter, but this class is gone as of D3271.<span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);"> So we can just make the QMap a member of the model.<br />
<br />
While at it, make it a `QMap<int, QByteArray>` instead of `QMap<int, QString>`.</span> <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">So we can just make</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">This avoids</span> the <span style="padding: 0 2px; color: #802b2b; background: rgba(251, 175, 175, .7);">QMap a member of the model</span><span style="padding: 0 2px; color: #3e6d35; background: rgba(151, 234, 151, .6);">need of converting back (using `QString::toUtf8()`)</span>.</div></div></div><br /><div><strong>REPOSITORY</strong><div><div>rARK Ark</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D3272" rel="noreferrer">https://phabricator.kde.org/D3272</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>elvisangelaccio, rthomsen<br /><strong>Cc: </strong>kde-utils-devel, tctara<br /></div>