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

From
Dmitry Osipenko <digetx@gmail.com>
Date
2021-12-10 18:56:40
Also in
linux-acpi, linux-m68k, linux-mips, linux-pm, linux-riscv, linux-sh, linux-tegra, lkml, xen-devel
Reply to
Rafael J. Wysocki, 2021-12-10
10.12.2021 21:35, Rafael J. Wysocki пишет:
On Fri, Dec 10, 2021 at 7:16 PM Dmitry Osipenko [off-list ref] wrote:
quoted
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.
Alright, I'll drop this patch then because it can't be done in parallel
without creating the merge conflict. I'll try not to forget to come back
to this later on.
Thread (63 messages) 63 messages, 4 authors, last reply 2021-12-13
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help