<br>Hello list,<br><br>Can anybody help me to understand the file utilis/pdftohtml.1. We wanted to know the purpose of .1 files and how much important to modify this whenever we do changes in related files like pdftohtml.cc and others.<br>
<br>Thanks &amp; regards<br>Leena C<br><br><div class="gmail_quote">On Sun, Jul 25, 2010 at 9:27 PM, Albert Astals Cid <span dir="ltr">&lt;<a href="mailto:aacid@kde.org">aacid@kde.org</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
A Dimarts, 6 de juliol de 2010, leena chourey va escriure:<br>
&gt; Dear Albert,<br>
<br>
Hi<br>
<div class="im"><br>
&gt; Thanks for your response.<br>
&gt;<br>
&gt; As discussed in the last mail, we have modified the patch so that:<br>
&gt;<br>
&gt;    - There is no behavioural change in pdftohtml -c &lt;filename&gt; means it<br>
&gt;    produces exactly the same output it did before.<br>
&gt;    - Defined new option as pdftohtml -s &lt;filename&gt; to generate a single<br>
&gt; html file corresponding to a pdf file.<br>
&gt;<br>
&gt; Please check and give your feedback if any further change is required.<br>
<br>
</div>You are using a variable you deleted (tmp) in this chunk of code<br>
<br>
***********************<br>
       delete tmp;<br>
