Re: staging: ks7010: Rename jump labels

From: SF Markus Elfring
Date: Thu Jul 21 2016 - 11:38:16 EST

Next message: Tejun Heo: "Re: [PATCH v1 3/3] cgroup: relax common ancestor restriction for direct descendants"
Previous message: James Bottomley: "Re: [PATCH v1 3/3] cgroup: relax common ancestor restriction for direct descendants"
In reply to: SF Markus Elfring: "Re: staging: ks7010: Rename jump labels"
Next in thread: Jean Delvare: "Re: staging: ks7010: Rename jump labels"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

> That being said... checkpatch does not complain about leading space
> before labels. Not even with --strict. So why are you mentioning it here?

I remembered a warning like "INDENTED_LABEL" instead.
https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/tree/scripts/checkpatch.pl?id=92d21ac74a9e3c09b0b01c764e530657e4c85c49#n4326

How do you generally think about jump label renaming?

Regards,
Markus

Next message: Tejun Heo: "Re: [PATCH v1 3/3] cgroup: relax common ancestor restriction for direct descendants"
Previous message: James Bottomley: "Re: [PATCH v1 3/3] cgroup: relax common ancestor restriction for direct descendants"
In reply to: SF Markus Elfring: "Re: staging: ks7010: Rename jump labels"
Next in thread: Jean Delvare: "Re: staging: ks7010: Rename jump labels"
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]