Re: [PATCH v3 4/4] archive-tar: drop return value

From: Remi Galan Alfonso <remi.galan-alfonso@ensimag.grenoble-inp.fr>
Date: Jun 24, 2016, 11:49 UTC
Message-ID: <1865881092.716701.1466768964884.JavaMail.zimbra@ensimag.grenoble-inp.fr>
In-Reply-To: <20160623232158.GD3668@sigill.intra.peff.net>

Hi Peff,

Jeff King <peff@peff.net> writes:

Show 8 quoted lines

> @@ -413,7 +411,7 @@ static int write_tar_archive(const struct archiver *ar,
>  {
>          int err = 0;
>  
> -        err = write_global_extended_header(args);
> +        write_global_extended_header(args);
>          if (!err)
>                  err = write_archive_entries(args, write_tar_entry);

If we drop the error code from 'write_global_extended_header' then the above 'if (!err)' becomes useless (always evaluates to 'true' since 'err' is set to '0').

>          if (!err)

Thanks, Rémi

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.