checkpatch warning of struct indentation

checkpatch warning of struct indentation

[Steven Rostedt]

Hi,

I'm now seeing the following warnings from checkpatch:

#325: FILE: kernel/trace/trace_stat.c:21:
+	void 			*stat;

ERROR: "foo 	*bar" should be "foo *bar"
#334: FILE: kernel/trace/trace_stat.c:27:
+	struct tracer_stat 	*ts;

ERROR: "foo		*bar" should be "foo	*bar"
#337: FILE: kernel/trace/trace_stat.c:30:
+	struct dentry		*file;

This is for:

struct tracer_stat_session {
        struct list_head        session_list;
        struct tracer_stat      *ts;
        struct list_head        stat_list;
        struct mutex            stat_mutex;
        struct dentry           *file;
};

Which looks a hell of a lot better than:

struct tracer_stat_session {
        struct list_head session_list;
        struct tracer_stat *ts;
        struct list_head stat_list;
        struct mutex stat_mutex;
        struct dentry *file;
};

We probably do not want to warn on such things.

-- Steve

Re: checkpatch warning of struct indentation

[Frédéric Weisbecker]

2009/1/12 Steven Rostedt <rostedt@goodmis.org>:
>
> Hi,
>
> I'm now seeing the following warnings from checkpatch:
>
> #325: FILE: kernel/trace/trace_stat.c:21:
> +       void                    *stat;
>
> ERROR: "foo     *bar" should be "foo *bar"
> #334: FILE: kernel/trace/trace_stat.c:27:
> +       struct tracer_stat      *ts;
>
> ERROR: "foo             *bar" should be "foo    *bar"
> #337: FILE: kernel/trace/trace_stat.c:30:
> +       struct dentry           *file;


Oh my patch :-)
Yeah I had the same errors when I checked it. I ignored them because
I because I remembered checkpatch.pl didn't warn about such things before.
And it seems to warn about such statements since very recently...


> This is for:
>
> struct tracer_stat_session {
>        struct list_head        session_list;
>        struct tracer_stat      *ts;
>        struct list_head        stat_list;
>        struct mutex            stat_mutex;
>        struct dentry           *file;
> };
>
> Which looks a hell of a lot better than:
>
> struct tracer_stat_session {
>        struct list_head session_list;
>        struct tracer_stat *ts;
>        struct list_head stat_list;
>        struct mutex stat_mutex;
>        struct dentry *file;
> };
>
> We probably do not want to warn on such things.
>
> -- Steve
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> Please read the FAQ at  http://www.tux.org/lkml/
>

Re: checkpatch warning of struct indentation

[Steven Rostedt]

On Mon, 12 Jan 2009, Fr?d?ric Weisbecker wrote:

> 2009/1/12 Steven Rostedt <rostedt@goodmis.org>:
> >
> > Hi,
> >
> > I'm now seeing the following warnings from checkpatch:
> >
> > #325: FILE: kernel/trace/trace_stat.c:21:
> > +       void                    *stat;
> >
> > ERROR: "foo     *bar" should be "foo *bar"
> > #334: FILE: kernel/trace/trace_stat.c:27:
> > +       struct tracer_stat      *ts;
> >
> > ERROR: "foo             *bar" should be "foo    *bar"
> > #337: FILE: kernel/trace/trace_stat.c:30:
> > +       struct dentry           *file;
> 
> 
> Oh my patch :-)
> Yeah I had the same errors when I checked it. I ignored them because
> I because I remembered checkpatch.pl didn't warn about such things before.
> And it seems to warn about such statements since very recently...

Heh, I should have CC'd you on this complaint ;-)

Yeah, this is a new warning and since we do a lot of these types of
indentation, and they are marked as "ERROR", I would like to see these go 
away. Perhaps they need to test for parenthesis, so:

 int func(foo		*bar)

gets caught.


[off topic, funny English grammar]

I noticed that you said:

   And it seems to warn about such statements since very recently

This sounds strange. I know in German (and I know you are not German, but 
it's what I have most experience with) the word "seit" gets translated 
into "since" for such statements as above. A lot of Germans that I know 
make this funny sounding phrase. I would have written it like:

   And, recently, it seems to warn about such statements.

Don't take this as a criticism. I'm the last person to criticize anyone's 
grammar. For being an English speaking native, my grammar sucks ;-)
And your statement may indeed be correct grammar. It just sounds a little
funny to me.

