3 ms·
That code line looked uncomfortably wrong: if (dbs_per_call > server.dbnum || timelimit_exit) dbs_per_call = server.dbnum; So, there's a single da
by willvarfar 8y ago
That code line looked uncomfortably wrong:
if (dbs_per_call > server.dbnum || timelimit_exit)
dbs_per_call = server.dbnum;
So, there's a single database... what if its `server.dbnum` is zero? This is then setting `dbs_per_call` to zero? Is `dbs_per_call` a global variable? Etc.
Of course we don't have the wider code context here, but that if-statement looks like its the kind of check to exit a loop and it ought be followed by a `break` or `return` statement etc.