From e6ba3ce8a6e5d817cae55a26d05eec58706ccf54 Mon Sep 17 00:00:00 2001
From: "Dylan K. Taylor" <odigiman@gmail.com>
Date: Sun, 19 Jan 2020 17:13:33 +0000
Subject: [PATCH] item: populate missing return type information

---
 src/pocketmine/item/Consumable.php  | 2 ++
 src/pocketmine/item/Durable.php     | 2 ++
 src/pocketmine/item/Item.php        | 9 +++++++++
 src/pocketmine/item/ItemFactory.php | 4 ++++
 4 files changed, 17 insertions(+)

diff --git a/src/pocketmine/item/Consumable.php b/src/pocketmine/item/Consumable.php
index 5977c37f44..97f1df158c 100644
--- a/src/pocketmine/item/Consumable.php
+++ b/src/pocketmine/item/Consumable.php
@@ -49,6 +49,8 @@ interface Consumable{
 	 * Called when this Consumable is consumed by mob, after standard resulting effects have been applied.
 	 *
 	 * @param Living $consumer
+	 *
+	 * @return void
 	 */
 	public function onConsume(Living $consumer);
 }
diff --git a/src/pocketmine/item/Durable.php b/src/pocketmine/item/Durable.php
index 07fa510024..6b548e7a79 100644
--- a/src/pocketmine/item/Durable.php
+++ b/src/pocketmine/item/Durable.php
@@ -42,6 +42,8 @@ abstract class Durable extends Item{
 	 * Sets whether the item will take damage when used.
 	 *
 	 * @param bool $value
+	 *
+	 * @return void
 	 */
 	public function setUnbreakable(bool $value = true){
 		$this->setNamedTagEntry(new ByteTag("Unbreakable", $value ? 1 : 0));
diff --git a/src/pocketmine/item/Item.php b/src/pocketmine/item/Item.php
index d1e21183a6..ae26592d57 100644
--- a/src/pocketmine/item/Item.php
+++ b/src/pocketmine/item/Item.php
@@ -123,6 +123,9 @@ class Item implements ItemIds, \JsonSerializable{
 	/** @var Item[] */
 	private static $creative = [];
 
+	/**
+	 * @return void
+	 */
 	public static function initCreativeItems(){
 		self::clearCreativeItems();
 
@@ -140,6 +143,8 @@ class Item implements ItemIds, \JsonSerializable{
 	/**
 	 * Removes all previously added items from the creative menu.
 	 * Note: Players who are already online when this is called will not see this change.
+	 *
+	 * @return void
 	 */
 	public static function clearCreativeItems(){
 		Item::$creative = [];
@@ -154,6 +159,8 @@ class Item implements ItemIds, \JsonSerializable{
 	 * Note: Players who are already online when this is called will not see this change.
 	 *
 	 * @param Item $item
+	 *
+	 * @return void
 	 */
 	public static function addCreativeItem(Item $item){
 		Item::$creative[] = clone $item;
@@ -164,6 +171,8 @@ class Item implements ItemIds, \JsonSerializable{
 	 * Note: Players who are already online when this is called will not see this change.
 	 *
 	 * @param Item $item
+	 *
+	 * @return void
 	 */
 	public static function removeCreativeItem(Item $item){
 		$index = self::getCreativeItemIndex($item);
diff --git a/src/pocketmine/item/ItemFactory.php b/src/pocketmine/item/ItemFactory.php
index 3c51f3173f..37c576b567 100644
--- a/src/pocketmine/item/ItemFactory.php
+++ b/src/pocketmine/item/ItemFactory.php
@@ -46,6 +46,9 @@ class ItemFactory{
 	/** @var \SplFixedArray */
 	private static $list = null;
 
+	/**
+	 * @return void
+	 */
 	public static function init(){
 		self::$list = new \SplFixedArray(65536);
 
@@ -285,6 +288,7 @@ class ItemFactory{
 	 * @param Item $item
 	 * @param bool $override
 	 *
+	 * @return void
 	 * @throws \RuntimeException if something attempted to override an already-registered item without specifying the
 	 * $override parameter.
 	 */