<br><br><div class="gmail_quote">---------- Forwarded message ----------<br>From: <b class="gmail_sendername">leena chourey</b> <span dir="ltr">&lt;<a href="mailto:leenagour@gmail.com">leenagour@gmail.com</a>&gt;</span><br>
Date: Tue, Jul 6, 2010 at 4:49 PM<br>Subject: Re: [poppler] Accessibility of PDF documents (corrected patch attached)<br>To: Albert Astals Cid &lt;<a href="mailto:aacid@kde.org">aacid@kde.org</a>&gt;<br>Cc: <a href="mailto:poppler@lists.freedesktop.org">poppler@lists.freedesktop.org</a>, onkar &lt;<a href="mailto:onkarpotdar@gmail.com">onkarpotdar@gmail.com</a>&gt;, Aparna Ramamurthy &lt;<a href="mailto:aparna.ramamurthy@gmail.com">aparna.ramamurthy@gmail.com</a>&gt;<br>
<br><br>Dear Albert,<br><br>Thanks for your response.<br><br>As discussed in the last mail, we have modified the patch so that:<br><ul><li>There is no behavioural change in pdftohtml -c &lt;filename&gt; means it produces exactly the same output it did before. </li>



<li>Defined new option as pdftohtml -s &lt;filename&gt; to generate a single html file corresponding to a pdf file.</li></ul>Please check and give your feedback if any further change is required.<br><br>With best regard<br>



Leena C<div><div></div><div class="h5"><br><br><div class="gmail_quote">On Wed, Jun 23, 2010 at 1:19 AM, Albert Astals Cid <span dir="ltr">&lt;<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>&gt;</span> wrote:<br>
<blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">


A Dimarts, 22 de juny de 2010, leena chourey va escriure:<br>
<div>&gt; Dear Albert,<br>
&gt;<br>
&gt; Thanks for giving detail comment to patch.<br>
&gt; Please check updates given inline:<br>
<br>
</div>Please do not forget to CC the poppler mailing list.<br>
<div><div></div><div><br>
&gt;<br>
&gt; On Thu, Jun 17, 2010 at 4:14 AM, Albert Astals Cid &lt;<a href="mailto:aacid@kde.org" target="_blank">aacid@kde.org</a>&gt; wrote:<br>
&gt; &gt; A Dimecres, 16 de juny de 2010, omkar va escriure:<br>
&gt; &gt; &gt; Dear Albert,<br>
&gt; &gt; &gt;<br>
&gt; &gt; &gt; Please find the corrected patch for &quot;accessibility of pdf document &quot;<br>
&gt; &gt; &gt; and give your feedback.<br>
&gt; &gt;<br>
&gt; &gt; Hi, some comments:<br>
&gt; &gt;  * The comments like<br>
&gt; &gt;  // One more parameter(int j) is added in the getCSStyle function by CDAC<br>
&gt; &gt;<br>
&gt; &gt; developer Team<br>
&gt; &gt;<br>
&gt; &gt;   need to be removed, if each line had near it who coded it, the code<br>
&gt; &gt;   will<br>
&gt; &gt;<br>
&gt; &gt; be<br>
&gt; &gt; twice as big and much more unreadable<br>
&gt;<br>
&gt; Done, deleted all unwanted comments<br>
&gt;<br>
&gt; &gt;   * The spacing of your patches could be better, that is<br>
&gt; &gt;<br>
&gt; &gt; GooString* HtmlFontAccu::getCSStyle(int i, GooString* content ,int j){<br>
&gt; &gt; should be<br>
&gt; &gt; +GooString* HtmlFontAccu::getCSStyle(int i, GooString* content, int j){<br>
&gt; &gt; but that&#39;s nothing huge, i can fix it<br>
&gt;<br>
&gt; Updated accordingly.<br>
&gt;<br>
&gt; &gt;   * You are leaking (i.e. not deleting) jStr in both<br>
&gt; &gt;<br>
&gt; &gt; HtmlFontAccu::getCSStyle<br>
&gt; &gt; and HtmlFontAccu::CSStyle<br>
&gt;<br>
&gt; Deleted jStr<br>
&gt;<br>
&gt; &gt;  * I see that the new HtmlPage::complexHtml and the old<br>
&gt; &gt;<br>
&gt; &gt; HtmlPage::dumpComplex<br>
&gt; &gt; are very simple, i if you reused the code instead of copying it<br>
&gt; &gt;<br>
&gt; &gt;  * This introduces a behavioural change that is unaccetable, i understand<br>
&gt; &gt;<br>
&gt; &gt; you<br>
&gt; &gt; want pdftohtml to produce a different (in your opinion better) output,<br>
&gt; &gt; for that you&#39;ll have to introduce a new comandline option to pdftohtml<br>
&gt; &gt; (something<br>
&gt; &gt; like --singlehtml) or something like that<br>
&gt;<br>
&gt; For last 2 point we want some clarification.<br>
&gt; As you said behavioural change is unacceptable and also suggested to<br>
&gt; introduce a new command line option to generate single html. So if we do as<br>
&gt; following, will it be acceptable?<br>
&gt;<br>
&gt;    - *Existing is:*<br>
&gt;    Command line option: pdftohtml -c  &lt;filename&gt;<br>
&gt;    Function called:<br>
&gt;<br>
&gt;<br>
&gt;                       dumpComplex<br>
&gt;    ()<br>
&gt;    {<br>
&gt;        Read from input file<br>
&gt;        Write into file to Generates pagewise html format<br>
&gt;    }<br>
&gt;<br>
&gt;<br>
&gt;    - *Proposed changes:*<br>
&gt;    New Command line option : pdftohtml -s &lt;filename&gt;         //Checked,<br>
&gt;    nothing  is already defined for -s            (pdftohtml -c  &lt;filename&gt;<br>
&gt; will exists as it is)<br>
&gt;<br>
&gt;<br>
&gt;    - Function called:<br>
&gt;<br>
&gt;                          dumpSingle()  //new function similar to<br>
&gt; dumpComplex {<br>
&gt;        Read from input file<br>
&gt;        Write into file to append single html format<br>
&gt;    }<br>
&gt;<br>
&gt;    - A function to “Read from input file” can be defined and call it in<br>
&gt; both dumpComplex() and dumpSingle(), So that code duplication can be<br>
&gt; removed (for second last point of your mail).<br>
&gt;    - And with -s option (for --single Html) behavioural change will be<br>
&gt;    defined separately. (-c will not be affected)<br>
<br>
</div></div>To be clear, pdftohtml -c should produce exactly the same output it did before<br>
your patch, pdftohtml -s you can output your version.<br>
<br>
So yes, i think i kind of agree with your proposal.<br>
<font color="#888888"><br>
Albert<br>
</font><div><div></div><div><br>
<br>
&gt;<br>
&gt;<br>
&gt; For your opinion<br>
&gt;<br>
&gt; With Regards<br>
&gt; Leena C &amp; Onkar P<br>
&gt; (for CDAC Accessibility Team)<br>
</div></div></blockquote></div><br><br clear="all"><br></div></div><font color="#888888">-- <br>Leena C<br>
</font></div><br><br clear="all"><br>-- <br>Leena C<br>