In a lot of cases, (for Germans) "seit" can correctly be translated into
"since" but there are times that it just sounds funny.

A common phrase from Germans are:

  I've been doing this since three years.

Just an observation, carry on ;-)

-- Steve


> 
> 
> > This is for:
> >
> > struct tracer_stat_session {
> >        struct list_head        session_list;
> >        struct tracer_stat      *ts;
> >        struct list_head        stat_list;
> >        struct mutex            stat_mutex;
> >        struct dentry           *file;
> > };
> >
> > Which looks a hell of a lot better than:
> >
> > struct tracer_stat_session {
> >        struct list_head session_list;
> >        struct tracer_stat *ts;
> >        struct list_head stat_list;
> >        struct mutex stat_mutex;
> >        struct dentry *file;
> > };
> >
> > We probably do not want to warn on such things.
> >
> > -- Steve
> >
> > --
> > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > the body of a message to majordomo@vger.kernel.org
> > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > Please read the FAQ at  http://www.tux.org/lkml/
> >
> 
> 

Re: checkpatch warning of struct indentation

[Frederic Weisbecker]

On Mon, Jan 12, 2009 at 01:35:43PM -0500, Steven Rostedt wrote:
> 
> On Mon, 12 Jan 2009, Fr?d?ric Weisbecker wrote:
> 
> > 2009/1/12 Steven Rostedt <rostedt@goodmis.org>:
> > >
> > > Hi,
> > >
> > > I'm now seeing the following warnings from checkpatch:
> > >
> > > #325: FILE: kernel/trace/trace_stat.c:21:
> > > +       void                    *stat;
> > >
> > > ERROR: "foo     *bar" should be "foo *bar"
> > > #334: FILE: kernel/trace/trace_stat.c:27:
> > > +       struct tracer_stat      *ts;
> > >
> > > ERROR: "foo             *bar" should be "foo    *bar"
> > > #337: FILE: kernel/trace/trace_stat.c:30:
> > > +       struct dentry           *file;
> > 
> > 
> > Oh my patch :-)
> > Yeah I had the same errors when I checked it. I ignored them because
> > I because I remembered checkpatch.pl didn't warn about such things before.
> > And it seems to warn about such statements since very recently...
> 
> Heh, I should have CC'd you on this complaint ;-)
> 
> Yeah, this is a new warning and since we do a lot of these types of
> indentation, and they are marked as "ERROR", I would like to see these go 
> away. Perhaps they need to test for parenthesis, so:
> 
>  int func(foo		*bar)
> 
> gets caught.
> 
> 
> [off topic, funny English grammar]
> 
> I noticed that you said:
> 
>    And it seems to warn about such statements since very recently
> 
> This sounds strange. I know in German (and I know you are not German, but 
> it's what I have most experience with) the word "seit" gets translated 
> into "since" for such statements as above. A lot of Germans that I know 
> make this funny sounding phrase. I would have written it like:
> 
>    And, recently, it seems to warn about such statements.
> 
> Don't take this as a criticism. I'm the last person to criticize anyone's 
> grammar. For being an English speaking native, my grammar sucks ;-)
> And your statement may indeed be correct grammar. It just sounds a little
> funny to me.
> 
> In a lot of cases, (for Germans) "seit" can correctly be translated into
> "since" but there are times that it just sounds funny.
> 
> A common phrase from Germans are:
> 
>   I've been doing this since three years.
> 
> Just an observation, carry on ;-)
> 
> -- Steve
> 


Hehe. Yes, I always hesitate when I have to talk about elapsed time,
especially when it is an uncertain time...
And French/German are often confused with "since" and "for" while in
french we have only "depuis" and in german: "seit"...

Thanks, I'm always glad to be corrected in my english, hoping
it will be fixed by the time... :-)
 
> > 
> > 
> > > This is for:
> > >
> > > struct tracer_stat_session {
> > >        struct list_head        session_list;
> > >        struct tracer_stat      *ts;
> > >        struct list_head        stat_list;
> > >        struct mutex            stat_mutex;
> > >        struct dentry           *file;
> > > };
> > >
> > > Which looks a hell of a lot better than:
> > >
> > > struct tracer_stat_session {
> > >        struct list_head session_list;
> > >        struct tracer_stat *ts;
> > >        struct list_head stat_list;
> > >        struct mutex stat_mutex;
> > >        struct dentry *file;
> > > };
> > >
> > > We probably do not want to warn on such things.
> > >
> > > -- Steve
> > >
> > > --
> > > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> > > the body of a message to majordomo@vger.kernel.org
> > > More majordomo info at  http://vger.kernel.org/majordomo-info.html
> > > Please read the FAQ at  http://www.tux.org/lkml/
> > >
> > 
> > 

