Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 16 Jun 2020 11:01:02 -0600
From:      Ian Lepore <ian@freebsd.org>
To:        Toomas Soome <tsoome@FreeBSD.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r362217 - head/stand/common
Message-ID:  <55903c38d363aef2a6f6d0075dd4526b86d51258.camel@freebsd.org>
In-Reply-To: <202006160705.05G753T4057972@repo.freebsd.org>
References:  <202006160705.05G753T4057972@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, 2020-06-16 at 07:05 +0000, Toomas Soome wrote:
> Author: tsoome
> Date: Tue Jun 16 07:05:03 2020
> New Revision: 362217
> URL: https://svnweb.freebsd.org/changeset/base/362217
> 
> Log:
>   loader: variable i is unused without MBR/GPT support built in
>   
>   Because i is only used as index in for loop, declare it in for
> statement.
> 

As much as I prefer doing it this way, style(9) doesn't allow for
variable declarations inside a for() statement (or even inside a local
block, which is just too 1980s for me, but it is still our standard).

-- Ian




Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?55903c38d363aef2a6f6d0075dd4526b86d51258.camel>