svn commit: r329899 - head/stand/lua

Kyle Evans kevans at FreeBSD.org
Sat Feb 24 03:43:11 UTC 2018


Author: kevans
Date: Sat Feb 24 03:43:10 2018
New Revision: 329899
URL: https://svnweb.freebsd.org/changeset/base/329899

Log:
  lualoader: Correct test and name
  
  The functionality was correct, but our style guidelines tend to request that
  we shy away from using boolean operations in place of explicit comparisons
  to nil.

Modified:
  head/stand/lua/config.lua

Modified: head/stand/lua/config.lua
==============================================================================
--- head/stand/lua/config.lua	Sat Feb 24 03:38:51 2018	(r329898)
+++ head/stand/lua/config.lua	Sat Feb 24 03:43:10 2018	(r329899)
@@ -130,11 +130,11 @@ local function check_nextboot()
 		return
 	end
 
-	local function check_nextboot_disabled(text)
-		return not text:match("^nextboot_enable=\"NO\"")
+	local function check_nextboot_enabled(text)
+		return text:match("^nextboot_enable=\"NO\"") == nil
 	end
 
-	if not config.parse(nextboot_file, true, check_nextboot_disabled) then
+	if not config.parse(nextboot_file, true, check_nextboot_enabled) then
 		-- This only fails if it actually hit a parse error
 		print("Failed to parse nextboot configuration: '" ..
 		    nextboot_file .. "'")


More information about the svn-src-head mailing list