Re: [PATCH] sequencer: mark a file-local symbol as static
- From
Eric Sunshine <sunshine@sunshineco.com>
- Date
- Mar 12, 2019, 02:58 UTC
- Message-ID
- <CAPig+cTqdeVXyYZarGOb+AiLOERO0HpK+ykYgrAddyFzPG0+Mg@mail.gmail.com>
- In-Reply-To
- <d5d47b6b-25d9-a649-e13d-76e98e07ed51@ramsayjones.plus.com>
On Mon, Mar 11, 2019 at 10:53 PM Ramsay Jones <ramsay@ramsayjones.plus.com> wrote:
Show 7 quoted lines
> If you need to re-roll your 'dl/merge-cleanup-scissors-fix' branch,
> could you please squash this into the relevant patch (commit 9bcdf520cb
> ("sequencer.c: define get_config_from_cleanup", 2019-03-10)).
>
> /* note that we assume that cleanup_config_mapping[0] contains the default settings */
> -struct cleanup_config_mapping cleanup_config_mappings[] = {
> +static struct cleanup_config_mapping cleanup_config_mappings[] = {This was also suggested during review[1].
[1]: http://public-inbox.org/git/CAPig+cQGvmi-237s1Leb1NpOMA0683PkCG4HBCKn_+x5YDHnCQ@mail.gmail.com/