Thread (63 messages) 63 messages, 4 authors, 2021-12-13

Re: [PATCH v4 07/25] reboot: Remove extern annotation from function prototypes

From: "Rafael J. Wysocki" <rafael@kernel.org>
Date: 2021-12-10 18:35:33
Also in: linux-acpi, linux-m68k, linux-mips, linux-pm, linux-riscv, linux-sh, linux-tegra, lkml, xen-devel

On Fri, Dec 10, 2021 at 7:16 PM Dmitry Osipenko [off-list ref] wrote:
10.12.2021 21:09, Rafael J. Wysocki пишет:
quoted
On Fri, Nov 26, 2021 at 7:02 PM Dmitry Osipenko [off-list ref] wrote:
quoted
There is no need to annotate function prototypes with 'extern', it makes
code less readable. Remove unnecessary annotations from <reboot.h>.

Signed-off-by: Dmitry Osipenko <digetx@gmail.com>
I'm not sure that this is really useful.

Personally, I tend to respect the existing conventions like this.

Surely, this change is not required for the rest of the series to work.
Problem that such things start to spread all over the kernel with a
copy-paste approach if there is nobody to clean up the code.

This is not a common convention and sometimes it's getting corrected [1].

[1] https://git.kernel.org/linus/6d7434931
In separate patches outside of series adding new features, if one is
so inclined.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help