nil check for multiGet pointless?
mrusme opened this issue · comments
◤◢◤◢◤◢◤◢ commented
I'm looking at the following code and I'm wondering whether it makes sense:
Lines 308 to 315 in 13fa812
From what I see multiGet
cannot be nil, because &multiGet.DataRequest
would throw an invalid memory address / nil pointer error. Hence the check if multiGet == nil
seems kind of pointless here.
Either that check should happen earlier or not at all, imho.