<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">2016-08-25 14:09 GMT+03:00 Pino Toscano <span dir="ltr"><<a href="mailto:ptoscano@redhat.com" target="_blank">ptoscano@redhat.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Wednesday, 24 August 2016 23:59:53 CEST Matteo Cafasso wrote:<br>
> The find_inode API allows the User to search all the entries referring<br>
> to a given inode and returns a tsk_dirent structure for each of them.<br>
><br>
> As I didn't want to change unrelated code, there is a little bit<br>
> of code duplication at the moment. Plan is to refactor the logic<br>
> in a dedicated set of patches.<br>
<br>
</span>The general idea looks ok, but I'd rather see the duplication dealt<br>
with sooner than later.<br></blockquote><div>In the previous submissions, non related changes were rejected therefore I thought that was the custom.<br><br></div><div>Moreover I'll add another API find_block (block_number --> tsk_dirents referring to it) and I think is easier to refactor the code once all the use cases are in place as the picture gets more clear.<br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks,<br>
<span class="HOEnZb"><font color="#888888">--<br>
Pino Toscano</font></span><br>______________________________<wbr>_________________<br>
Libguestfs mailing list<br>
<a href="mailto:Libguestfs@redhat.com">Libguestfs@redhat.com</a><br>
<a href="https://www.redhat.com/mailman/listinfo/libguestfs" rel="noreferrer" target="_blank">https://www.redhat.com/<wbr>mailman/listinfo/libguestfs</a><br></blockquote></div><br></div></div>