<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jun 25, 2013 at 4:50 AM, Fabio D'Urso <span dir="ltr"><<a href="mailto:fabiodurso@hotmail.it" target="_blank">fabiodurso@hotmail.it</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid"><div class="im">On Monday, June 24, 2013 11:57:35 PM Jaydeep Solanki wrote:<br>

> Hello,<br>
> I have put together some of my work, in a branch named 'epub-qtextdoc'.<br>
><br>
> It would be really nice if you can review it.<br>
<br>
</div>Hi Jaydeep,<br>
<br>
I've never worked on epubs, therefore I can't really comment on your code.<br>
<br>
I noticed that there are some indentation changes in your patch "epub change<br>
document loading method" that affect code that would otherwise be unaffected.<br>
Such changes make diffs unnecessarily long and harder to read, because it's<br>
not clear what code has been changed or not (and they also confuse git's<br>
history tracking).<br>
<br>
Sorry again for not being able to review the actual code,<br></blockquote><div>I agree with you, on this.</div><div>The file was previously using indentation of width 2, & I used width 4.</div><div>Would it be a good idea to push another commit with indent width 2 ? because again it will create diff, that has indentation change.</div>
<div>Or Albert can just kill the branch, & I'll push it again, with indent width 2 + some documentation, for making it easier to read.</div><div> </div><div>Cheers,</div><div>Jaydeep</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;padding-left:1ex;border-left-color:rgb(204,204,204);border-left-width:1px;border-left-style:solid">

<br>
Fabio<br>
<div class="HOEnZb"><div class="h5"><br>
_______________________________________________<br>
Okular-devel mailing list<br>
<a href="mailto:Okular-devel@kde.org">Okular-devel@kde.org</a><br>
<a href="https://mail.kde.org/mailman/listinfo/okular-devel" target="_blank">https://mail.kde.org/mailman/listinfo/okular-devel</a><br>
</div></div></blockquote></div><br></div></div>