<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - Add option to pdfinfo to print javascript" href="https://bugs.freedesktop.org/show_bug.cgi?id=67693#c9">Comment # 9</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - Add option to pdfinfo to print javascript" href="https://bugs.freedesktop.org/show_bug.cgi?id=67693">bug 67693</a> from <a class="email" href="mailto:aacid@kde.org" title="Albert Astals Cid <aacid@kde.org>"> Albert Astals Cid</a> <pre>After a review of the code i can't find anything wrong. The JSUtil* files seem like they belong more to utils/ than to poppler/ specially after the addition of the "Field Activated" things, that would not be much useful to an external program as they are untranslatable (maybe an enum and then the char * in pdfinfo makes more sense and then keep JSUtil* in poppler/) Also, I understand you've run the new code over all the files and got no crashes because of the new code?</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the assignee for the bug.</li> </ul> </body> </html>