<br>
-      fprintf(pageFile,&quot;%s\n&lt;HTML&gt;\n&lt;HEAD&gt;\n&lt;TITLE&gt;Page %d&lt;/TITLE&gt;\n\n&quot;,<br>
-             DOCTYPE, page);<br>
+      if (!singleHtml)<br>
+          fprintf(pageFile,&quot;%s\n&lt;HTML&gt;\n&lt;HEAD&gt;\n&lt;TITLE&gt;Page %d&lt;/TITLE&gt;\n\n&quot;,<br>
DOCTYPE, page);<br>
+      else<br>
+          fprintf(pageFile,&quot;%s\n&lt;HTML&gt;\n&lt;HEAD&gt;\n&lt;TITLE&gt; %s&lt;/TITLE&gt;\n\n&quot;,<br>
DOCTYPE, tmp-&gt;getCString());////file name<br>
***********************<br>
<br>
I&#39;m also concerned about you adding various &lt;HTML&gt; to the same .html page, my<br>
limited HTML knowledge says you can only have one of those.<br>
<br>
Also it would be necessary that you update the pdftohtml.1 file (the man page)<br>
adding the new option.<br>
<font color="#888888"><br>
Albert<br>
</font><div><div></div><div class="h5"><br>
&gt;<br>
&gt; With best regard<br>
&gt; Leena C<br>
&gt;<br>
&gt; On Wed, Jun 23, 2010 at 1:19 AM, Albert Astals Cid &lt;<a href="mailto:aacid@kde.org">aacid@kde.org</a>&gt; wrote:<br>
&gt; &gt; A Dimarts, 22 de juny de 2010, leena chourey va escriure:<br>
&gt; &gt; &gt; Dear Albert,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Thanks for giving detail comment to patch.<br>
&gt; &gt;<br>
&gt; &gt; &gt; Please check updates given inline:<br>
&gt; &gt; Please do not forget to CC the poppler mailing list.<br>
&gt; &gt;<br>
&gt; &gt; &gt; On Thu, Jun 17, 2010 at 4:14 AM, Albert Astals Cid &lt;<a href="mailto:aacid@kde.org">aacid@kde.org</a>&gt;<br>
&gt; &gt;<br>
&gt; &gt; wrote:<br>
&gt; &gt; &gt; &gt; A Dimecres, 16 de juny de 2010, omkar va escriure:<br>
&gt; &gt; &gt; &gt; &gt; Dear Albert,<br>
&gt; &gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; &gt; Please find the corrected patch for &quot;accessibility of pdf document<br>
&gt; &gt; &gt; &gt; &gt; &quot; and give your feedback.<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; Hi, some comments:<br>
&gt; &gt; &gt; &gt;  * The comments like<br>
&gt; &gt; &gt; &gt;  // One more parameter(int j) is added in the getCSStyle function by<br>
&gt; &gt;<br>
&gt; &gt; CDAC<br>
&gt; &gt;<br>
&gt; &gt; &gt; &gt; developer Team<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt;   need to be removed, if each line had near it who coded it, the code<br>
&gt; &gt; &gt; &gt;   will<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; be<br>
&gt; &gt; &gt; &gt; twice as big and much more unreadable<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Done, deleted all unwanted comments<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt;   * The spacing of your patches could be better, that is<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; GooString* HtmlFontAccu::getCSStyle(int i, GooString* content ,int<br>
&gt; &gt; &gt; &gt; j){ should be<br>
&gt; &gt; &gt; &gt; +GooString* HtmlFontAccu::getCSStyle(int i, GooString* content, int<br>
&gt; &gt; &gt; &gt; j){ but that&#39;s nothing huge, i can fix it<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Updated accordingly.<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt;   * You are leaking (i.e. not deleting) jStr in both<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; HtmlFontAccu::getCSStyle<br>
&gt; &gt; &gt; &gt; and HtmlFontAccu::CSStyle<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Deleted jStr<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt;  * I see that the new HtmlPage::complexHtml and the old<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt; HtmlPage::dumpComplex<br>
&gt; &gt; &gt; &gt; are very simple, i if you reused the code instead of copying it<br>
&gt; &gt; &gt; &gt;<br>
&gt; &gt; &gt; &gt;  * This introduces a behavioural change that is unaccetable, i<br>
&gt; &gt;<br>
&gt; &gt; understand<br>
&gt; &gt;<br>
&gt; &gt; &gt; &gt; you<br>
&gt; &gt; &gt; &gt; want pdftohtml to produce a different (in your opinion better)<br>
&gt; &gt; &gt; &gt; output, for that you&#39;ll have to introduce a new comandline option to<br>
&gt; &gt; &gt; &gt; pdftohtml (something<br>
&gt; &gt; &gt; &gt; like --singlehtml) or something like that<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; For last 2 point we want some clarification.<br>
&gt; &gt; &gt; As you said behavioural change is unacceptable and also suggested to<br>
&gt; &gt; &gt; introduce a new command line option to generate single html. So if we<br>
&gt; &gt; &gt; do<br>
&gt; &gt;<br>
&gt; &gt; as<br>
&gt; &gt;<br>
&gt; &gt; &gt; following, will it be acceptable?<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    - *Existing is:*<br>
&gt; &gt; &gt;    Command line option: pdftohtml -c  &lt;filename&gt;<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    Function called:<br>
&gt; &gt; &gt;                       dumpComplex<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    ()<br>
&gt; &gt; &gt;    {<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;        Read from input file<br>
&gt; &gt; &gt;        Write into file to Generates pagewise html format<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    }<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    - *Proposed changes:*<br>
&gt; &gt; &gt;    New Command line option : pdftohtml -s &lt;filename&gt;         //Checked,<br>
&gt; &gt; &gt;    nothing  is already defined for -s            (pdftohtml -c<br>
&gt; &gt;<br>
&gt; &gt;  &lt;filename&gt;<br>
&gt; &gt;<br>
&gt; &gt; &gt; will exists as it is)<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    - Function called:<br>
&gt; &gt; &gt;                          dumpSingle()  //new function similar to<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; dumpComplex {<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;        Read from input file<br>
&gt; &gt; &gt;        Write into file to append single html format<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    }<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    - A function to “Read from input file” can be defined and call it in<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; both dumpComplex() and dumpSingle(), So that code duplication can be<br>
&gt; &gt; &gt; removed (for second last point of your mail).<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt;    - And with -s option (for --single Html) behavioural change will be<br>
&gt; &gt; &gt;    defined separately. (-c will not be affected)<br>
&gt; &gt;<br>
&gt; &gt; To be clear, pdftohtml -c should produce exactly the same output it did<br>
&gt; &gt; before<br>
&gt; &gt; your patch, pdftohtml -s you can output your version.<br>
&gt; &gt;<br>
&gt; &gt; So yes, i think i kind of agree with your proposal.<br>
&gt; &gt;<br>
&gt; &gt; Albert<br>
&gt; &gt;<br>
&gt; &gt; &gt; For your opinion<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; With Regards<br>
&gt; &gt; &gt; Leena C &amp; Onkar P<br>
&gt; &gt; &gt; (for CDAC Accessibility Team)<br>
</div></div></blockquote></div><br>