From: Tanay Abhra <tanayabh@gmail.com>
To: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
Cc: Matthieu Moy <Matthieu.Moy@grenoble-inp.fr>,
Junio C Hamano <gitster@pobox.com>,
GIT Mailing-list <git@vger.kernel.org>
Subject: Re: [PATCH] cache.h: add missing NORETURN on git_die_config*()
Date: Tue, 05 Aug 2014 19:39:47 +0530 [thread overview]
Message-ID: <53E0E5AB.9050404@gmail.com> (raw)
In-Reply-To: <53E0E2E0.2070709@ramsay1.demon.co.uk>
On 8/5/2014 7:27 PM, Ramsay Jones wrote:
>
> Commit 3a2a9527 ("config: add `git_die_config()` to the config-set
> API", 01-08-2014) added git_die_config() and git_die_config_linenr()
> functions, but forgot to include the NORETURN attribute in their
> declarations. Sparse complains like so:
>
> SP config.c
> config.c:1567:6: error: symbol 'git_die_config_linenr' redeclared \
> with different type (originally declared at cache.h:1419) \
> - different modifiers
> config.c:1579:6: error: symbol 'git_die_config' redeclared with \
> different type (originally declared at cache.h:1418) \
> - different modifiers
>
> Suppress the sparse errors by adding NORETURN to the function
> declarations in the cache.h header file.
>
> Signed-off-by: Ramsay Jones <ramsay@ramsay1.demon.co.uk>
> ---
>
> Hi Tanay,
>
> If you need to re-roll the patches in the 'ta/config-set-1' branch,
> could you please squash this into the relevant patch. Thanks!
>
Sorry, my bad. I will squash it in the relevant patch for my next re roll.
Thanks,
Tanay Abhra.
prev parent reply other threads:[~2014-08-05 14:09 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-05 13:57 [PATCH] cache.h: add missing NORETURN on git_die_config*() Ramsay Jones
2014-08-05 14:09 ` Tanay Abhra [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=53E0E5AB.9050404@gmail.com \
--to=tanayabh@gmail.com \
--cc=Matthieu.Moy@grenoble-inp.fr \
--cc=git@vger.kernel.org \
--cc=gitster@pobox.com \
--cc=ramsay@ramsay1.demon.co.uk \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.