Re: [PATCH] CodingGuidelines: mention "static" and "extern"

From: Eric Sunshine <sunshine@sunshineco.com>
Date: Feb 8, 2018, 23:14 UTC
Message-ID: <CAPig+cT_ddVSEvdnMTNhtxWJMm4=DLyq8-hSDXa5X0i6rXa3xA@mail.gmail.com>
In-Reply-To: <20180208213806.GA6381@sigill.intra.peff.net>

On Thu, Feb 8, 2018 at 4:38 PM, Jeff King <peff@peff.net> wrote:

Show 11 quoted lines

> Subject: [PATCH] CodingGuidelines: mention "static" and "extern"
> [...]
>
> Signed-off-by: Jeff King <peff@peff.net>
> ---
> diff --git a/Documentation/CodingGuidelines b/Documentation/CodingGuidelines
> @@ -386,6 +386,11 @@ For C programs:
> + - Variables and functions local to a given source file should be marked
> +   with "static". Variables that are visible to other source files
> +   must be declared with "extern" in header files. However, function
> +   declarations should not use "extern", as that is already the default.

Perhaps:

    ... as that is already the default, unless declarations in the
    header are already "extern", in which case consistency
    may favor mirroring existing usage.

or something.