On 05/28/13 20:48, Jordan Justen wrote:
> On Thu, May 16, 2013 at 6:01 AM, Laszlo Ersek <ler...@redhat.com> wrote:

>> +  }
>> +  else {
> 
> Nit-pick: we generally use "} else {"

Apologies :(

> I don't think your version violates the coding style.

Apparently not, 6.2.1 "Vertical Spacing" says

  A close brace may share a line with the else {, else if () {, and
  do-while constructs.

    } else {
    } else if {

Thus, "may".

If you don't mind I'll skimp on reformatting these, there must be a
hundred occurrences.

Thanks,
Laszlo

------------------------------------------------------------------------------
Introducing AppDynamics Lite, a free troubleshooting tool for Java/.NET
Get 100% visibility into your production application - at no cost.
Code-level diagnostics for performance bottlenecks with <2% overhead
Download for free and get started troubleshooting in minutes.
http://p.sf.net/sfu/appdyn_d2d_ap1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to