Re: [PATCH v2 1/3] *.[ch]: remove extern from function declarations using spatch

From: Thomas Gummerer <t.gummerer@gmail.com>
Date: Apr 15, 2019, 19:19 UTC
Message-ID: <20190415191955.GE1704@hank.intra.tgummerer.com>
In-Reply-To: <7895255fdbe027b671a77dac6d0ee2e4700974be.1555352526.git.liu.denton@gmail.com>

On 04/15, Denton Liu wrote: [...]

Show 7 quoted lines

> -extern int write_archive_entries(struct archiver_args *args, write_archive_entry_fn_t write_entry);
> -extern void *object_file_to_archive(const struct archiver_args *args,
> +int write_archive_entries(struct archiver_args *args, write_archive_entry_fn_t write_entry);
> +void *object_file_to_archive(const struct archiver_args *args,
>  				    const char *path, const struct object_id *oid,
>  				    unsigned int mode, enum object_type *type,
>  				    unsigned long *sizep);

Just skimming this patch, I notice that this now leaves a bunch of oddly indented parameter lists behind. We might want to re-indent those as well, as part of this patch or a follow up patch (so this patch can remain purely mechanical).

Personally I dislike this inconsistent indentation more than the extern keyword, but not sure how others feel about it.