Some minor updates to FastInv (#2)

* Added final keyword to effectively final objects.

* Added "this" keyword to object declaration.

Author: Jitse Boonstra

src/main/java/fr/mrmicky/fastinv/FastInv.java

@@ -28,13 +28,13 @@
  */
 public class FastInv implements InventoryHolder {
 
-    private Predicate<Player> closeFilter;
+    private final Map<Integer, Consumer<InventoryClickEvent>> itemHandlers = new HashMap<>();
 
     private Set<Consumer<InventoryOpenEvent>> openHandlers;
     private Set<Consumer<InventoryCloseEvent>> closeHandlers;
     private Set<Consumer<InventoryClickEvent>> clickHandlers;
-    private Map<Integer, Consumer<InventoryClickEvent>> itemHandlers = new HashMap<>();
 
+    private Predicate<Player> closeFilter;
     private Inventory inventory;
 
     /**

src/main/java/fr/mrmicky/fastinv/ItemBuilder.java

@@ -21,16 +21,16 @@
  */
 public class ItemBuilder {
 
-    private ItemStack item;
-    private ItemMeta meta;
+    private final ItemStack item;
+    private final ItemMeta meta;
 
     public ItemBuilder(Material material) {
         this(new ItemStack(material));
     }
 
     public ItemBuilder(ItemStack item) {
         this.item = Objects.requireNonNull(item, "item");
-        meta = item.getItemMeta();
+        this.meta = item.getItemMeta();
     }
 
     public ItemBuilder type(Material material) {
@@ -107,7 +107,7 @@ public ItemBuilder addLore(String line) {
         List<String> lore = meta.getLore();
 
         if (lore == null) {
-            return lore(Collections.singletonList(line));
+            return lore(line);
         }
 
         lore.add(line);