<div dir="auto">Well.. it seems the mail list forgot to notify me over time. I will check what the issue is. Thanks for CCing me and thanks for reviewing the code.<div dir="auto"><br></div><div dir="auto">Best regards,</div><div dir="auto">Moritz </div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Rob Landley <<a href="mailto:rob@landley.net">rob@landley.net</a>> schrieb am Do., 19. Jan. 2023, 22:55:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 1/18/23 09:37, enh wrote:<br>
> On Wed, Jan 18, 2023 at 3:14 AM Rob Landley <<a href="mailto:rob@landley.net" target="_blank" rel="noreferrer">rob@landley.net</a><br>
> <mailto:<a href="mailto:rob@landley.net" target="_blank" rel="noreferrer">rob@landley.net</a>>> wrote:<br>
> <br>
>     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>
>     ><br>
>     <a href="https://android-review.googlesource.com/c/platform/external/toybox/+/2385040?tab=comments" rel="noreferrer 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>
> <br>
> yeah, i just thought that since moritz is learning C, he'd be interested in<br>
> trying to fix those things himself.<br>
> <br>
> (to go with the already-given advice of "use asan", it's also probably worth<br>
> suggesting he try coverage builds too, to know which parts of the code<br>
> have/haven't been executed during testing.)<br>
<br>
Easiest thing to do is cc: him to be sure. :)<br>
<br>
Rob<br>
</blockquote></div>