-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On 03/17/2014 04:43 PM, Guillaume J. Charmes wrote:
> In pkg/selinux/selinux.go:
> 
>> +				t := c1 +				c1 = c2 +				c2 = t +			} +		} +		mcs =
>> fmt.Sprintf("s0:c%d,c%d", c1, c2) +		if mcsExists(mcs) { +			continue +
>> } +		mcsAdd(mcs) +		break +	} +	return mcs +} +func FreeContext(con
>> string) { +	var scon SELinuxContext
> 
> Why do you declare the variable here? You use if in the |if| scope but
> nowhere else.
> 
> — Reply to this email directly or view it on GitHub 
> <https://github.com/dotcloud/docker/pull/4211/files#r10677887>.
> 
Moved to within scope.
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1
Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/

iEYEARECAAYFAlMoUc4ACgkQrlYvE4MpobPykgCfVWeuKdpTabDpGeN9kFNHKioS
rgUAoOPawDcADVlJDPeD8FLnk29V9TUX
=sIQQ
-----END PGP SIGNATURE-----