refactor(debugger): improve parameter names in ShouldStop method

dev-kas · dev-kas · commit 8c0d7f3e6f53 · 2025-05-20T20:53:43.000+05:30
- Renamed 'line' parameter to 'filename' for clarity
- Renamed 'col' parameter to 'line' to better reflect its purpose
- Improves code readability and maintainability
diff --git a/debugger/debugger.go b/debugger/debugger.go
@@ -47,8 +47,8 @@ var Debuggables = map[ast.NodeType]struct{}{
 
 // Internal API
 
-func (d *Debugger) ShouldStop(line string, col int) bool {
-	return d.BreakpointManager.Has(line, col)
+func (d *Debugger) ShouldStop(filename string, line int) bool {
+	return d.BreakpointManager.Has(filename, line)
 }
 
 func (d *Debugger) IsDebuggable(astNode ast.Stmt) bool {

Original file line number	Diff line number	Diff line change
`@@ -47,8 +47,8 @@ var Debuggables = map[ast.NodeType]struct{}{`
`47`	`47`
`48`	`48`	`// Internal API`
`49`	`49`
`50`		`-func (d *Debugger) ShouldStop(line string, col int) bool {`
`51`		`- return d.BreakpointManager.Has(line, col)`
	`50`	`+func (d *Debugger) ShouldStop(filename string, line int) bool {`
	`51`	`+ return d.BreakpointManager.Has(filename, line)`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`func (d *Debugger) IsDebuggable(astNode ast.Stmt) bool {`