[Toybox] [CLEANUP] syslogd - Pass 5
Rob Landley
rob at landley.net
Sun Aug 25 06:09:23 PDT 2013
On 08/24/2013 05:20:06 AM, Felix Janda wrote:
> Should have tested the previous two patches more than that they
> compile.
>
> The changes
>
> - facstr = dec(LOG_FAC(pri), facilitynames);
> + facstr = dec(pri & LOG_FACMASK, facilitynames);
>
> - for (tk = "!=*"; tk; tk++, bits <<= 1) {
> + for (tk = "!=*"; *tk; tk++, bits <<= 1) {
>
> fix Pass 3 while
>
> - while (file && !strcmp(file->filename, tokens[1])) file =
> file->next;
> + while (file && strcmp(file->filename, tokens[1])) file =
> file->next;
>
> fixes Pass 4.
>
> If none of these have been applied yet, I could prepare new patches
> which don't leave the toy broken in between them.
I like the incremental nature of it, but I'm still not sure why patch 4
_doesn't_ seem to break defconfig. (C is weird about structs it hasn't
seen the declaration for yet...)
I need to frown at it a bit more...
Rob
1377436163.0
More information about the Toybox
mailing list