bensku
commented
7 years ago It seems you cloned master while I was, let's say, experimenting. Try again now :)
Closed CDAGaming closed 7 years ago
bensku
commented
7 years ago It seems you cloned master while I was, let's say, experimenting. Try again now :)
CDAGaming
commented
7 years ago
` E:>cmd.exe Microsoft Windows [Version 10.0.16251.0] (c) 2017 Microsoft Corporation. All rights reserved.
E:>cd forked repos/minecraft repos
E:\Forked Repos\Minecraft Repos>cd Skrypt
E:\Forked Repos\Minecraft Repos\Skrypt>gradlew clean build Starting a Gradle Daemon (subsequent builds will be faster) :clean :compileJava
FIXME rename ^^^^^^^^^^^^ FIXME rename
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 70) private final static HashMap<String, ItemType> aliases_english = new HashMap<String, ItemType>(10000); ^^^^^^^ Redundant specification of type arguments <String, ItemType>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 71) private final static HashMap<String, ItemType> aliases_localised = new HashMap<String, ItemType>(1000); ^^^^^^^ Redundant specification of type arguments <String, ItemType>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 85) private final static HashMap<Integer, MaterialName> materialNames_english = new HashMap<Integer, MaterialName>(Material.values().length); ^^^^^^^ Redundant specification of type arguments <Integer, MaterialName>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 86) private final static HashMap<Integer, MaterialName> materialNames_localised = new HashMap<Integer, MaterialName>(Material.values().length); ^^^^^^^ Redundant specification of type arguments <Integer, MaterialName>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 185) final LinkedHashMap<String, ItemType> r = new LinkedHashMap<String, ItemType>(); // LinkedHashMap to preserve order for item names ^^^^^^^^^^^^^ Redundant specification of type arguments <String, ItemType>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 416) final NonNullPair<Short, Short> data = new NonNullPair<Short, Short>(Short.valueOf(d.dataMin), Short.valueOf(d.dataMax)); ^^^^^^^^^^^ Redundant specification of type arguments <Short, Short>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 570) final Map<Enchantment, Integer> enchantments = new HashMap<Enchantment, Integer>(); ^^^^^^^ Redundant specification of type arguments <Enchantment, Integer>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 770) final ArrayList aliasNodes = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 838) final HashMap<String, ItemType> vs = new HashMap<String, ItemType>(); ^^^^^^^ Redundant specification of type arguments <String, ItemType>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemData.java (at line 207) return new SingleItemIterator(new ItemStack(typeid, 1, dataMin == -1 ? 0 : dataMin));
^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 81) final ArrayList types = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 92) // TODO empty == unenchanted, add expression "unenchanted- "
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO empty == unenchanted, add expression "unenchanted
- "
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 163) enchantments = new HashMap<Enchantment, Integer>(i.getEnchantments()); ^^^^^^^ Redundant specification of type arguments <Enchantment, Integer>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 173) // TODO metadata - spawners, skulls, etc. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO metadata - spawners, skulls, etc.
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 189) enchantments = new HashMap<Enchantment, Integer>(i.enchantments); ^^^^^^^ Redundant specification of type arguments <Enchantment, Integer>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 307) // TODO metadata ^^^^^^^^^^^^^ TODO metadata
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 479) enchantments = enchs = new HashMap<Enchantment, Integer>(); ^^^^^^^ Redundant specification of type arguments <Enchantment, Integer>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 486) this.enchantments = new HashMap<Enchantment, Integer>(enchantments); ^^^^^^^ Redundant specification of type arguments <Enchantment, Integer>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 551) return new SingleItemIterable(i);
^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
REMIND If this ItemType represents multiple items with OR, this function will immediately return false.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND If this ItemType represents multiple items with OR, this function will immediately return false.
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 808) public boolean removeAll(final List... lists) {
^^^^^
Type safety: Potential heap pollution via varargs parameter lists
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 825) public boolean removeFrom(final List... lists) {
^^^^^
Type safety: Potential heap pollution via varargs parameter lists
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 1181) List rawNames = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\MaterialName.java (at line 32) final HashMap<NonNullPair<Short, Short>, NonNullPair<String, String>> names = new HashMap<NonNullPair<Short, Short>, NonNullPair<String, String>>(); ^^^^^^^ Redundant specification of type arguments <NonNullPair<Short,Short>, NonNullPair<String,String>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\MaterialName.java (at line 45) NonNullPair<String, String> s = names.get(new NonNullPair<Short, Short>(Short.valueOf(dataMin), Short.valueOf(dataMax))); ^^^^^^^^^^^ Redundant specification of type arguments <Short, Short>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\MaterialName.java (at line 50) s = names.get(new NonNullPair<Short, Short>((short) -1, (short) -1)); ^^^^^^^^^^^ Redundant specification of type arguments <Short, Short>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\MaterialName.java (at line 60) final NonNullPair<String, String> s = names.get(new NonNullPair<Short, Short>(Short.valueOf(dataMin), Short.valueOf(dataMax))); ^^^^^^^^^^^ Redundant specification of type arguments <Short, Short>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\bukkitutil\HealthUtils.java (at line 105) return e.getMaxHealth() / 2; ^^^^^^^^^^^^^^ The method getMaxHealth() from the type Damageable is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\bukkitutil\HealthUtils.java (at line 124) e.setMaxHealth(Math.max(Skript.EPSILON / 2, health * 2)); // 0 is not allowed, so just use a small value - smaller than Skript.EPSILON though to compare as 0 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The method setMaxHealth(double) from the type Damageable is deprecated
TODO check all updates and find out which ones are not required ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO check all updates and find out which ones are not required
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\bukkitutil\UnresolvedOfflinePlayer.java (at line 111) @SuppressWarnings("deprecation") ^^^^^^^^^^^^^ Unnecessary @SuppressWarnings("deprecation")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\ConfigurationSerializer.java (at line 57) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 89) // TODO vectors ^^^^^^^^^^^^ TODO vectors
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 431) // FIXME update doc ^^^^^^^^^^^^^^^^ FIXME update doc
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 449) // REMIND allow shortcuts '[over]world', 'nether' and '[the]end' (server.properties: 'level-name=world') // inconsistent with 'world is "..."' ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND allow shortcuts '[over]world', 'nether' and '[the]end' (server.properties: 'level-name=world') // inconsistent with 'world is "..."'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 713) // TODO return an unresolved player and resolve it on a different thread after the command was parsed, and block the command until it is ready ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO return an unresolved player and resolve it on a different thread after the command was parsed, and block the command until it is ready
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 714) // FIXME add note to changelog if not fixed in the next update ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME add note to changelog if not fixed in the next update
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitClasses.java (at line 1070) // REMIND make my own damage cause class (that e.g. stores the attacker entity, the projectile, or the attacking block) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND make my own damage cause class (that e.g. stores the attacker entity, the projectile, or the attacking block)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitEventValues.java (at line 187) // REMIND workaround of the event's location being at the entity in block events that have an entity event value ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND workaround of the event's location being at the entity in block events that have an entity event value
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\BukkitEventValues.java (at line 781) // CraftItemEvent REMIND maybe re-add this when Skript parser is reworked? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND maybe re-add this when Skript parser is reworked?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 53) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 67) case RESET: // REMIND reset entity? (unshear, remove held item, reset weapon/armour, ...) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND reset entity? (unshear, remove held item, reset weapon/armour, ...)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 141) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 162) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 184) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 277) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultChangers.java (at line 282) return null; // REMIND regenerate? ^^^^^^^^^^^^^^^^^^ REMIND regenerate?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultComparators.java (at line 204) // TODO !Update with every version [entities] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [entities]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultComparators.java (at line 213) > :compileJava // entityMaterials.put(Fish.class, Material.RAW_FISH); // TODO 1.7 ^^^^^^^^ TODO 1.7
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultConverters.java (at line 83) // TODO improve handling of interfaces ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO improve handling of interfaces
FIXME generate/write documentation ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME generate/write documentation
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultFunctions.java (at line 306) // FIXME document on ExprLocation as well ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME document on ExprLocation as well
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\SkriptClasses.java (at line 465) // TODO remove ^^^^^^^^^^^ TODO remove
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\Serializer.java (at line 126) @SuppressWarnings({"unchecked", "null"}) ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\VectorArithmethic.java (at line 37) return new Vector().add(value).add(difference); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Vector' needs unchecked conversion to conform to '@NonNull Vector'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\VectorArithmethic.java (at line 42) return new Vector().add(value).subtract(difference); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Vector' needs unchecked conversion to conform to '@NonNull Vector'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\VectorArithmethic.java (at line 47) return value.clone().multiply(multiplier); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Vector' needs unchecked conversion to conform to '@NonNull Vector'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\VectorArithmethic.java (at line 52) return value.clone().divide(divider); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Vector' needs unchecked conversion to conform to '@NonNull Vector'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\Argument.java (at line 60) private transient WeakHashMap<Event, T[]> current = new WeakHashMap<Event, T[]>(); ^^^^^^^^^^^ Redundant specification of type arguments <Event, T[]>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\Argument.java (at line 99) d = (Expression<? extends T>) new SimpleLiteral(def, false);
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\Argument.java (at line 113) return new Argument(name, d, type, single, index, def != null || forceOptional);
^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\CommandHelp.java (at line 53) private final LinkedHashMap<String, Object> arguments = new LinkedHashMap<String, Object>(); ^^^^^^^^^^^^^ Redundant specification of type arguments <String, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\Commands.java (at line 83) //TODO option to disable replacement ofs in command arguments?
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO option to disable replacement of s in command arguments?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 121) activeAliases = new ArrayList(aliases);
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 249) private transient Map<String, Command> overriddenAliases = new HashMap<String, Command>(); ^^^^^^^ Redundant specification of type arguments <String, Command>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 283) activeAliases = new ArrayList(this.aliases);
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 301) private transient Collection helps = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 316) final ArrayList as = new ArrayList((Collection) topics.get(aliases));
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\command\ScriptCommand.java (at line 338) final ArrayList as = new ArrayList((Collection) topics.get(aliases));
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\conditions\CondContains.java (at line 154) // TODO support similar odd contains checks for inventories ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO support similar odd contains checks for inventories
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\config\Config.java (at line 257) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\config\Config.java (at line 276) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
REMIND breaks saving - separator argument can be different from config.sepator ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND breaks saving - separator argument can be different from config.sepator
TODO list special expressions for events and event values ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO list special expressions for events and event values
TODO compare doc in code with changed one of the webserver and warn about differences? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO compare doc in code with changed one of the webserver and warn about differences?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffBan.java (at line 93) Bukkit.getBanList(BanList.Type.NAME).addBan(((Player) o).getName(), reason, expires, source); // FIXME [UUID] ban UUID ^^^^^^^^^^^^^^^^^^^^^ FIXME [UUID] ban UUID
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffCancelEvent.java (at line 75) return true; // TODO warning if some event(s) cannot be cancelled even though some can (needs a way to be suppressed) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO warning if some event(s) cannot be cancelled even though some can (needs a way to be suppressed)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffChange.java (at line 271) changed.change(e, delta, mode); // REMIND use a random element out of delta if changed only supports changing a single instance ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND use a random element out of delta if changed only supports changing a single instance
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffEquip.java (at line 126) switch (item.getType()) {// TODO !Update with every version [items] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [items]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffEquip.java (at line 172) // //REMIND this + think... ^^^^^^^^^^^^^^^^^^^^^^ REMIND this + think...
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffExit.java (at line 56) public class EffExit extends Effect { // TODO [code style] warn user about code after a stop effect ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO [code style] warn user about code after a stop effect
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffIgnite.java (at line 103) return "set " + entities.toString(e, debug) + " on fire for " + (duration != null ? duration.toString(e, debug) : Timespan.fromTicks(DEFAULT_DURATION).toString()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^ The method fromTicks(int) from the type Timespan is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffLog.java (at line 103) final File f = new File(logsFolder, s); // REMIND what if s contains '..'? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND what if s contains '..'?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffPush.java (at line 81) en.setVelocity(en.getVelocity().add(mod)); // REMIND add NoCheatPlus exception to players ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND add NoCheatPlus exception to players
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\EffToggle.java (at line 66) // TODO !Update with every version [blocks] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [blocks]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\effects\mirre\EffScriptOptionLoops.java (at line 20) package ch.njol.skript.effects.mirre; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ A default nullness annotation has not been specified for the package ch.njol.skript.effects.mirre
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 70) public abstract class EntityData implements SyntaxElement, YggdrasilExtendedSerializable {// TODO extended horse support, zombie villagers // REMIND unit
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO extended horse support, zombie villagers //
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 70) public abstract class EntityData implements SyntaxElement, YggdrasilExtendedSerializable {// TODO extended horse support, zombie villagers // REMIND unit
^^^^^^^^^^^
REMIND unit
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 79) private final static List<EntityDataInfo<?>> infos = new ArrayList<EntityDataInfo<?>>(); ^^^^^^^^^ Redundant specification of type arguments <EntityData.EntityDataInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 150) Classes.registerClass(new ClassInfo(EntityData.class, "entitydata")
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 245) final EntityDataInfo info = new EntityDataInfo(dataClass, name, codeNames, defaultName, entityClass);
^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 448) final List list = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 469) final List list = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityData.java (at line 482) final List list = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityType.java (at line 41) Classes.registerClass(new ClassInfo(EntityType.class, "entitytype")
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\EntityType.java (at line 47) .defaultExpression(new SimpleLiteral(new EntityType(Entity.class, 1), true))
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\MinecartData.java (at line 40) @SuppressWarnings("deprecation") ^^^^^^^^^^^^^ Unnecessary @SuppressWarnings("deprecation")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\SimpleEntityData.java (at line 236) // TODO !Update with every version [entities] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [entities]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\entity\ZombieVillagerData.java (at line 35) private final static boolean villagerSupport = Skript.isRunningMinecraft(1, 11); // TODO test on 1.9/1.10 ^^^^^^^^^^^^^^^^^^^^^ TODO test on 1.9/1.10
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtBlock.java (at line 63) // TODO 'block destroy' event for any kind of block destruction (player, water, trampling, fall (sand, toches, ...), etc) -> BlockPhysicsEvent? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO 'block destroy' event for any kind of block destruction (player, water, trampling, fall (sand, toches, ...), etc) -> BlockPhysicsEvent?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtBlock.java (at line 64) // REMIND attacking an item frame first removes its item; include this in on block damage? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND attacking an item frame first removes its item; include this in on block damage?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtBlock.java (at line 135) //Hacky code to fix Java 8 compilation problems... TODO maybe use lambdas instead and break Java 7? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO maybe use lambdas instead and break Java 7?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtCommand.java (at line 38) public class EvtCommand extends SkriptEvent { // TODO condition to check whether a given command exists, & a conditon to check whether it's a custom skript command ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO condition to check whether a given command exists, & a conditon to check whether it's a custom skript command
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 30) import org.bukkit.event.player.PlayerPickupItemEvent; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The type PlayerPickupItemEvent is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 62) // TODO limit to InventoryAction.PICKUP_* and similar (e.g. COLLECT_TOCURSOR) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO limit to InventoryAction.PICKUP
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 73) Skript.registerEvent("Pick Up", EvtItem.class, PlayerPickupItemEvent.class, "[player] (pick[ ]up|picking up) [[of] %itemtypes%]") ^^^^^^^^^^^^^^^^^^^^^ The type PlayerPickupItemEvent is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 77) // TODO brew event ^^^^^^^^^^^^^^^ TODO brew event
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 123) } else if (e instanceof PlayerPickupItemEvent) { ^^^^^^^^^^^^^^^^^^^^^ The type PlayerPickupItemEvent is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 124) is = ((PlayerPickupItemEvent) e).getItem().getItemStack(); ^^^^^^^^^^^^^^^^^^^^^ The type PlayerPickupItemEvent is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtItem.java (at line 124) is = ((PlayerPickupItemEvent) e).getItem().getItemStack(); ^^^^^^^^^ The method getItem() from the type PlayerPickupItemEvent is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtMoveOn.java (at line 54) public class EvtMoveOn extends SelfRegisteringSkriptEvent { // TODO on jump ^^^^^^^^^^^^ TODO on jump
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtPressurePlate.java (at line 39) // TODO is EntityInteractEvent similar for entities? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO is EntityInteractEvent similar for entities?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtPressurePlate.java (at line 59) // TODO !Update with every version [blocks] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [blocks]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\EvtSkript.java (at line 40) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\SimpleEvents.java (at line 242) // TODO improve - on fish [of %entitydata%] (and/or itemtype), on reel, etc. ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO improve - on fish [of %entitydata%] (and/or itemtype), on reel, etc.
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\events\SimpleEvents.java (at line 275) // TODO level up/down ^^^^^^^^^^^^^^^^^^ TODO level up/down
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\base\EventValueExpression.java (at line 81) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\base\WrapperExpression.java (at line 72) protected ConvertedExpression<T, ? extends R> getConvertedExpr(final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprAltitude.java (at line 44) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
TODO should 'amount of [item]' return the size of the stack? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO should 'amount of [item]' return the size of the stack?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprAttacked.java (at line 65) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprBlocks.java (at line 58) "[the] blocks %direction% [%locations%]", // TODO doesn't loop all blocks? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO doesn't loop all blocks?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprChatRecipients.java (at line 69) @SuppressWarnings({"unchecked", "null"}) ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprColorOf.java (at line 82) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprCoordinate.java (at line 61) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprDamage.java (at line 77) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprDirection.java (at line 74) // TODO think about parsing statically & dynamically (also in general) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO think about parsing statically & dynamically (also in general)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprDirection.java (at line 76) // TODO direction of %location% (from|relative to) %location% ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO direction of %location% (from|relative to) %location%
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprDrops.java (at line 85) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprDurability.java (at line 73) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
TODO actually allow to have triggers execute for cancelled events ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO actually allow to have triggers execute for cancelled events
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprEventCancelled.java (at line 79) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprFacing.java (at line 91) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprFoodLevel.java (at line 86) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprGameMode.java (at line 88) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprHealth.java (at line 100) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprIdOf.java (at line 161) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLastDamageCause.java (at line 71) return dmgEvt.getCause(); ^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'EntityDamageEvent.DamageCause' needs unchecked conversion to conform to '@NonNull EntityDamageEvent.DamageCause'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLastDamageCause.java (at line 81) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLevel.java (at line 58) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLevelProgress.java (at line 62) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLightLevel.java (at line 77) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
FIXME doesn't parse - update documentation when fixed ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME doesn't parse - update documentation when fixed
TODO expression to get the current # of execution (e.g. loop-index/number/count/etc (not number though)); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO expression to get the current # of execution (e.g. loop-index/number/count/etc (not number though));
TODO make a 'line %number% of %text%' expression and figure out how to deal with signs (4 lines, delete = empty, etc...) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO make a 'line %number% of %text%' expression and figure out how to deal with signs (4 lines, delete = empty, etc...)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLore.java (at line 137) // TODO test (especially remove) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO test (especially remove)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLore.java (at line 178) final int l = Math2.fit(0, n.intValue() - 1, 99); // TODO figure out the actual maximum ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO figure out the actual maximum
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprMaxHealth.java (at line 53) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprMaxHealth.java (at line 98) en.resetMaxHealth(); ^^^^^^^^^^^^^^^^ The method resetMaxHealth() from the type Damageable is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprMessage.java (at line 191)ilding 6% > :compileJava @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprName.java (at line 257) // TODO find a better method for handling changes (in general) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO find a better method for handling changes (in general)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprName.java (at line 259) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprParse.java (at line 127) if (p == null || !p.canParse(ParseContext.COMMAND)) { // TODO special parse context? ^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO special parse context?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprPassenger.java (at line 68) public class ExprPassenger extends SimpleExpression { // REMIND create 'vehicle' and 'passenger' expressions for vehicle enter/exit events?
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
REMIND create 'vehicle' and 'passenger' expressions for vehicle enter/exit events?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprPotionEffect.java (at line 36) // REMIND create ^^^^^^^^^^^^^ REMIND create
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprRemainingAir.java (at line 56) return Timespan.fromTicks(e.getRemainingAir()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The method fromTicks(int) from the type Timespan is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSignText.java (at line 110) // TODO allow add, remove, and remove all (see ExprLore) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO allow add, remove, and remove all (see ExprLore)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSkull.java (at line 28) import org.bukkit.entity.Skeleton.SkeletonType; ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The type Skeleton.SkeletonType is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSkull.java (at line 75) if (o instanceof SkeletonData ? ((SkeletonData) o).isWither() : ((Skeleton) o).getSkeletonType() == SkeletonType.WITHER) { ^^^^^^^^^^^^^^^^^ The method getSkeletonType() from the type Skeleton is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSkull.java (at line 75) if (o instanceof SkeletonData ? ((SkeletonData) o).isWither() : ((Skeleton) o).getSkeletonType() == SkeletonType.WITHER) { ^^^^^^^^^^^^ The type Skeleton.SkeletonType is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSkull.java (at line 75) if (o instanceof SkeletonData ? ((SkeletonData) o).isWither() : ((Skeleton) o).getSkeletonType() == SkeletonType.WITHER) { ^^^^^^ The field Skeleton.SkeletonType.WITHER is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSpawn.java (at line 87) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprSpeed.java (at line 65) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprTarget.java (at line 108) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprTime.java (at line 73) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprTypeOf.java (at line 72) protected ConvertedExpression<Object, ? extends R> getConvertedExpr(final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprUnbreakable.java (at line 84) ((ItemMeta) meta).spigot().setUnbreakable(true); ^^^^^^^^^^^^^^^^^^^^ The method setUnbreakable(boolean) from the type ItemMeta.Spigot is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprUUID.java (at line 42) // TODO [UUID] update documentation after release. Add note about requiring Bukkit 1.7.(9/10)? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO [UUID] update documentation after release. Add note about requiring Bukkit 1.7.(9/10)?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprValue.java (at line 38) // static { // REMIND add this (>2.0) ^^^^^^^^^^^^^^^^^^^^^^ REMIND add this (>2.0)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprValue.java (at line 73) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprVehicle.java (at line 122) ((Entity) o).setPassenger(p); ^^^^^^^^^^^^^^^ The method setPassenger(Entity) from the type Entity is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprVehicle.java (at line 129) v.setPassenger(p); ^^^^^^^^^^^^^^^ The method setPassenger(Entity) from the type Entity is deprecated
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprWeather.java (at line 85) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprYawPitch.java (at line 74) @SuppressWarnings({"unchecked", "null"}) ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\hooks\regions\RegionsPlugin.java (at line 42) // REMIND support more plugins? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND support more plugins?
TODO: make this an expression ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO: make this an expression
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Expression.java (at line 135) public Expression<? extends R> getConvertedExpression(final Class... to);
^^
Type safety: Potential heap pollution via varargs parameter to
This method must return true if this is a {@link #isSingle() single} expression. // TODO make this method irrelevant for single expressions ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO make this method irrelevant for single expressions
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ExpressionList.java (at line 114) final ArrayList r = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ExpressionList.java (at line 193) return new ExpressionList(exprs, (Class) Utils.getSuperType(to), and, this);
^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ExpressionList.java (at line 230) final ArrayList<Class<?>> r = new ArrayList<Class<?>>(Arrays.asList(l)); ^^^^^^^^^ Redundant specification of type arguments <Class<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ExpressionList.java (at line 330) return new SimpleLiteral(values, returnType, and);
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ExpressionList.java (at line 335) return new LiteralList(ls, returnType, and);
^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\Function.java (at line 71) // TODO allow setting parameters by name ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO allow setting parameters by name
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\Function.java (at line 84) // FIXME what happens with a delay in a function? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME what happens with a delay in a function?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\FunctionReference.java (at line 189) l.addAll(Arrays.asList(parameters[i].getArray(e))); // TODO what if an argument is not available? pass null or abort? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO what if an argument is not available? pass null or abort?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\FunctionReference.java (at line 193) params[i] = parameters[i].getArray(e); // TODO what if an argument is not available? pass null or abort? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO what if an argument is not available? pass null or abort?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\Functions.java (at line 150) final String name = "" + m.group(1); // TODO check for name uniqueness (currently functions with same name silently override each other) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO check for name uniqueness (currently functions with same name silently override each other)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\function\ScriptFunction.java (at line 74) // REMIND track possible types of local variables (including undefined variables) (consider functions, commands, and EffChange) - maybe make a general interface for this purpose ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND track possible types of local variables (including undefined variables) (consider functions, commands, and EffChange) - maybe make a general interface for this purpose
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Literal.java (at line 37) public Literal<? extends R> getConvertedExpression(Class... to);
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\LiteralList.java (at line 73) return new LiteralList(exprs, (Class) Utils.getSuperType(classes), and, this);
^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\LiteralList.java (at line 91) return new SimpleLiteral(values, getReturnType(), and);
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Loop.java (at line 45) private transient Map<Event, Object> current = new WeakHashMap<Event, Object>(); ^^^^^^^^^^^ Redundant specification of type arguments <Event, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Loop.java (at line 46) private transient Map<Event, Iterator<?>> currentIter = new WeakHashMap<Event, Iterator<?>>(); ^^^^^^^^^^^ Redundant specification of type arguments <Event, Iterator<?>>
TODO? replace with {@link #DUMMY} + {@link SkriptParser#PARSE_LITERALS} ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO? replace with {@link #DUMMY} + {@link SkriptParser#PARSE_LITERALS}
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 382) final Variable<? extends T> var = parseVariable(expr, types); ^^^^^ types cannot be resolved to a variable
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 395) final FunctionReference fr = parseFunction(types);
^^^^^
types cannot be resolved to a variable
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 398) return new ExprFunctionCall(fr); ^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: The constructor ExprFunctionCall(FunctionReference) belongs to the raw type ExprFunctionCall. References to generic type ExprFunctionCall should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 398) return new ExprFunctionCall(fr); ^^^^^^^^^^^^^^^^ ExprFunctionCall is a raw type. References to generic type ExprFunctionCall should be parameterized
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 407) if (expr.startsWith("\"") && expr.endsWith("\"") && expr.length() != 1 && (types[0] == Object.class || CollectionUtils.contains(types, String.class))) { ^^^^^ types cannot be resolved to a variable
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 407) if (expr.startsWith("\"") && expr.endsWith("\"") && expr.length() != 1 && (types[0] == Object.class || CollectionUtils.contains(types, String.class))) { ^^^^^ types cannot be resolved to a variable
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 410) e = (Expression<?>) parse(expr, (Iterator) Skript.getExpressions(types), null); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: Unchecked invocation parse(String, Iterator, null) of the generic method parse(String, Iterator<? extends SyntaxElementInfo>, String) of type SkriptParser
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 410) e = (Expression<?>) parse(expr, (Iterator) Skript.getExpressions(types), null); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: The expression of type Iterator needs unchecked conversion to conform to Iterator<? extends SyntaxElementInfo>
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 410) e = (Expression<?>) parse(expr, (Iterator) Skript.getExpressions(types), null); ^^^^^ types cannot be resolved to a variable
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 413) for (final Class<? extends T> t : types) { ^^^^^ types cannot be resolved to a variable
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 417) return (Expression<? extends T>) e; ^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: Unchecked cast from Expression<capture#70-of ?> to Expression<? extends T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 417) return (Expression<? extends T>) e; ^^^^^^^^^^^^^^^^^^^^^^^^^^^ Unnecessary cast from Expression<capture#70-of ?> to Expression<? extends T>
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 422) Class[] objTypes = (Class[]) types; // Java generics... ?
^^^^^
types cannot be resolved to a variable
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 429) log.printError(e.toString(null, false) + " " + Language.get("is") + " " + notOfType(types), ErrorQuality.NOT_AN_EXPRESSION); ^^^^^ types cannot be resolved to a variable
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 685) final Expression<?> e = p.parseExpression(c); ^^^^^^^^^^^^^^^^^^^^ Type safety: A generic array of Class<? extends capture#111-of ?> is created for a varargs parameter
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 688) return (Expression<? extends T>) e; ^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: Unchecked cast from Expression<capture#115-of ?> to Expression<? extends T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 688) return (Expression<? extends T>) e; ^^^^^^^^^^^^^^^^^^^^^^^^^^^ Unnecessary cast from Expression<capture#115-of ?> to Expression<? extends T>
ERROR in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 695) final Expression<? extends T> r = parseSingleExpr(false, null, vi); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type mismatch: cannot convert from Expression<capture#117-of ?> to Expression<? extends T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 734) return (Expression<? extends T>) new UnparsedLiteral(expr, log.getError()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: Unchecked cast from UnparsedLiteral to Expression<? extends T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 734) return (Expression<? extends T>) new UnparsedLiteral(expr, log.getError()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Unnecessary cast from UnparsedLiteral to Expression<? extends T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 843) return new LiteralList<>(ls, (Class) Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Type safety: Unchecked cast from Class<capture#133-of ?> to Class
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 847) return new ExpressionList<>(es, (Class) Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Type safety: Unchecked cast from Class<capture#134-of ?> to Class
TODO find a better name for this ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO find a better name for this
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SyntaxElement.java (at line 62) // TODO [code style] add equals to be able to find out whether two elements are equal (useful for e.g. 'if : ... [else] if not : ...') ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO [code style] add equals to be able to find out whether two elements are equal (useful for e.g. 'if : ... [else] if not : ...')
TODO implement this ^^^^^^^^^^^^^^^^^^^ TODO implement this
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\UnparsedLiteral.java (at line 83) public Literal<? extends R> getConvertedExpression(final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\UnparsedLiteral.java (at line 88) public Literal<? extends R> getConvertedExpression(final ParseContext context, final Class<? extends R>... to) {
^^ Building 6% > :compileJava
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\UnparsedLiteral.java (at line 98) return new SimpleLiteral(r, false);
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\ConvertedExpression.java (at line 68) public static <F, T> ConvertedExpression<F, T> newInstance(final Expression v, final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\ConvertedExpression.java (at line 70) for (final Class c : to) { // REMIND try more converters? -> also change WrapperExpression (and maybe ExprLoopValue)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
REMIND try more converters? -> also change WrapperExpression (and maybe ExprLoopValue)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\ConvertedExpression.java (at line 78) return new ConvertedExpression<F, T>(v, c, conv); ^^^^^^^^^^^^^^^^^^^ Redundant specification of type arguments <F, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\ConvertedExpression.java (at line 178) return c.check((T) f); ^^^^^ Type safety: Unchecked cast from F to T
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\ConvertedLiteral.java (at line 102) return new ArrayIterator(data);
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\SimpleExpression.java (at line 166) // TODO return a kleenean (UNKNOWN if 'all' is null or empty) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO return a kleenean (UNKNOWN if 'all' is null or empty)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\SimpleExpression.java (at line 198) protected ConvertedExpression<T, ? extends R> getConvertedExpr(final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\SimpleExpression.java (at line 274) protected final boolean setTime(final int time, final Expression<?> mustbeDefaultVar, final Class<? extends Event>... applicableEvents) { ^^^^^^^^^^^^^^^^ Type safety: Potential heap pollution via varargs parameter applicableEvents
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\util\SimpleExpression.java (at line 308) return new ArrayIterator(getArray(e));
^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Variable.java (at line 193) public Variable getConvertedExpression(final Class... to) {
^^
Type safety: Potential heap pollution via varargs parameter to
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Variable.java (at line 241) @SuppressWarnings({"deprecation"}) ^^^^^^^^^^^^^ Unnecessary @SuppressWarnings("deprecation")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Variable.java (at line 378) @SuppressWarnings("unchecked") ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\VariableString.java (at line 697) /* REMIND allow special characters? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND allow special characters?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\localization\Language.java (at line 236) l.putAll(load(new FileInputStream(f), name)); ^^^^^^^^^^^^^^^^^^^^^^ Potential resource leak: '' may not be closed
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\localization\Noun.java (at line 43) // TODO remove NO_GENDER and add boolean/flag uncountable (e.g. Luft: 'die Luft', aber nicht 'eine Luft') ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO remove NO_GENDER and add boolean/flag uncountable (e.g. Luft: 'die Luft', aber nicht 'eine Luft')
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\log\ParseLogHandler.java (at line 38) private final List log = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\log\RetainingLogHandler.java (at line 40) private final Deque log = new LinkedList();
^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\log\RetainingLogHandler.java (at line 189) final Collection r = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\mirre\FilterPrintStream.java (at line 20) package ch.njol.skript.mirre; ^^^^^^^^^^^^^^^^^^^^ A default nullness annotation has not been specified for the package ch.njol.skript.mirre
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 77) private final static List<ClassInfo<?>> tempClassInfos = new ArrayList<ClassInfo<?>>(); ^^^^^^^^^ Redundant specification of type arguments <ClassInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 78) private final static HashMap<Class<?>, ClassInfo<?>> exactClassInfos = new HashMap<Class<?>, ClassInfo<?>>(); ^^^^^^^ Redundant specification of type arguments <Class<?>, ClassInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 79) private final static HashMap<Class<?>, ClassInfo<?>> superClassInfos = new HashMap<Class<?>, ClassInfo<?>>(); ^^^^^^^ Redundant specification of type arguments <Class<?>, ClassInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 80) private final static HashMap<String, ClassInfo<?>> classInfosByCodeName = new HashMap<String, ClassInfo<?>>(); ^^^^^^^ Redundant specification of type arguments <String, ClassInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 164) final Set s = new HashSet();
^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 184) final List<ClassInfo<?>> classInfos = new ArrayList<ClassInfo<?>>(tempClassInfos.size()); ^^^^^^^^^ Redundant specification of type arguments <ClassInfo<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 654) assert Enum.class.isAssignableFrom(Kleenean.class) && Tag.getType(Kleenean.class) == Tag.T_ENUM : Tag.getType(Kleenean.class); // TODO why is this check here? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO why is this check here?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Classes.java (at line 683) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 44) public final static Collection<ComparatorInfo<?, ?>> comparators = new ArrayList<ComparatorInfo<?, ?>>(); ^^^^^^^^^ Redundant specification of type arguments <Comparator.ComparatorInfo<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 58) comparators.add(new ComparatorInfo<T1, T2>(t1, t2, c)); ^^^^^^^^^^^^^^ Redundant specification of type arguments <T1, T2>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 65) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 83) private final static Map<Pair<Class<?>, Class<?>>, Comparator<?, ?>> comparatorsQuickAccess = new HashMap<Pair<Class<?>, Class<?>>, Comparator<?, ?>>(); ^^^^^^^ Redundant specification of type arguments <Pair<Class<?>,Class<?>>, Comparator<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 88) final Pair<Class<?>, Class<?>> p = new Pair<Class<?>, Class<?>>(f, s); ^^^^ Redundant specification of type arguments <Class<?>, Class<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 124) return first ? new ConvertedComparator<F, S>(info.c, c2) : new InverseComparator<F, S>(new ConvertedComparator<S, F>(c2, info.c)); ^^^^^^^^^^^^^^^^^ Redundant specification of type arguments <F, S>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 130) return !first ? new ConvertedComparator<F, S>(c1, info.c) : new InverseComparator<F, S>(new ConvertedComparator<S, F>(info.c, c1)); ^^^^^^^^^^^^^^^^^ Redundant specification of type arguments <F, S>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Comparators.java (at line 142) return first ? new ConvertedComparator<F, S>(c1, info.c, c2) : new InverseComparator<F, S>(new ConvertedComparator<S, F>(c2, info.c, c1)); ^^^^^^^^^^^^^^^^^ Redundant specification of type arguments <F, S>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 46) private static List<ConverterInfo<?, ?>> converters = new ArrayList<ConverterInfo<?, ?>>(50); ^^^^^^^^^ Redundant specification of type arguments <Converter.ConverterInfo<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 71) final ConverterInfo<F, T> info = new ConverterInfo<F, T>(from, to, converter, options); ^^^^^^^^^^^^^ Redundant specification of type arguments <F, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 87) // REMIND how to manage overriding of converters? - shouldn't actually matter ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND how to manage overriding of converters? - shouldn't actually matter
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 115) return new ConverterInfo<F, T>((Class) first.from, (Class) second.to, new ChainedConverter<F, M, T>((Converter<F, M>) first.converter, (Converter<M, T>) second.converter), first.options | second.options);
^^^^^^^^^^^^^
Redundant specification of type arguments <F, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 115) return new ConverterInfo<F, T>((Class) first.from, (Class) second.to, new ChainedConverter<F, M, T>((Converter<F, M>) first.converter, (Converter<M, T>) second.converter), first.options | second.options);
^^^^^^^^^^^^^^^^
Redundant specification of type arguments <F, M, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 133) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 183) final List l = new ArrayList(o.length);
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 210) final List l = new ArrayList(o.length);
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 221) private final static Map<Pair<Class<?>, Class<?>>, Converter<?, ?>> convertersCache = new HashMap<Pair<Class<?>, Class<?>>, Converter<?, ?>>(); ^^^^^^^ Redundant specification of type arguments <Pair<Class<?>,Class<?>>, Converter<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\Converters.java (at line 255) final Pair<Class<?>, Class<?>> p = new Pair<Class<?>, Class<?>>(from, to); ^^^^ Redundant specification of type arguments <Class<?>, Class<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 62) private final static List<EventValueInfo<?, ?>> defaultEventValues = new ArrayList<EventValueInfo<?, ?>>(30); ^^^^^^^^^ Redundant specification of type arguments <EventValues.EventValueInfo<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 63) private final static List<EventValueInfo<?, ?>> futureEventValues = new ArrayList<EventValueInfo<?, ?>>(); ^^^^^^^^^ Redundant specification of type arguments <EventValues.EventValueInfo<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 64) private final static List<EventValueInfo<?, ?>> pastEventValues = new ArrayList<EventValueInfo<?, ?>>(); ^^^^^^^^^ Redundant specification of type arguments <EventValues.EventValueInfo<?,?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 104) public static <T, E extends Event> void registerEventValue(final Class e, final Class c, final Getter<T, E> g, final int time, final @Nullable String excludeErrorMessage, final @Nullable Class<? extends E>... excludes) {
^^^^^^^^
Type safety: Potential heap pollution via varargs parameter excludes
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 110) eventValues.add(i, new EventValueInfo<E, T>(e, c, g, excludeErrorMessage, excludes)); ^^^^^^^^^^^^^^ Redundant specification of type arguments <E, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 114) eventValues.add(new EventValueInfo<E, T>(e, c, g, excludeErrorMessage, excludes)); ^^^^^^^^^^^^^^ Redundant specification of type arguments <E, T>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 118) public static <T, E extends Event> void registerEventValue(final Class e, final Class c, final ch.njol.skript.classes.SerializableGetter<T, E> g, final int time, final @Nullable String excludeErrorMessage, final @Nullable Class<? extends E>... excludes) {
^^^^^^^^
Type safety: Potential heap pollution via varargs parameter excludes
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\registrations\EventValues.java (at line 136) @SuppressWarnings({"null", "unchecked"}) ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\ScriptLoader.java (at line 129) public static void setCurrentEvent(final String name, final @Nullable Class<? extends Event>... events) { ^^^^^^ Type safety: Potential heap pollution via varargs parameter events
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\ScriptLoader.java (at line 451) // TODO allow to make these override existing variables ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO allow to make these override existing variables
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\ScriptLoader.java (at line 711) final Signature<?> func = Functions.loadSignature(config.getFileName(), node); ^^^^ The value of the local variable func is not used
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Skript.java (at line 133) // TODO meaningful error if someone uses an %expression with percent signs% outside of text or a variable ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO meaningful error if someone uses an %expression with percent signs% outside of text or a variable
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Skript.java (at line 354) Documentation.generate(); // TODO move to test classes? ^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO move to test classes?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Skript.java (at line 748) EvtSkript.onSkriptStop(); // TODO [code style] warn user about delays in Skript stop events ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO [code style] warn user about delays in Skript stop events
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Skript.java (at line 857) // TODO localise Infinity, -Infinity, NaN (and decimal point?) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO localise Infinity, -Infinity, NaN (and decimal point?)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Skript.java (at line 1053) @SuppressWarnings({"unchecked"}) ^^^^^^^^^^^ Unnecessary @SuppressWarnings("unchecked")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 80) // TODO /skript scripts show/list - lists all enabled and/or disabled scripts in the scripts folder and/or subfolders (maybe add a pattern [using * and *]) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO /skript scripts show/list - lists all enabled and/or disabled scripts in the scripts folder and/or subfolders (maybe add a pattern [using and **])
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 81) // TODO document this command on the website ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO document this command on the website
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 124) private final static ArgsMessage m_changes_title = new ArgsMessage(NODE + ".update.changes.title"); ^^^^^^^^^^^^^^^ The value of the field SkriptCommand.m_changes_title is not used
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 139) private final static void message(final CommandSender sender, String what, final Object... args) { ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ The method message(CommandSender, String, Object...) from the type SkriptCommand is never used locally
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 282) if (f == null) // TODO allow disabling deleted/renamed scripts ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO allow disabling deleted/renamed scripts
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 351) // TODO not supported yet ^^^^^^^^^^^^^^^^^^^^^^ TODO not supported yet
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptCommand.java (at line 353) // TODO not supported yet ^^^^^^^^^^^^^^^^^^^^^^ TODO not supported yet
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\SkriptConfig.java (at line 101) public final static Option usePlayerUUIDsInVariableNames = new Option("use player UUIDs in variable names", false); // TODO change to true later (as well as in the default config)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO change to true later (as well as in the default config)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\Updater.java (at line 312) // TODO automatic downloading ^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO automatic downloading
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\BlockStateBlock.java (at line 389) // TODO Auto-generated method stub ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO Auto-generated method stub
TODO !Update with every version [blocks] - also update aliases-.sk ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [blocks] - also update aliases-.sk
@param applyPhysics TODO add effect that sets block without physics checks ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO add effect that sets block without physics checks
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\BlockUtils.java (at line 256) // REMIND rails? ^^^^^^^^^^^^^ REMIND rails?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\BlockUtils.java (at line 264) // TODO !Update with every version [blocks] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [blocks]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\chat\BungeeConverter.java (at line 43) if (origin.color != null) // TODO this is crappy way to copy color over... ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO this is crappy way to copy color over...
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\chat\ChatMessages.java (at line 203) code.updateComponent(current, param); // Call ChatCode update ^^^^^ Null type safety: The expression of type 'String' needs unchecked conversion to conform to '@NonNull String'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\DelayedChangeBlock.java (at line 387) // TODO Auto-generated method stub ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO Auto-generated method stub
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\EnchantmentType.java (at line 101) // REMIND flags? ^^^^^^^^^^^^^ REMIND flags?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\EquipmentSlot.java (at line 177) return "the " + slot.name().toLowerCase(Locale.ENGLISH) + " of " + Classes.toString(e.getHolder()); // TODO localise? ^^^^^^^^^^^^^^ TODO localise?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\EquipmentSlot.java (at line 192) return false; // TODO maybe fix this ^^^^^^^^^^^^^^^^^^^ TODO maybe fix this
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\PotionEffectUtils.java (at line 89) // REMIND flags? ^^^^^^^^^^^^^ REMIND flags?
Represents a container for a single item. This can e.g. be an inventory slot or an item frame. // TODO actually make an item frame slot ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO actually make an item frame slot
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Timespan.java (at line 39) public class Timespan implements YggdrasilSerializable, Comparable { // REMIND unit
^^^^^^^^^^^
REMIND unit
TODO !Update with every version [blocks] ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO !Update with every version [blocks]
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Utils.java (at line 399) private final static Pattern stylePattern = Pattern.compile("<([^<>]+)>"); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Pattern' needs unchecked conversion to conform to '@NonNull Pattern'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Utils.java (at line 467) // TODO improve ^^^^^^^^^^^^ TODO improve
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Utils.java (at line 543) return c; ^ Null type safety: The expression of type 'Class<capture#39-of ?>' needs unchecked conversion to conform to '@NonNull Class<?>'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\VisualEffect.java (at line 76) return Direction.getFacing(((Direction) raw).getDirection(l), false); // TODO allow this to not be a literal ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO allow this to not be a literal
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\WeatherType.java (at line 115) // REMIND flags? ^^^^^^^^^^^^^ REMIND flags?
TODO create a metadata table to store some properties (e.g. Skript version, Yggdrasil version) -- but what if some variables cannot be converted? move them to a different table? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO create a metadata table to store some properties (e.g. Skript version, Yggdrasil version) -- but what if some variables cannot be converted? move them to a different table?
TODO create my own database connector or find a better one ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO create my own database connector or find a better one
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 119) final SynchronizedReference db = new SynchronizedReference(null);
^^^^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 270) if (db != null) ^^ Redundant null check: The variable db cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 298) if (db != null) ^^ Redundant null check: The variable db cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 328) // TODO don't print this message when Skript loads (because scripts are loaded after variables and take some time) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO don't print this message when Skript loads (because scripts are loaded after variables and take some time)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 369) if (db == null || !db.open()) { ^^ Null comparison always yields false: The variable db cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 432) if (db != null) ^^ Redundant null check: The variable db cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 469) // REMIND get the actual maximum size from the database ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND get the actual maximum size from the database
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 525) if (closed || db.get() == null) ^^^^^^^^ Null comparison always yields false: The method get() cannot return null
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 543) if (!closed) { // Skript may have been disabled in the meantime // TODO not fixed ^^^^^^^^^^^^^^ TODO not fixed
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 549) if (closed || db.get() == null) ^^^^^^^^ Null comparison always yields false: The method get() cannot return null
TODO use a database (SQLite) instead and only load a limited amount of variables into RAM - e.g. 2 GB (configurable). If more variables are available they will be loaded when ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO use a database (SQLite) instead and only load a limited amount of variables into RAM - e.g. 2 GB (configurable). If more variables are available they will be loaded when
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 68) private final NotifyingReference changesWriter = new NotifyingReference();
^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 253) final ArrayList r = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 277) while ((cw = changesWriter.get()) == null) { ^^^^^^^^^^^^^^^^^^^^^^^^^^ Null comparison always yields false: The variable cw cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 277) while ((cw = changesWriter.get()) == null) { try { changesWriter.wait(); } catch (final InterruptedException e) { Thread.currentThread().interrupt(); } } ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Dead code
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 331) if (changesWriter.get() != null) ^^^^^^^^^^^^^^^^^^^ Redundant null check: The method get() cannot return null
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 402) Skript.error("Unable to make a final save of the database '" + databaseName + "' (no variables are lost): " + ExceptionUtils.toString(e)); // FIXME happens at random - check locks/threads ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME happens at random - check locks/threads
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 106) static List storages = new ArrayList();
^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 139) if (tvs != null) ^^^ Redundant null check: The variable tvs cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 142) break; ^^^^^^ Dead code
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 243) private final static WeakHashMap<Event, VariablesMap> localVariables = new WeakHashMap<Event, VariablesMap>(); ^^^^^^^^^^^ Redundant specification of type arguments <Event, VariablesMap>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 299) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 333) final static SynchronizedReference<Map<String, NonNullPair<Object, VariablesStorage>>> tempVars = new SynchronizedReference<Map<String, NonNullPair<Object, VariablesStorage>>>(new HashMap<String, NonNullPair<Object, VariablesStorage>>()); ^^^^^^^^^^^^^^^^^^^^^ Redundant specification of type arguments <Map<String,NonNullPair<Object,VariablesStorage>>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 355) if (tvs != null) { ^^^ Redundant null check: The variable tvs cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 367)Building 6% > :compileJava tvs.put(name, new NonNullPair<Object, VariablesStorage>(value, source)); ^^^^^^^^^^^ Redundant specification of type arguments <Object, VariablesStorage>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 372) variablesLock.writeLock().lock(); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Dead code
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 374) variables.setVariable(name, value); ^^^^ Null type safety: The expression of type 'String' needs unchecked conversion to conform to '@NonNull String'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 383) s.save(name, v != null ? v.type : null, v != null ? v.data : null); ^^^^ Null type safety: The expression of type 'String' needs unchecked conversion to conform to '@NonNull String'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 385) source.save(name, null, null); ^^^^ Null type safety: The expression of type 'String' needs unchecked conversion to conform to '@NonNull String'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\Variables.java (at line 444) final static BlockingQueue queue = new LinkedBlockingQueue();
^^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesMap.java (at line 46) if ('0' <= c1 && c1 <= '9' && '0' <= c2 && c2 <= '9') { // TODO negative numbers? what about {blah-%number%}? // '-' < '0' ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO negative numbers? what about {blah-%number%}? // '-' < '0'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesMap.java (at line 73) final HashMap<String, Object> hashMap = new HashMap<String, Object>(); ^^^^^^^ Redundant specification of type arguments <String, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesMap.java (at line 74) final TreeMap<String, Object> treeMap = new TreeMap<String, Object>(); ^^^^^^^ Redundant specification of type arguments <String, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesMap.java (at line 138) parent.put(n, current = new TreeMap<String, Object>(variableNameComparator)); ^^^^^^^ Redundant specification of type arguments <String, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesMap.java (at line 172) final TreeMap<String, Object> c = new TreeMap<String, Object>(variableNameComparator); ^^^^^^^ Redundant specification of type arguments <String, Object>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesStorage.java (at line 42) // FIXME ! large databases (>25 MB) cause the server to be unresponsive instead of loading slowly ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ FIXME ! large databases (>25 MB) cause the server to be unresponsive instead of loading slowly
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesStorage.java (at line 51) final LinkedBlockingQueue changesQueue = new LinkedBlockingQueue(QUEUE_SIZE);
^^^^^^^^^^^^^^^^^^^
Redundant specification of type arguments
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\VariablesStorage.java (at line 258) // REMIND add repetitive error and/or stop saving variables altogether? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ REMIND add repetitive error and/or stop saving variables altogether?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\BidiHashMap.java (at line 126) // TODO check how changes to the sets affect the map ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO check how changes to the sets affect the map
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\CollectionUtils.java (at line 86) public static boolean containsAny(final @Nullable T[] array, final @Nullable T... os) {
^^
Type safety: Potential heap pollution via varargs parameter os
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\CollectionUtils.java (at line 96) public static boolean containsAll(final @Nullable T[] array, final @Nullable T... os) {
^^
Type safety: Potential heap pollution via varargs parameter os
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\CollectionUtils.java (at line 301) public final static Set intersection(final @Nullable Set... sets) {
^^^^
Type safety: Potential heap pollution via varargs parameter sets
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\CollectionUtils.java (at line 322) public final static Set union(final @Nullable Set... sets) {
^^^^
Type safety: Potential heap pollution via varargs parameter sets
TODO this should actually only be an Iterator ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO this should actually only be an Iterator
// TODO make sure that it's unique ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO make sure that it's unique
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\DefaultYggdrasilInputStream.java (at line 178) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\DefaultYggdrasilOutputStream.java (at line 31) private final static Charset UTF_8 = Charset.forName("UTF-8"); ^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Charset' needs unchecked conversion to conform to '@NonNull Charset'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\DefaultYggdrasilOutputStream.java (at line 215) c = c.getComponentType(); ^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Class<capture#7-of ?>' needs unchecked conversion to conform to '@NonNull Class<?>'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\JRESerializer.java (at line 77) f.putPrimitive("mostSigBits", Long.valueOf(((UUID)o).getMostSignificantBits())); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Long' needs unchecked conversion to conform to '@NonNull Object'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\JRESerializer.java (at line 78) f.putPrimitive("leastSigBits", Long.valueOf(((UUID)o).getLeastSignificantBits())); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Null type safety: The expression of type 'Long' needs unchecked conversion to conform to '@NonNull Object'
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\JRESerializer.java (at line 135) return (E) new UUID(((Long)fields.getPrimitive("mostSigBits", Long.TYPE)).longValue(), ((Long)fields.getPrimitive("leastSigBits", Long.TYPE)).longValue()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Unnecessary cast from Long to Long
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\JRESerializer.java (at line 135) return (E) new UUID(((Long)fields.getPrimitive("mostSigBits", Long.TYPE)).longValue(), ((Long)fields.getPrimitive("leastSigBits", Long.TYPE)).longValue()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Unnecessary cast from Long to Long
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\PseudoEnum.java (at line 57) @SuppressWarnings({"unchecked", "null"}) ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\PseudoEnum.java (at line 60) info = (Info) getInfo(getClass());
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Unnecessary cast from PseudoEnum.Info to PseudoEnum.Info
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\PseudoEnum.java (at line 139) @SuppressWarnings({"unchecked", "null"}) ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\PseudoEnum.java (at line 276) Info info = (Info) infos.get(getDeclaringClass(c));
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Type safety: Unchecked cast from PseudoEnum.Info<capture#17-of ?> to PseudoEnum.Info
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\xml\YggXMLInputStream.java (at line 80) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\xml\YggXMLInputStream.java (at line 137) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\xml\YggXMLInputStream.java (at line 176) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\xml\YggXMLInputStream.java (at line 318) // TODO error if not at EOF? ^^^^^^^^^^^^^^^^^^^^^^^^^ TODO error if not at EOF?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\xml\YggXMLOutputStream.java (at line 183) @SuppressWarnings("null") ^^^^^^ Unnecessary @SuppressWarnings("null")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\Yggdrasil.java (at line 177) if (c != null) { // TODO error if not serialisable? ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO error if not serialisable?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\Yggdrasil.java (at line 265) // TODO use field handlers/new enum handlers ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO use field handlers/new enum handlers
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\Yggdrasil.java (at line 333) if (o != null) ^ Redundant null check: The variable o cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\Yggdrasil.java (at line 335) throw new YggdrasilException("YggdrasilSerializer " + s + " returned null from deserialize(" + c + ", new Fields())"); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Dead code
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\Yggdrasil.java (at line 369) // TODO command line, e.g. convert to XML ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO command line, e.g. convert to XML
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilInputStream.java (at line 94) // // TODO create this and a handler (for Enums as well) ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO create this and a handler (for Enums as well)
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilOutputStream.java (at line 173) if (fields == null) ^^^^^^ Null comparison always yields false: The variable fields cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilOutputStream.java (at line 174) throw new YggdrasilException("The serializer of " + c + " returned null"); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Dead code
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilOutputStream.java (at line 181) if (fields == null) ^^^^^^ Null comparison always yields false: The variable fields cannot be null at this location
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilOutputStream.java (at line 182) throw new YggdrasilException("The serialize() method of " + c + " returned null"); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Dead code
graph will cause Yggdrasil to throw an exception upon serialisation as no reference to the object will be available when deserialising the object. // TODO allow this ^^^^^^^^^^^^^^^ TODO allow this
Please note that you must not change the return value of this function ever - it is not saved in the stream. // TODO clarify ^^^^^^^^^^^^ TODO clarify
389 problems (9 errors, 380 warnings) :compileJava FAILED
FAILURE: Build failed with an exception.
What went wrong: Execution failed for task ':compileJava'.
Try: Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output.
BUILD FAILED
Total time: 1 mins 42.381 secs
E:\Forked Repos\Minecraft Repos\Skrypt> `