From b5f56198344a774fa2ca230c72bb3de3137a4020 Mon Sep 17 00:00:00 2001 From: Wesley Wolfe Date: Sat, 8 Feb 2014 06:05:41 -0500 Subject: Pulling all pending Bukkit-JavaDoc changes --- src/main/java/org/bukkit/DyeColor.java | 40 +++++++++++----------- src/main/java/org/bukkit/SkullType.java | 2 +- src/main/java/org/bukkit/entity/Player.java | 2 +- .../bukkit/event/entity/EntityShootBowEvent.java | 5 ++- .../player/PlayerAchievementAwardedEvent.java | 2 +- src/main/java/org/bukkit/plugin/Plugin.java | 4 +-- .../org/bukkit/plugin/PluginDescriptionFile.java | 1 + .../java/org/bukkit/plugin/java/JavaPlugin.java | 20 +++++++---- 8 files changed, 41 insertions(+), 35 deletions(-) diff --git a/src/main/java/org/bukkit/DyeColor.java b/src/main/java/org/bukkit/DyeColor.java index 2c2f3ca6..214806e6 100644 --- a/src/main/java/org/bukkit/DyeColor.java +++ b/src/main/java/org/bukkit/DyeColor.java @@ -10,67 +10,67 @@ import com.google.common.collect.ImmutableMap; public enum DyeColor { /** - * Represents white dye + * Represents white dye. */ WHITE(0x0, 0xF, Color.WHITE, Color.fromRGB(0xF0F0F0)), /** - * Represents orange dye + * Represents orange dye. */ ORANGE(0x1, 0xE, Color.fromRGB(0xD87F33), Color.fromRGB(0xEB8844)), /** - * Represents magenta dye + * Represents magenta dye. */ MAGENTA(0x2, 0xD, Color.fromRGB(0xB24CD8), Color.fromRGB(0xC354CD)), /** - * Represents light blue dye + * Represents light blue dye. */ LIGHT_BLUE(0x3, 0xC, Color.fromRGB(0x6699D8), Color.fromRGB(0x6689D3)), /** - * Represents yellow dye + * Represents yellow dye. */ YELLOW(0x4, 0xB, Color.fromRGB(0xE5E533), Color.fromRGB(0xDECF2A)), /** - * Represents lime dye + * Represents lime dye. */ LIME(0x5, 0xA, Color.fromRGB(0x7FCC19), Color.fromRGB(0x41CD34)), /** - * Represents pink dye + * Represents pink dye. */ PINK(0x6, 0x9, Color.fromRGB(0xF27FA5), Color.fromRGB(0xD88198)), /** - * Represents gray dye + * Represents gray dye. */ GRAY(0x7, 0x8, Color.fromRGB(0x4C4C4C), Color.fromRGB(0x434343)), /** - * Represents silver dye + * Represents silver dye. */ SILVER(0x8, 0x7, Color.fromRGB(0x999999), Color.fromRGB(0xABABAB)), /** - * Represents cyan dye + * Represents cyan dye. */ CYAN(0x9, 0x6, Color.fromRGB(0x4C7F99), Color.fromRGB(0x287697)), /** - * Represents purple dye + * Represents purple dye. */ PURPLE(0xA, 0x5, Color.fromRGB(0x7F3FB2), Color.fromRGB(0x7B2FBE)), /** - * Represents blue dye + * Represents blue dye. */ BLUE(0xB, 0x4, Color.fromRGB(0x334CB2), Color.fromRGB(0x253192)), /** - * Represents brown dye + * Represents brown dye. */ BROWN(0xC, 0x3, Color.fromRGB(0x664C33), Color.fromRGB(0x51301A)), /** - * Represents green dye + * Represents green dye. */ GREEN(0xD, 0x2, Color.fromRGB(0x667F33), Color.fromRGB(0x3B511A)), /** - * Represents red dye + * Represents red dye. */ RED(0xE, 0x1, Color.fromRGB(0x993333), Color.fromRGB(0xB3312C)), /** - * Represents black dye + * Represents black dye. */ BLACK(0xF, 0x0, Color.fromRGB(0x191919), Color.fromRGB(0x1E1B1B)); @@ -129,7 +129,7 @@ public enum DyeColor { } /** - * Gets the color that this dye represents + * Gets the color that this dye represents. * * @return The {@link Color} that this dye represents */ @@ -138,7 +138,7 @@ public enum DyeColor { } /** - * Gets the firework color that this dye represents + * Gets the firework color that this dye represents. * * @return The {@link Color} that this dye represents */ @@ -199,7 +199,7 @@ public enum DyeColor { } /** - * Gets the DyeColor with the given color value + * Gets the DyeColor with the given color value. * * @param color Color value to get the dye by * @return The {@link DyeColor} representing the given value, or null if @@ -210,7 +210,7 @@ public enum DyeColor { } /** - * Gets the DyeColor with the given firework color value + * Gets the DyeColor with the given firework color value. * * @param color Color value to get dye by * @return The {@link DyeColor} representing the given value, or null if diff --git a/src/main/java/org/bukkit/SkullType.java b/src/main/java/org/bukkit/SkullType.java index 5647241d..abd07c27 100644 --- a/src/main/java/org/bukkit/SkullType.java +++ b/src/main/java/org/bukkit/SkullType.java @@ -1,7 +1,7 @@ package org.bukkit; /** - * Represents the types of skulls + * Represents the different types of skulls. */ public enum SkullType { SKELETON, diff --git a/src/main/java/org/bukkit/entity/Player.java b/src/main/java/org/bukkit/entity/Player.java index 3c096f5b..44c9b5a7 100644 --- a/src/main/java/org/bukkit/entity/Player.java +++ b/src/main/java/org/bukkit/entity/Player.java @@ -755,7 +755,7 @@ public interface Player extends HumanEntity, Conversable, CommandSender, Offline /** * Sets the players current saturation level * - * @param value Exhaustion level + * @param value Saturation level */ public void setSaturation(float value); diff --git a/src/main/java/org/bukkit/event/entity/EntityShootBowEvent.java b/src/main/java/org/bukkit/event/entity/EntityShootBowEvent.java index a631e2cb..f8c91a13 100644 --- a/src/main/java/org/bukkit/event/entity/EntityShootBowEvent.java +++ b/src/main/java/org/bukkit/event/entity/EntityShootBowEvent.java @@ -30,10 +30,9 @@ public class EntityShootBowEvent extends EntityEvent implements Cancellable { } /** - * Gets the bow ItemStack used to fire the arrow; is null if the shooter - * is a skeleton + * Gets the bow ItemStack used to fire the arrow. * - * @return the bow involved in this event, or null + * @return the bow involved in this event */ public ItemStack getBow() { return bow; diff --git a/src/main/java/org/bukkit/event/player/PlayerAchievementAwardedEvent.java b/src/main/java/org/bukkit/event/player/PlayerAchievementAwardedEvent.java index 6e3fc3bb..e33fade8 100644 --- a/src/main/java/org/bukkit/event/player/PlayerAchievementAwardedEvent.java +++ b/src/main/java/org/bukkit/event/player/PlayerAchievementAwardedEvent.java @@ -20,7 +20,7 @@ public class PlayerAchievementAwardedEvent extends PlayerEvent implements Cancel /** * Gets the Achievement being awarded. - * + * * @return the achievement being awarded */ public Achievement getAchievement() { diff --git a/src/main/java/org/bukkit/plugin/Plugin.java b/src/main/java/org/bukkit/plugin/Plugin.java index af58c9de..7bdc809c 100644 --- a/src/main/java/org/bukkit/plugin/Plugin.java +++ b/src/main/java/org/bukkit/plugin/Plugin.java @@ -169,11 +169,11 @@ public interface Plugin extends TabExecutor { public ChunkGenerator getDefaultWorldGenerator(String worldName, String id); /** - * Returns the primary logger associated with this server instance. The + * Returns the plugin logger associated with this server's logger. The * returned logger automatically tags all log messages with the plugin's * name. * - * @return Logger associated with this server + * @return Logger associated with this plugin */ public Logger getLogger(); diff --git a/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java b/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java index 1b5d820f..9e6c1d94 100644 --- a/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java +++ b/src/main/java/org/bukkit/plugin/PluginDescriptionFile.java @@ -617,6 +617,7 @@ public final class PluginDescriptionFile { * # Having an empty declaration is useful for defining the description, permission, and messages from a configuration dynamically * apocalypse: * + * Note: command names may not have a colon in their name. * * @return the commands this plugin will register */ diff --git a/src/main/java/org/bukkit/plugin/java/JavaPlugin.java b/src/main/java/org/bukkit/plugin/java/JavaPlugin.java index 8fd726bd..d1c3321c 100644 --- a/src/main/java/org/bukkit/plugin/java/JavaPlugin.java +++ b/src/main/java/org/bukkit/plugin/java/JavaPlugin.java @@ -417,14 +417,15 @@ public abstract class JavaPlugin extends PluginBase { * resided in a different jar / classloader. * * @param clazz the class desired - * @return the plugin that provides and implements said class, or null - * if called from an inconsistent static initialization block + * @return the plugin that provides and implements said class * @throws IllegalArgumentException if clazz is null * @throws IllegalArgumentException if clazz does not extend {@link * JavaPlugin} - * @throws IllegalArgumentException if clazz was not provided by a - * plugin, for example, if called with + * @throws IllegalStateException if clazz was not provided by a plugin, + * for example, if called with * JavaPlugin.getPlugin(JavaPlugin.class) + * @throws IllegalStateException if called from the static initializer for + * given JavaPlugin * @throws ClassCastException if plugin that provided the class does not * extend the class */ @@ -445,9 +446,14 @@ public abstract class JavaPlugin extends PluginBase { } /** - * This method provides fast access to the plugin that has {@link - * #getProvidingPlugin(Class) provided} the given plugin class, which is - * usually the plugin that implemented it. + * This method provides fast access to the plugin that has provided the + * given class. + * + * @throws IllegalArgumentException if the class is not provided by a + * JavaPlugin + * @throws IllegalArgumentException if class is null + * @throws IllegalStateException if called from the static initializer for + * given JavaPlugin */ public static JavaPlugin getProvidingPlugin(Class clazz) { Validate.notNull(clazz, "Null class cannot have a plugin"); -- cgit v1.2.3