dice
dice copied to clipboard
Redundant conditional in evalGET
Describe the bug Not a bug but more of something that caught my eye when reading the codebase. It's my first time looking at or attempting anything open source. Lmk if this is the wrong category to file this. I wanted to open this issue so I could make a corresponding PR and fix this.
Package: core File: eval.go Method: evalGET
Get(key) will delete any expired key already, so the conditional listed below is redundant in code.
if hasExpired(obj) {
return RESP_NIL
}
Expected behavior Remove redundant condition since Get(key) takes care of the case.