Re: [PATCH 2/3] usb: type-c: USB Type-C Connector System Software Interface

[BjÃrn Mork]

Andy Shevchenko <andy.shevchenko@xxxxxxxxx> writes:

> To me out: sounds out, like printing error and return code, or
> something like that. Here the case is different.
>
> And since we have two full switches it might be hard to have on one
> screen out code and exact goto. See my point now?

No, sorry.  Not really.

The pattern

   out:
      return whatever;

is so common in the kernel that I cannot see any point starting a
discussion about it:

 bjorn@nemi:/usr/local/src/git/linux$ git grep -A1 ^out:|grep return|wc -l
 3622


And I also fail to see any relation between the explanation you come up
with here and the text in CodingStyle, which was what you initially
referred to.  There is no specific interpreation of the label name "out"
there AFAICS.

I apologize if you think I am being an ass now. I am.  I'm just too fed
up with coding style arguments with no other reason than a vague pointer
to some document.  If you're going to point out coding style problems,
then you could at least make an effort explaining why a style change
improves the code.  My bet is that most of the time you'll find that the
comment is unnecessary...



BjÃrn