On 1/18/23 09:37, enh wrote: > On Wed, Jan 18, 2023 at 3:14 AM Rob Landley <[email protected] > <mailto:[email protected]>> wrote: > > On 1/17/23 20:00, enh via Toybox wrote: > > fyi, there were some fly-by code review comments against git.c when i > did the > > last toybox update: > > > > > > https://android-review.googlesource.com/c/platform/external/toybox/+/2385040?tab=comments > > > > > (i'm assuming moritz reads the list?) > > In theory, yes. > > I've done some very simple code style cleanup, but detoured into reading > the gif > file format docs before trying to work out object lifetime rules, and > it's now > like 6 things down on my stack because interrupts... :) > > (Pending. It's there for a reason. Seems like a good start though...) > > yeah, i just thought that since moritz is learning C, he'd be interested in > trying to fix those things himself. > > (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.)
Easiest thing to do is cc: him to be sure. :) Rob _______________________________________________ Toybox mailing list [email protected] http://lists.landley.net/listinfo.cgi/toybox-landley.net
