<html>
<head>
<base href="https://bugs.freedesktop.org/">
</head>
<body>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - make it possible to extract digital signature data (also in pdfsig)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=99271#c20">Comment # 20</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - make it possible to extract digital signature data (also in pdfsig)"
href="https://bugs.freedesktop.org/show_bug.cgi?id=99271">bug 99271</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>Some comments:
* Please provide a commit log
* The code is full of memory leaks, every time you "use" an Object, you need
to free() it, yes the API is awful, maybe you just want to wait until i merge
my better_object branch so that you don't have to worry about it
* Please move the getHashAlgorithmName function to pdfsig, it is something
that i guess some people would like to translate (CJK?, Arabic?) and we don't
provide any translations support on the poppler core, so better not to have
things that people may want to show on a UI on poppler core either
* Don't break the ABI on the Qt side of things, just introduce another
function
* The extra code in FormFieldSignature::validate seems like it really should
be in the core since other bindings will want to do the same?</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>