We don't need to convert the icon which is loaded by name to a pixmap here. The conversion happens later in the DecorationRole data() call.
Details
Details
- Reviewers
elvisangelaccio - Group Reviewers
Ark - Commits
- R36:81828188101f: Don't covert icons to pixmaps multiple times
Open archive. Icons in the file list appear and are properly sized.
Trigger overwrite dialog. Icons appear and are properly sized.
Verified performance improvement with hotspot (see highlighted entries)
before:
after:
Diff Detail
Diff Detail
- Repository
- R36 Ark
- Branch
- conv
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 18917 Build 18935: arc lint + arc unit