<html>
    <head>
      <base href="https://bugs.freedesktop.org/" />
    </head>
    <body>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - support for digital signatures"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=16770#c75">Comment # 75</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - support for digital signatures"
   href="https://bugs.freedesktop.org/show_bug.cgi?id=16770">bug 16770</a>
              from <span class="vcard"><a class="email" href="mailto:ajohnson@redneon.com" title="Adrian Johnson <ajohnson@redneon.com>"> <span class="fn">Adrian Johnson</span></a>
</span></b>
        <pre>(In reply to Albert Astals Cid from <a href="show_bug.cgi?id=16770#c74">comment #74</a>)
<span class="quote">> (In reply to Adrian Johnson from <a href="show_bug.cgi?id=16770#c73">comment #73</a>)
> > (In reply to Albert Astals Cid from <a href="show_bug.cgi?id=16770#c72">comment #72</a>)
> > > > Large file support.
> > > 
> > > Do you think this should be a blocker for the initial release or something
> > > we can improve later?
> > 
> > I think it should be included.

> Do you have any file like that or a way to create it so the guys can test it?</span >

I don't have a file. <a class="bz_bug_link 
          bz_status_RESOLVED  bz_closed"
   title="RESOLVED FIXED - pdfinfo/pdffonts cannot process >2GB files"
   href="show_bug.cgi?id=44085">Bug 44085</a> contains a large file test case. But I don't
know how to sign it.

Implementing large file support is simply a case of using obj.isInt64()/
obj.getInt64() when getting the range offsets.

In addition, the code should avoid reading the entire range into memory instead
processing it in smaller chunks. From a quick look a the nss headers it appears
to have support for incremental hashing. This is good practice even for < 2GB
files as I have seen scanned PDFs that are in the 100's of MB range.

I don't mind adding the getInt64() stuff myself as that is easy to do without
needing to understand the rest of the code. But the incremental hashing would
best be done by the original developer of the patch.

So at a a minimum if we can get the incremental hashing in then I can sort out
the rest of the 64-bit support.</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>