Sudoku review #1

Open
BordedDev wants to merge 19 commits from BordedDev/sudoku:main into main
pull from: BordedDev/sudoku:main
merge into: retoor:main
Conversation 3 Commits 19 Files Changed 8 +1 -3
1 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit f1c1a02877 - Show all commits

4
reviews/BordedDev/sudoku.modified.js
View File

@ -29,9 +29,7 @@ class EventHandler {
if (this.suppresEvents) return [] if (this.suppresEvents) return []
this.eventCount++ this.eventCount++
const returnValue = this.events[event].listeners.map((listener) => { const returnValue = this.events[event].listeners.map((listener) => {
const returnValue = listener(data) return listener(data) ?? null
if (returnValue == undefined) return null
return returnValue
}) })
this.events[event].callCount++ this.events[event].callCount++
if (this.debugEvents) { if (this.debugEvents) {