[Okular-devel] [Bug 257370] "Trim Margins" cuts too much

[Fabio D'Urso]

https://bugs.kde.org/show_bug.cgi?id=257370

Fabio D'Urso <fabiodurso at hotmail.it> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |FIXED
      Latest Commit|                            |http://commits.kde.org/okul
                   |                            |ar/edbb4ef9f5aa8f120558b9d4
                   |                            |f4b9f68970100c4b

--- Comment #8 from Fabio D'Urso <fabiodurso at hotmail.it> ---
Git commit edbb4ef9f5aa8f120558b9d4f4b9f68970100c4b by Fabio D'Urso.
Committed on 17/07/2012 at 20:35.
Pushed by fabiod into branch 'KDE/4.9'.

Call Generator::signalPixmapRequestDone _after_ saving the calculated bounding
box

Fixes a bug that causes the extraction of a wrong bounding box:

If the request queue is not empty, signalPixmapRequestDone causes a new
pixmap request to be started, thus overwriting mPixmapGenerationThread's
mCalcBoundingBox before it is read by the if in the next line.

Now signalPixmapRequestDone is called after the bounding box is saved,
so that new requests are started only after all data from
mPixmapGenerationThread have been saved.
REVIEW: 105600

M  +1    -1    core/generator.cpp

http://commits.kde.org/okular/edbb4ef9f5aa8f120558b9d4f4b9f68970100c4b

-- 
You are receiving this mail because:
You are the assignee for the bug.