Re: staging: ks7010: Rename jump labels

From: Jean Delvare
Date: Thu Jul 21 2016 - 15:13:52 EST

Hi Markus,

On Thu, 21 Jul 2016 17:37:52 +0200, SF Markus Elfring wrote:
> > 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.

"#goto labels aren't indented, allow a single space however"

Can't be clearer :-)

> How do you generally think about jump label renaming?

Renaming from "out0:", "out1:" etc to something meaningful, yes. Did
you have anything else in mind?

Jean Delvare
SUSE L3 Support