Skript is a Spigot plugin which allows server admins to customize their server easily, but without the hassle of programming a plugin or asking/paying someone to program a plugin for them.
After compiling latest from master branch, this is my log
`
E:>cmd.exe
Microsoft Windows [Version 10.0.16251.0]
(c) 2017 Microsoft Corporation. All rights reserved.
E:>cd forked repos/minecraft repos/skrypt
E:\Forked Repos\Minecraft Repos\Skrypt>mvn install -B
[INFO] Scanning for projects...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 0.258 s
[INFO] Finished at: 2017-08-01T16:49:54-07:00
[INFO] Final Memory: 6M/243M
[INFO] ------------------------------------------------------------------------
[ERROR] The goal you specified requires a project to execute but there is no POM in this directory (E:\Forked Repos\Minecraft Repos\Skrypt). Please verify you invoked Maven from the correct directory. -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MissingProjectException
E:\Forked Repos\Minecraft Repos\Skrypt>gradlew build
Starting a Gradle Daemon (subsequent builds will be faster)
:compileJava
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\Aliases.java (at line 58)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\aliases\ItemType.java (at line 648)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\bukkitutil\PlayerUtils.java (at line 42)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\classes\data\DefaultFunctions.java (at line 44)
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 of s 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 153)
// 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")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\config\SectionNode.java (at line 384)
REMIND breaks saving - separator argument can be different from config.sepator
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
REMIND breaks saving - separator argument can be different from config.sepator
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\doc\Documentation.java (at line 53)
TODO list special expressions for events and event values
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO list special expressions for events and event values
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\doc\Documentation.java (at line 54)
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")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprAmount.java (at line 38)
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")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprEventCancelled.java (at line 37)
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")
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLocationAt.java (at line 39)
FIXME doesn't parse - update documentation when fixed
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
FIXME doesn't parse - update documentation when fixed
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLoopValue.java (at line 56)
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));
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\expressions\ExprLore.java (at line 53)
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 0% > :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?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Conditional.java (at line 30)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\Expression.java (at line 150)
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 186)
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 190)
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 73)
// 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<?>>
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\ParseContext.java (at line 33)
TODO? replace with {@link #DUMMY} + {@link SkriptParser#PARSE_LITERALS}
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO? replace with {@link #DUMMY} + {@link SkriptParser#PARSE_LITERALS}
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 401)
// TODO move them elsewhere, this method needs to be as fast as possible
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO move them elsewhere, this method needs to be as fast as possible
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 489)
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 489)
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>ompileJava
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 489)
e = (Expression<?>) parse(expr, (Iterator) Skript.getExpressions(types), null);
^^^^^^^^
Iterator is a raw type. References to generic type Iterator should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 948)
return new LiteralList(ls, Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Type safety: The constructor LiteralList(Literal[], Class, boolean) belongs to the raw type LiteralList. References to generic type LiteralList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 948)
return new LiteralList(ls, Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^
LiteralList is a raw type. References to generic type LiteralList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 952)
return new ExpressionList(es, Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Type safety: The constructor ExpressionList(Expression[], Class, boolean) belongs to the raw type ExpressionList. References to generic type ExpressionList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 952)
return new ExpressionList(es, Utils.getSuperType(exprRetTypes), !and.isFalse());
^^^^^^^^^^^^^^
ExpressionList is a raw type. References to generic type ExpressionList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SyntaxElement.java (at line 30)
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\Testable.java (at line 27)
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) {
^^
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\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 273)
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 307)
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 720)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\BlockUtils.java (at line 35)
TODO !Update with every version [blocks] - also update aliases-.sk
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO !Update with every version [blocks] - also update aliases-.sk
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\BlockUtils.java (at line 103)
@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?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Slot.java (at line 30)
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 slotava
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\util\Utils.java (at line 318)
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?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 51)
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?
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\DatabaseStorage.java (at line 52)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\variables\FlatFileStorage.java (at line 54)
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)
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\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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\util\coll\iterator\EnumerationIterable.java (at line 28)
TODO this should actually only be an Iterator
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TODO this should actually only be an Iterator
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\ClassResolver.java (at line 38)
// 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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilSerializer.java (at line 52)
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
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\yggdrasil\YggdrasilSerializer.java (at line 54)
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
bensku
commented
7 years ago
After compiling latest from master branch, this is my log
`
E:>cmd.exe Microsoft Windows [Version 10.0.16251.0] (c) 2017 Microsoft Corporation. All rights reserved.
E:>cd forked repos/minecraft repos/skrypt
E:\Forked Repos\Minecraft Repos\Skrypt>mvn install -B [INFO] Scanning for projects... [INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 0.258 s [INFO] Finished at: 2017-08-01T16:49:54-07:00 [INFO] Final Memory: 6M/243M [INFO] ------------------------------------------------------------------------ [ERROR] The goal you specified requires a project to execute but there is no POM in this directory (E:\Forked Repos\Minecraft Repos\Skrypt). Please verify you invoked Maven from the correct directory. -> [Help 1] [ERROR] [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch. [ERROR] Re-run Maven using the -X switch to enable full debug logging. [ERROR] [ERROR] For more information about the errors and possible solutions, please read the following articles: [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MissingProjectException
E:\Forked Repos\Minecraft Repos\Skrypt>gradlew build Starting a Gradle Daemon (subsequent builds will be faster) :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 153) // 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 0% > :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 186) 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 190) 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 73) // 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}
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 401) // TODO move them elsewhere, this method needs to be as fast as possible ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TODO move them elsewhere, this method needs to be as fast as possible
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 489) 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 489) 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>ompileJava
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 489) e = (Expression<?>) parse(expr, (Iterator) Skript.getExpressions(types), null); ^^^^^^^^ Iterator is a raw type. References to generic type Iterator should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 948) return new LiteralList(ls, Utils.getSuperType(exprRetTypes), !and.isFalse()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: The constructor LiteralList(Literal[], Class, boolean) belongs to the raw type LiteralList. References to generic type LiteralList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 948) return new LiteralList(ls, Utils.getSuperType(exprRetTypes), !and.isFalse()); ^^^^^^^^^^^ LiteralList is a raw type. References to generic type LiteralList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 952) return new ExpressionList(es, Utils.getSuperType(exprRetTypes), !and.isFalse()); ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ Type safety: The constructor ExpressionList(Expression[], Class, boolean) belongs to the raw type ExpressionList. References to generic type ExpressionList should be parameterized
WARNING in E:\Forked Repos\Minecraft Repos\Skrypt\src\main\java\ch\njol\skript\lang\SkriptParser.java (at line 952) return new ExpressionList(es, Utils.getSuperType(exprRetTypes), !and.isFalse()); ^^^^^^^^^^^^^^ ExpressionList is a raw type. References to generic type ExpressionList should be parameterized
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) {
^^
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\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 273) 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 307) 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 720) 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 slotava ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ 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) 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
374 problems (374 warnings) :processResources :classes :jar :assemble :licenseMain Unknown file extension: src\main\resources\lang\english.lang Unknown file extension: src\main\resources\features.sk Unknown file extension: src\main\resources\lang\german.lang Unknown file extension: src\main\resources\scripts-- Files prefixed with a hyphen are disabled! -- Unknown file extension: src\main\resources\scripts\block head.sk Unknown file extension: src\main\resources\scripts\command with cooldown.sk Unknown file extension: src\main\resources\scripts\disable weather.sk Unknown file extension: src\main\resources\scripts\drop fixes.sk Unknown file extension: src\main\resources\scripts\equip anything.sk Unknown file extension: src\main\resources\scripts\eternal day.sk Unknown file extension: src\main\resources\scripts\furnace automatisation.sk Unknown file extension: src\main\resources\scripts\homes.sk Unknown file extension: src\main\resources\scripts\item command.sk Unknown file extension: src\main\resources\scripts\kill counter.sk Unknown file extension: src\main\resources\scripts\nerf endermen.sk Unknown file extension: src\main\resources\scripts\plant with hoe.sk Unknown file extension: src\main\resources\scripts\simple god mode.sk Unknown file extension: src\main\resources\scripts\simple motd.sk Unknown file extension: src\main\resources\scripts\teleport with compass.sk Unknown file extension: src\main\resources\version Unknown file extension: src\main\resources\aliases-german.sk Unknown file extension: src\main\resources\aliases-english.sk Unknown file extension: src\main\resources\config.sk :licenseMain UP-TO-DATE :licenseTest UP-TO-DATE :license UP-TO-DATE :compileTestJava Download https://jcenter.bintray.com/org/easymock/easymock/3.4/easymock-3.4.jar Download https://jcenter.bintray.com/org/objenesis/objenesis/2.2/objenesis-2.2.jar Note: E:\Forked Repos\Minecraft Repos\Skrypt\src\test\java\ch\njol\skript\registrations\ClassesTest.java uses or overrides a deprecated API. Note: Recompile with -Xlint:deprecation for details. :processTestResources NO-SOURCE :testClasses :test
ch.njol.skript.aliases.AliasesTest > testNames FAILED org.junit.ComparisonFailure at AliasesTest.java:95
ch.njol.skript.localization.NounTest > testNormalizePluralMarkers FAILED org.junit.ComparisonFailure at NounTest.java:65
ch.njol.skript.localization.NounTest > testGetPlural FAILED org.junit.ComparisonFailure at NounTest.java:47
10 tests completed, 3 failed, 1 skipped :test FAILED
FAILURE: Build failed with an exception.
What went wrong: Execution failed for task ':test'.
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: 2 mins 0.017 secs
E:\Forked Repos\Minecraft Repos\Skrypt> `