<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 18, 2023 at 3:14 AM Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On 1/17/23 20:00, enh via Toybox wrote:<br>
> fyi, there were some fly-by code review comments against git.c when i did the<br>
> last toybox update:<br>
> <br>
> <a href="https://android-review.googlesource.com/c/platform/external/toybox/+/2385040?tab=comments" rel="noreferrer" target="_blank">https://android-review.googlesource.com/c/platform/external/toybox/+/2385040?tab=comments</a> <br>
> <br>
> (i'm assuming moritz reads the list?)<br>
<br>
In theory, yes.<br>
<br>
I've done some very simple code style cleanup, but detoured into reading the gif<br>
file format docs before trying to work out object lifetime rules, and it's now<br>
like 6 things down on my stack because interrupts... :)<br>
<br>
(Pending. It's there for a reason. Seems like a good start though...)<br></blockquote><div><br></div><div>yeah, i just thought that since moritz is learning C, he'd be interested in trying to fix those things himself.</div><div><br></div><div>(to go with the already-given advice of "use asan", it's also probably worth suggesting he try coverage builds too, to know which parts of the code have/haven't been executed during testing.)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
Rob<br>
</blockquote></div></div>