@andrepl You talk about concern for how nms and the deobfuscation handles constants... this is why Bukkit conventions are to do minimal diffs so constant value changes are easily seen where they are done. There is no reason to embed the short1 variable under the if even if it's only used under the if.  Optimizing such a small piece of code that will ultimately be irrelevant is a negligble gain compared to an easily recognized diff change for the constant later.  Generally speaking, leave things where they are unless there is a specific logic requirement or notable performance impact.