Bug 68730 - EMF wrongly rendered
Summary: EMF wrongly rendered
Status: RESOLVED FIXED
Alias: None
Product: LibreOffice
Classification: Unclassified
Component: graphics stack (show other bugs)
Version:
(earliest affected)
4.2.0.0.alpha0+ Master
Hardware: Other All
: medium normal
Assignee: Fridrich Strba
URL:
Whiteboard:
Keywords: regression
Depends on:
Blocks:
 
Reported: 2013-08-29 22:16 UTC by Michael Stahl (allotropia)
Modified: 2013-08-30 17:46 UTC (History)
2 users (show)

See Also:
Crash report or crash signature:

Attachments
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Stahl (allotropia) 2013-08-29 22:16:21 UTC 
with this attachment:

https://bugs.freedesktop.org/attachment.cgi?id=51359

the EMF image on page 5 looks wrong in current master,
the drawings and the text are out of whack,
was working in 4.0.5.2.
Comment 1 Michael Stahl (allotropia) 2013-08-30 11:22:38 UTC 
bibisecting this is scary, i think i've seen 3 differently wrong
renderings of this along the way...

bibisect says: 04c2e4f5836ee72a36da69846e5efe41a0dde83d..3a6513047409b8e8f6295152b33385ad8ef93681

after reverting both of these the rendering looks right:


commit 2b700a710e44d2d09f45793f5f2448268e66d4fe
Author:     Fridrich Štrba <fridrich.strba@bluewin.ch>
AuthorDate: Wed Jun 12 10:03:28 2013 +0200

    EMF+: aBaseTransform is a hardcoded identity matrix, don't bother with it
    
commit f387c5da5a824df6cf2874155c13445a86804475
Author:     Fridrich Štrba <fridrich.strba@bluewin.ch>
AuthorDate: Wed Jun 12 07:03:16 2013 +0200

    Pass also the bounding box in device units to the EMF+ part
Comment 2 Michael Stahl (allotropia) 2013-08-30 17:46:08 UTC 
Fridrich fixed it in commit 21ee132af1b888f50a9d916337011879570ca357, thanks