<html>
    <head>
      <base href="https://bugs.freedesktop.org/">
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Embedded TrueType Symbols with accents not rendered correctly"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101855#c5">Comment # 5</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - Embedded TrueType Symbols with accents not rendered correctly"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=101855">bug 101855</a>
              from <span class="vcard"><a class="email" href="mailto:simon.shugar@veeva.com" title="Simon Shugar <simon.shugar@veeva.com>"> <span class="fn">Simon Shugar</span></a>
</span></b>
        <pre>Created <span class=""><a href="attachment.cgi?id=132794" name="attach_132794" title="Patch to add">attachment 132794</a> <a href="attachment.cgi?id=132794&action=edit" title="Patch to add">[details]</a></span>
Patch to add

Added useMacRoman = gTrue to ((flags & fontSymbolic) && macRomanCmap >= 0) 
else if. 

My question is should we just remove ((flags & fontSymbolic) && macRomanCmap >=
0) and rely on (macRomanCmap >= 0) or does it have a purpose? I looked at the
history and this logic was added as a merge from XPDF however the comment about
the logic was never updated. 

Comment Logic:

  // To match up with the Adobe-defined behaviour, we choose a cmap
  // like this:
  // 1. If the PDF font has an encoding:
  //    1a. If the PDF font specified MacRomanEncoding and the
  //        TrueType font has a Macintosh Roman cmap, use it, and
  //        reverse map the char names through MacRomanEncoding to
  //        get char codes.
  //    1b. If the PDF font is not symbolic or the PDF font is not
  //        embedded, and the TrueType font has a Microsoft Unicode
  //        cmap or a non-Microsoft Unicode cmap, use it, and use the
  //        Unicode indexes, not the char codes.
  //    1c. If the PDF font is symbolic and the TrueType font has a
  //        Microsoft Symbol cmap, use it, and use char codes
  //        directly (possibly with an offset of 0xf000).
  //    1d. If the TrueType font has a Macintosh Roman cmap, use it,
  //        as in case 1a.
  // 2. If the PDF font does not have an encoding or the PDF font is
  //    symbolic:
  //    2a. If the TrueType font has a Macintosh Roman cmap, use it,
  //        and use char codes directly (possibly with an offset of
  //        0xf000).
  //    2b. If the TrueType font has a Microsoft Symbol cmap, use it,
  //        and use char codes directly (possible with an offset of
  //        0xf000).
  // 3. If none of these rules apply, use the first cmap and hope for
  //    the best (this shouldn't happen).</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>