Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1172458 > unrolled thread

Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement

Started byJuston Li <juston.h.li@gmail.com>
First post2015-06-25 23:30 +0200
Last post2015-06-26 06:10 +0200
Articles 2 — 2 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement Juston Li <juston.h.li@gmail.com> - 2015-06-25 23:30 +0200
    Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement Sudip Mukherjee <sudipm.mukherjee@gmail.com> - 2015-06-26 06:10 +0200

#1172458 — Re: [PATCH v3 16/19] staging: sm750fb: fix brace placement

FromJuston Li <juston.h.li@gmail.com>
Date2015-06-25 23:30 +0200
SubjectRe: [PATCH v3 16/19] staging: sm750fb: fix brace placement
Message-ID<pFmSC-3Rc-25@gated-at.bofh.it>
On Thu, Jun 25, 2015 at 1:31 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:
> Once you add the else statement, then that kind of doesn't make sense.
> Sudip is right.  It should be:
>
>         } else {
>                 /* commentary about else side */

Yeah, that does make more sense. I'll change it, thanks.

Patchset was responded by Greg's patch bot,  I'll resend when I figure out why.

Regards
Juston
--
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/

[toc] | [next] | [standalone]


#1172582

FromSudip Mukherjee <sudipm.mukherjee@gmail.com>
Date2015-06-26 06:10 +0200
Message-ID<pFt7I-4sx-15@gated-at.bofh.it>
In reply to#1172458
On Thu, Jun 25, 2015 at 02:27:02PM -0700, Juston Li wrote:
> On Thu, Jun 25, 2015 at 1:31 PM, Dan Carpenter <dan.carpenter@oracle.com> wrote:

> Patchset was responded by Greg's patch bot,  I'll resend when I figure out why.
Greg will not apply any patch since the merge window is open. That
patchbot will respond when anyone send patch to Greg during this time.
Just send the updated patch as a reply to the original patch with proper
version number and Greg will pick it up later.

regards
sudip
--
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/

[toc] | [prev] | [standalone]


Back to top | Article view | linux.kernel


csiph-web