Hi Bernhard, Thanks for the review.
On Sun, Apr 10, 2022 at 10:44 AM Bernhard Reutner-Fischer <rep.dot....@gmail.com> wrote: > Please also provide bloatcheck stats. > $ make baseline > # apply patch > $ make bloatcheck Oh, very neat feature. Will include that in v3. > We have open_read_close() which should considerably reduce size. > And there's a xopen_xwrite_close > strtoul: We have several checking conversion routines in include/xatonum.h > RUNSTATEDIR is probably PID_FILE_PATH > poll: maybe safe_poll() > non_creditable_seed = xasprintf("%s/%s"): concat_path_file Very nice set of helpers. Used all of those. > mkdir: bb_make_directory This one wasn't so useful: I don't actually want _recursive_ directory creation, I don't think. > Why do you open(), flock() instead of using O_EXCL? So that if the system crashes, the next boot can still move forward, or if the process crashes, the next run can still move forward. flock() is a runtime thing, where as O_EXCL is a "must be in filesystem" thing, which is much weaker. v3 coming right up! Thanks again for the feedback. Jason _______________________________________________ busybox mailing list busybox@busybox.net http://lists.busybox.net/mailman/listinfo/busybox