[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20170209151729.6c88b779@xeon-e3>
Date: Thu, 9 Feb 2017 15:17:29 -0800
From: Stephen Hemminger <stephen@...workplumber.org>
To: Thomas Gleixner <tglx@...utronix.de>
Cc: Stephen Hemminger <sthemmin@...rosoft.com>,
Haiyang Zhang <haiyangz@...rosoft.com>,
"x86@...nel.org" <x86@...nel.org>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Andy Lutomirski <luto@...capital.net>,
Ingo Molnar <mingo@...hat.com>,
"H. Peter Anvin" <hpa@...or.com>,
"devel@...uxdriverproject.org" <devel@...uxdriverproject.org>,
"virtualization@...ts.linux-foundation.org"
<virtualization@...ts.linux-foundation.org>
Subject: Re: [PATCH 1/2] hyperv: implement hv_get_tsc_page()
On Thu, 9 Feb 2017 21:14:25 +0100 (CET)
Thomas Gleixner <tglx@...utronix.de> wrote:
> On Thu, 9 Feb 2017, Stephen Hemminger wrote:
>
> > The actual code looks fine, but the style police will not like you.
> > { should be at start of line on functions.
> > And #else should be at start of line,
> >
> > But maybe this was just more of exchange mangling the mail.
>
> Looks like.
>
> > +struct ms_hyperv_tsc_page *hv_get_tsc_page(void) {
> > + return tsc_pg;
> > +}
> > +
>
> That's how it reads in a proper mail client connected to a proper mail
> server:
>
> > +struct ms_hyperv_tsc_page *hv_get_tsc_page(void)
> > +{
> > + return tsc_pg;
> > +}
>
> :)
Yup. it looks like the mail server is trying to be "helpful" by eliminating extra white space.
Powered by blists - more mailing lists