<div>Hi Rob,</div><div> </div><div>In du.c the human readable string for the size was printed in place. To make a common function</div><div>for both du and dd, I compiled this into pending.c. To return the reabable string</div>
<div>static char[] is used. This could be done with xmalloc and free, but that will require the user</div><div>of function to keep track of the returned pointer and free it.</div><div> </div><div>Other approach could have been to accept and char buffer as _out_ parameter for holding the string.</div>
<div> </div><div>your inputs are welcome.</div><div> </div><div>regards,</div><div>Ashwini<br><br></div><div class="gmail_quote">On Tue, Oct 29, 2013 at 6:32 AM, Rob Landley <span dir="ltr"><<a href="mailto:rob@landley.net" target="_blank">rob@landley.net</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">This one was deferred due to the common infrastructure changes. Applying it also modified a previously "done" command to use a lib/pending.c function, and a quick glance at that new function showed it adding a new static/global variable after I finally got rid of the itoa/utoa one. (I also need a lib/pending.h to go with lib/pending.c.)<br>

<br>
I appreciate the heads up and I'll try to carve out some time this week to at least catch up on the pending merges. (The time I've had recently went to build infrastructure fixes, cleaning up nl, and reading about 6000 of my 16,000 back emails. I'm less than 10,000 emails behind now! Woo!)<br>

<br>
Rob<div><div class="h5"><br>
<br>
On 10/16/2013 01:09:29 AM, Ashwini Sharma 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">
Hi Rob,<br>
incase you missed this mail, just to bring it on the top of ur list.<br>
<br>
regards,<br>
Ashwini<br>
On Tue, Sep 3, 2013 at 2:53 PM, Ashwini Sharma <<a href="mailto:ak.ashwini@gmail.com" target="_blank">ak.ashwini@gmail.com</a>> wrote:<br>
<br>
> Hi Rob,<br>
><br>
> Attached is the patch for DD.<br>
><br>
> I have also modified _du.c_, _pending.c_ and _lib.h_ to make a common<br>
> function for converting _size_ into human readable string.<br>
><br>
> Like to hear from you.<br>
><br>
> regards,<br>
> Ashwini<br>
><br>
<br>
</blockquote>
<br></div></div>
------quoted attachment------<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">
______________________________<u></u>_________________<br>
Toybox mailing list<br>
<a href="mailto:Toybox@lists.landley.net" target="_blank">Toybox@lists.landley.net</a><br>
<a href="http://lists.landley.net/listinfo.cgi/toybox-landley.net" target="_blank">http://lists.landley.net/<u></u>listinfo.cgi/toybox-landley.<u></u>net</a><br>
<br>
</blockquote>
<br>
</blockquote></div><br>