dave/BigChef
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 35 Wiki Activity

Simplify EvalRegex function by removing unnecessary panic handling and nil checks

Browse Source
This commit is contained in:
PhatPhuckDave
2025-08-22 16:12:38 +02:00
parent 969ccae25c
commit ec24e0713d
1 changed files with 11 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
processor/processor.go
View File

@@ -487,17 +487,8 @@ func EvalRegex(L *lua.LState) int {
evalRegexLogger := processorLogger.WithPrefix("evalRegex") evalRegexLogger := processorLogger.WithPrefix("evalRegex")
evalRegexLogger.Debug("Lua evalRegex function called") evalRegexLogger.Debug("Lua evalRegex function called")
defer func() { input := L.ToString(1)
if r := recover(); r != nil { pattern := L.ToString(2)
evalRegexLogger.Error("Panic in EvalRegex: %v", r)
// Push empty table on panic
emptyTable := L.NewTable()
L.Push(emptyTable)
}
}()
pattern := L.ToString(1)
input := L.ToString(2)
evalRegexLogger.Debug("Pattern: %q, Input: %q", pattern, input) evalRegexLogger.Debug("Pattern: %q, Input: %q", pattern, input)
@@ -506,13 +497,17 @@ func EvalRegex(L *lua.LState) int {
evalRegexLogger.Debug("Go regex matches: %v (count: %d)", matches, len(matches)) evalRegexLogger.Debug("Go regex matches: %v (count: %d)", matches, len(matches))
matchesTable := L.NewTable() if len(matches) > 0 {
for i, match := range matches { matchesTable := L.NewTable()
matchesTable.RawSetInt(i, lua.LString(match)) for i, match := range matches {
evalRegexLogger.Debug("Set table[%d] = %q", i, match) matchesTable.RawSetInt(i, lua.LString(match))
evalRegexLogger.Debug("Set table[%d] = %q", i, match)
}
L.Push(matchesTable)
} else {
L.Push(lua.LNil)
} }
L.Push(matchesTable)
evalRegexLogger.Debug("Pushed matches table to Lua stack") evalRegexLogger.Debug("Pushed matches table to Lua stack")
return 1 return 1