<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [PATCH] Arthur support for Type3 fonts"
href="https://bugs.freedesktop.org/show_bug.cgi?id=105772#c5">Comment # 5</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [PATCH] Arthur support for Type3 fonts"
href="https://bugs.freedesktop.org/show_bug.cgi?id=105772">bug 105772</a>
from <span class="vcard"><a class="email" href="mailto:aacid@kde.org" title="Albert Astals Cid <aacid@kde.org>"> <span class="fn">Albert Astals Cid</span></a>
</span></b>
<pre>(In reply to oliver.sander from <a href="show_bug.cgi?id=105772#c4">comment #4</a>)
<span class="quote">> >> - Why white? Why not any other color?
> > Because the code is buggy
>
> I'd happily try to fix the code, if somebody tells me what the startPage
> method is supposed to do.</span >
To be honest i don't think there's a definition of what is supposed to do other
than "do the things that you need to do when starting a new page"
<span class="quote">>
> >> - Apparently the method is rarely called: The renderToArthur method
> >> in file poppler-page.cc, for example, does not call it.
>
> > Are you sure about that? renderToArthur calls displayPageSlice that calls displaySlice that calls createGfx that calls Gfx constructor with pagenum that calls startPage
>
> Well, I said 'rarely', not 'never'. :-) </span >
I don't understand this, you said "renderToArthur method in file
poppler-page.cc, for example, does not call it." and i showed you that it does.
<span class="quote">> In particular, I had to make the
> renderToImage method fill the page with the page color, to get a background
> at all (in 7e844eae94bc4eda1c6dcc3460840b25f4ca7898) The code in startPage
> didn't seem to have any effect. But I readily admit that I was never quite
> sure what the proper location is for the code that initializes the page
> background.
>
> Tangentially related: What is the precise meaning of the IgnorePaperColor
> flag? Does it mean 'make the background transparent'?</span >
Yes, i'd say you awnt to hide that tmpimg.fill(bgColor); inside a check for not
IgnorePaperColor
<span class="quote">> >> - Most importantly: We want to use ArthurOutputDev to render type3
> >> glyphs. But they should have a transparent background, or rather,
> >> no background at all.
>
> > I don't understand this reasoning.
>
> The type3 code in the attached patch is partially cargo-culted from the
> cairo backend. It creates new ArthurOutputDev and Gfx objects for each
> glyph, and uses them to render a single glyph into a QPicture. The cairo
> backend calls startPage on the output device, which in the case of the
> ArthurOutputDev leads to the effect that each glyph has a white rectangular
> background. You will notice that when rendering type3 text on a non-white
> background.</span >
I see.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are the assignee for the bug.</li>
</ul>
</body>
</html>