Re: checkpatch warning of struct indentation

[Johannes Weiner]

On Mon, Jan 12, 2009 at 01:35:43PM -0500, Steven Rostedt wrote:

> In a lot of cases, (for Germans) "seit" can correctly be translated into
> "since" but there are times that it just sounds funny.

It can only be translated to `since' when you mean a single point in
the past and you have to use `for' when you refer to a whole period.

`since 29th august' vs. `for three years'

No?

> A common phrase from Germans are:
> 
>   I've been doing this since three years.

They slept in school :-)

	Hannes

Re: checkpatch warning of struct indentation

[Steven Rostedt]

On Tue, 13 Jan 2009, Johannes Weiner wrote:

> On Mon, Jan 12, 2009 at 01:35:43PM -0500, Steven Rostedt wrote:
> 
> > In a lot of cases, (for Germans) "seit" can correctly be translated into
> > "since" but there are times that it just sounds funny.
> 
> It can only be translated to `since' when you mean a single point in
> the past and you have to use `for' when you refer to a whole period.
> 
> `since 29th august' vs. `for three years'
> 
> No?

Yeah that could be it. I have no idea, since I'm only a native speaker
and I slept in most of my English classes ;-)

> 
> > A common phrase from Germans are:
> > 
> >   I've been doing this since three years.
> 
> They slept in school :-)

Perhaps they asked, "May I lend an English grammar book?" when they meant 
to say "May I borrow an English grammar book?" :-)

-- Steve

Re: checkpatch warning of struct indentation

[Johannes Weiner]

On Tue, Jan 13, 2009 at 09:54:13AM -0500, Steven Rostedt wrote:
> > > A common phrase from Germans are:
> > > 
> > >   I've been doing this since three years.
> > 
> > They slept in school :-)
> 
> Perhaps they asked, "May I lend an English grammar book?" when they meant 
> to say "May I borrow an English grammar book?" :-)

More likely they asked `may I become an English grammar book'.

German `etw. bekommen' translates to `to get sth'

> -- Steve

	Hannes

Re: checkpatch warning of struct indentation

[Ingo Molnar]

* Steven Rostedt <rostedt@goodmis.org> wrote:

> Hi,
> 
> I'm now seeing the following warnings from checkpatch:
> 
> #325: FILE: kernel/trace/trace_stat.c:21:
> +	void 			*stat;
> 
> ERROR: "foo 	*bar" should be "foo *bar"
> #334: FILE: kernel/trace/trace_stat.c:27:
> +	struct tracer_stat 	*ts;
> 
> ERROR: "foo		*bar" should be "foo	*bar"
> #337: FILE: kernel/trace/trace_stat.c:30:
> +	struct dentry		*file;
> 
> This is for:
> 
> struct tracer_stat_session {
>         struct list_head        session_list;
>         struct tracer_stat      *ts;
>         struct list_head        stat_list;
>         struct mutex            stat_mutex;
>         struct dentry           *file;
> };
> 
> Which looks a hell of a lot better than:
> 
> struct tracer_stat_session {
>         struct list_head session_list;
>         struct tracer_stat *ts;
>         struct list_head stat_list;
>         struct mutex stat_mutex;
>         struct dentry *file;
> };
> 
> We probably do not want to warn on such things.

yeah, those warnings look bogus.

	Ingo

Re: checkpatch warning of struct indentation

[Andy Whitcroft]

On Mon, Jan 12, 2009 at 12:57:23PM -0500, Steven Rostedt wrote:
> 
> Hi,
> 
> I'm now seeing the following warnings from checkpatch:
> 
> #325: FILE: kernel/trace/trace_stat.c:21:
> +	void 			*stat;
> 
> ERROR: "foo 	*bar" should be "foo *bar"
> #334: FILE: kernel/trace/trace_stat.c:27:
> +	struct tracer_stat 	*ts;
> 
> ERROR: "foo		*bar" should be "foo	*bar"
> #337: FILE: kernel/trace/trace_stat.c:30:
> +	struct dentry		*file;

Yes those are bogus.  They should be fixed in the latest push to akpm.
You can check that with this version:

  http://www.kernel.org/pub/linux/kernel/people/apw/checkpatch/checkpatch.pl-testing

-apw