Bukkit/Bukkit — pull request #1095 by bendem (2014-08-06 18:54:17+00:00)
Title: [B] Only loop trough op players when tab completing /deop. Fixes BUKKIT-5748

**The Issue**

The deop command currently goes trough all the offline players to and checks
if they're op when tab completing. With this commit, it only loops trough
the current operators.

**Justification for this PR**

Looping trough all offline players on big servers can be heavy for the server and is unecessary since there is a way to get the operators directly.

**PR Breakdown**

The new code iterates trough Bukkit.getOperators() instead of Bukkit.getOfflinePlayers() when tab completing the deop command. The now useless player.isOp() check has also be removed.

**Testing Results and Materials**

I'm not able to get a server with enough offline players to test the changes, but it seems pretty self explanatory to me.

**Relevant PR(s)**

This is PR contains a part of the changes proposed in #1094 but without the changes to the whitelist command.

**JIRA Ticket**
https://bukkit.atlassian.net/browse/BUKKIT-5748