https://bugs.documentfoundation.org/show_bug.cgi?id=101563

Aron Budea <ba...@caesar.elte.hu> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |NEW
         Resolution|FIXED                       |---

--- Comment #45 from Aron Budea <ba...@caesar.elte.hu> ---
I think I found where this is coming from.
I put a breakpoint in the mentioned GetLink() function, which was called here
during opening the file:

pSwGrfNode->SetGraphic(aGrf, rGrfObj.GetLink());
http://opengrok.libreoffice.org/xref/core/sw/source/core/docnode/swbaslnk.cxx#166

Then I followed a few levels deeper:

void GraphicObject::SetGraphic( const Graphic& rGraphic, const OUString& rLink
)
{
    SetGraphic( rGraphic );
    maLink = rLink;
}

The problem here is that maLink and rLink are the same, and SetGraphic(
rGraphic ) clears maLink, so the link is lost.

I'd change the line above to this:

pSwGrfNode->SetGraphic(aGrf, OUString(rGrfObj.GetLink()));

(maybe with a comment mentioning it's intentional, since rGrfObj is coming from
pSwGrfNode a couple of lines earlier)

I tested this particular change in Windows with the previous version of
GetLink() (returning reference), and it fixed the size of PDF export for me in
Windows 7.
Noel, would you mind updating the fix?

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
Libreoffice-bugs@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to