
On 11-Feb-2014 10:26, Tavmjong Bah wrote:
On Tue, 2014-02-11 at 09:57 -0800, mathog wrote:
Also, regarding bug 1278645, setting the above line in the <image> fixes the EMF/WMF image output. Which tells us that the default value for preserveAspectRatio is doing something wrong for the one image in the test case. That test is from start to finish just a (10,10) bitmap, but for some reason it gets trimmed down to a (5,5) in SPImage::print in sp-image.cpp in the default case, but stays (10,10) for "none".
There is an "if" statement in SPImage::print which I believe handles non-Cairo output that special cases 'none' from the rest. This is where to look for the fix.
That tests only
if (this->aspect_align == SP_ASPECT_NONE) {
there is nothing Cairo specific about that. The value for aspect_align at that point is 5 == SP_ASPECT_XMID_YMID. Hmm, "save as" operations that go through cairo never enter SPImage::print, and do output correctly. Now to find where those are going and compare the code in the two.
I don't have an easy way to view wmf and emf files...
Use inkscape trunk on any platform?
Thanks,
David Mathog mathog@...1176... Manager, Sequence Analysis Facility, Biology Division, Caltech