fvogt added inline comments. INLINE COMMENTS
> apol wrote in svg.cpp:317 > Please note this is only to make sure the regex was properly compiled. It > isn't matching there yet. It really does not look that way as you're immediately using captures after that. If that's really what you want (which I doubt), it should be `sizeHintedKeyExpr.isValid()` instead and be done before the foreach. Currently it would just add `QSize(0, 0)` to `elementsWithSizeHints` even for "öoiawze9pv5z2p3v4" as key. REPOSITORY R242 Plasma Framework (Library) REVISION DETAIL https://phabricator.kde.org/D22147 To: apol, #plasma, #frameworks, fvogt Cc: tcanabrava, fvogt, broulik, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns