Details
Details
- Reviewers
dfaure - Commits
- R243:2225527bb6be: Fix my fix for KCompressionDevice::seek
Diff Detail
Diff Detail
- Repository
- R243 KArchive
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
src/kcompressiondevice.cpp | ||
---|---|---|
208 | I wonder if we could avoid seeking twice, by doing seek(pos) only in the other cases (the deviceReadPos ==pos early return, and the pos == 0 case above). We don't need the seek(pos) in this if(), nor in the else(). |
Comment Actions
We can save some seek calls, but we still need it in the if, there's nothing that guaranteers that "QIODevice::pos" will be d->deviceReadPos so we need to seek to d->deviceReadPos.