Updated comments.

Author: ReFreezed

src/app.lua2p

@@ -623,8 +623,9 @@ _G.scriptEnvironmentGlobals = {
 	end,
 
 	validateUrls = function(urls)
-		-- @Robustness: Should we restrict context here?
+		-- @Robustness: Should we restrict context here? (I think not. 2021-05-01)
 		local ok = true
+
 		for _, url in ipairs(urls) do
 			if not urlExists(url) then
 				timestampPrintError("URL is missing: %s", url)
@@ -1044,7 +1045,7 @@ local function buildWebsite()
 	_G.autoLockPages       = getV("autoLockPages",     false,                 "boolean")
 	_G.noTrailingSlash     = getV("removeTrailingSlashFromPermalinks", false, "boolean") -- @Doc? @Hack? Also, this affects page.url too now. 2021-04-28
 
-	_G.redirectionLayout   = getV("redirectionLayout", redirectionLayout,     "string") -- @Doc
+	_G.redirectionLayout   = getV("redirectionLayout", redirectionLayout,     "string")
 
 	for k in pairs(config) do
 		timestampPrintWarning("Unknown config field '%s'.", k)
@@ -1527,7 +1528,7 @@ if autobuild then
 			end
 		end
 
-		if somethingChanged then  buildWebsite()  end -- @UX: Should we pcall this?
+		if somethingChanged then  buildWebsite()  end -- @UX: Should we pcall this? Probably!
 
 		if configChanged then
 			-- Recheck everything in case config.ignore* changed.

src/functions.lua2p

@@ -428,7 +428,7 @@ do
 				blockContents:find!("^ *for%s+"..PATTERN_IDENT.."%s*[,=]") or
 				blockContents:find!("^ *for%s+"..PATTERN_IDENT.."%s+in"..PATTERN_IDENT_END)
 			then
-				table.insert(out, blockContents) -- @Robustness: Surround value in parenthesis.
+				table.insert(out, blockContents) -- @Robustness: Surround value in parenthesis, when possible.
 				table.insert(out, " do ")
 
 				!!(TRIM_POS_AFTER_BLOCK)
@@ -684,8 +684,8 @@ do
 		if not ok then  error(errObj)  end
 
 		local templateResult = (table.concat(ctx.out)
-			:gsub("[ \t]+\n", "\n"  ) -- :Beautify
-			:gsub("\n\n\n+",  "\n\n") -- :Beautify  @UX Not sure if this line is actually good.
+			:gsub("[ \t]+\n", "\n"  ) -- :Beautify spaces at the end of lines.
+			:gsub("\n\n\n+",  "\n\n") -- :Beautify empty lines.  @Robustness: Not sure this is actually good. Might mess up <pre> tag contents.
 		)
 
 		return templateResult
@@ -774,15 +774,7 @@ end
 
 
 function _G.getLineNumber(s, pos)
-	local lineCount = 1
-	for posCurrent in s:gmatch"()\n" do -- @Speed
-		if posCurrent < pos then
-			lineCount = lineCount+1
-		else
-			break
-		end
-	end
-	return lineCount
+	return 1 + countSubStrings(s, 1, pos-1, "\n", true)
 end
 
 
@@ -1906,7 +1898,7 @@ do
 		local modTimeThumb = lfs.attributes(DIR_OUTPUT.."/"..pathThumbOutputRel, "modification")
 
 		if modTimeImage and modTimeImage == modTimeThumb and not ignoreModificationTimes then
-			-- @Note: This will bypass any file processor for JPG files. Not sure if OK. 2018-06-30
+			-- Note: This will bypass any file processor for JPG files. Not sure if OK. 2018-06-30 @Consistency
 			preserveExistingOutputFile("raw", pathThumbRel, "/"..pathThumbRel, pathImageRel)
 
 		else
@@ -2427,7 +2419,7 @@ function _G.cleanupPath(someKindOfPath)
 
 	repeat
 		local count
-		path, count = path:gsub("/[^/]+/%.%./", "/", 1) -- Not completely fool proof! @Robustness
+		path, count = path:gsub("/[^/]+/%.%./", "/", 1) -- Not completely fool proof! We may need to properly parse the path, I think. @Robustness
 	until count == 0
 
 	return path
@@ -2561,7 +2553,7 @@ do
 			socket.sleep(duration)
 		else
 			local endTime = getTimeBetter() + duration
-			repeat until getTimeBetter() >= endTime -- @Incomplete: Do something to waste time?
+			repeat until getTimeBetter() >= endTime -- Should we do some better operation to waste time?
 		end
 	end
 end