Add defensive comments.

This commit is contained in:
asofold 2012-12-28 00:17:27 +01:00
parent ef4b55ed7d
commit 2ea4c8ef6a
2 changed files with 2 additions and 2 deletions

View File

@ -206,7 +206,7 @@ public class ChatListener extends CheckListener implements INotifyReload {
// Tell TickTask to update cached permissions.
TickTask.requestPermissionUpdate(player.getName(), CheckType.CHAT);
// Force permission update.
TickTask.updatePermissions();
TickTask.updatePermissions(); // TODO: This updates ALL... something more efficient ?
// Reset captcha of player if needed.
synchronized(data){

View File

@ -35,7 +35,7 @@ public class Logins extends Check{
final long durBucket = 1000L * cc.loginsSeconds / 6;
final ActionFrequency freq = getActionFrequency(player.getWorld().getName(), 6, durBucket, cc.loginsPerWorldCount);
freq.update(now);
final boolean cancel = freq.score(1f) > cc.loginsLimit;
final boolean cancel = freq.score(1f) > cc.loginsLimit; // TODO: >= ... This will be 1 after the first login (!).
if (!cancel) freq.add(now, 1f);
return cancel;
}