Zsh Mailing List Archive
Messages sorted by:
Reverse Date,
Date,
Thread,
Author
Re: [PATCH] find RLIM_NLIMITS correctly on Cygwin
Jun T wrote on Mon, 23 Mar 2020 14:41 +0900:
> Changes to the data in known_resouces is done via the pointer current_letter:
> char *current_letter = &known_resources[i].opt;
> Constness of resinfo does not interfere with this.
Right.
> # and the data for unknown resource (if any) is setup using the pointer
> # resinfo_T *info = (resinfo_T *)zshcalloc(sizeof(resinfo_T));
> # and the assignment "resinfo[i] = info" does not conflict with the
> # constness of resinfo.
I agree about the assignment, but that allocation won't run for the case
at hand, will it? In the second for loop, «resinfo[i]» (in the 'if'
condition) will be non-NULL; it will point to an element of
«known_resources» whose «opt» member will have been changed to 'N' by
the first for loop.
> Using resinfo instead of resinfo_mutable gives no warning for me
> (gcc on Linux and clang on macOS).
>
> Do we really need resinfo_mutable?
I guess not. That must be a relic from an earlier draft of that patch.
I will fix it if we decide to add the runtime verification. Thanks for
the review. :-)
Cheers,
Daniel
Messages sorted by:
Reverse Date,
Date,
Thread,
Author