Ramkumar Ramachandra <artagnon@xxxxxxxxx> writes:
> The struct grab_1st_switch_cbdata has the field "found", which is set in
> grab_1st_switch() when a match is found. This information is redundant
> and unused by any caller: the return value of the function serves to
> communicate this information anyway. Remove the field.
>
> Signed-off-by: Ramkumar Ramachandra <artagnon@xxxxxxxxx>
> ---
Good.
> wt-status.c | 2 --
> 1 file changed, 2 deletions(-)
>
> diff --git a/wt-status.c b/wt-status.c
> index bf84a86..2511270 100644
> --- a/wt-status.c
> +++ b/wt-status.c
> @@ -1035,7 +1035,6 @@ got_nothing:
> }
>
> struct grab_1st_switch_cbdata {
> - int found;
> struct strbuf buf;
> unsigned char nsha1[20];
> };
> @@ -1059,7 +1058,6 @@ static int grab_1st_switch(unsigned char *osha1, unsigned char *nsha1,
> for (end = target; *end && *end != '\n'; end++)
> ;
> strbuf_add(&cb->buf, target, end - target);
> - cb->found = 1;
> return 1;
> }
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html