<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>