BETA: JavaDocs for some classes.

L2J_Server_BETA/java/com/l2jserver/Config.java

@@ -4292,8 +4292,8 @@ public final class Config
 	}
 	
 	/**
-	 * @param line
-	 * @return
+	 * @param line the string line to parse
+	 * @return a parsed float map
 	 */
 	private static TIntFloatHashMap parseConfigLine(String line)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/GameTimeController.java

@@ -27,8 +27,8 @@ import com.l2jserver.gameserver.instancemanager.DayNightSpawnManager;
 import com.l2jserver.gameserver.model.actor.L2Character;
 
 /**
- * Removed TimerThread watcher [DrHouse]
- *
+ * Removed TimerThread watcher [DrHouse]<br>
+ * One in-game day is 240 real minutes.
  * @version $Date: 2010/02/02 22:43:00 $
  */
 public class GameTimeController
@@ -49,14 +49,17 @@ public class GameTimeController
 	protected static TimerThread _timer;
 	
 	/**
-	 * one ingame day is 240 real minutes
-	 * @return 
+	 * Gets the single instance of GameTimeController.
+	 * @return single instance of GameTimeController
 	 */
 	public static GameTimeController getInstance()
 	{
 		return SingletonHolder._instance;
 	}
 	
+	/**
+	 * Instantiates a new game time controller.
+	 */
 	protected GameTimeController()
 	{
 		_gameStartTime = System.currentTimeMillis() - 3600000; // offset so that the server starts a day begin
@@ -69,29 +72,37 @@ public class GameTimeController
 		
 	}
 	
+	/**
+	 * Checks if is now night.
+	 * @return true, if is now night
+	 */
 	public boolean isNowNight()
 	{
 		return _isNight;
 	}
 	
+	/**
+	 * Gets the game time.
+	 * @return the game time
+	 */
 	public int getGameTime()
 	{
 		return (_gameTicks / (TICKS_PER_SECOND * 10));
 	}
 	
+	/**
+	 * Gets the game ticks.
+	 * @return the game ticks
+	 */
 	public static int getGameTicks()
 	{
 		return _gameTicks;
 	}
 	
 	/**
-	 * Add a L2Character to movingObjects of GameTimeController.<BR><BR>
-	 *
-	 * <B><U> Concept</U> :</B><BR><BR>
-	 * All L2Character in movement are identified in <B>movingObjects</B> of GameTimeController.<BR><BR>
-	 *
-	 * @param cha The L2Character to add to movingObjects of GameTimeController
-	 *
+	 * Add a L2Character to movingObjects of GameTimeController.<br>
+	 * All characters in movement are identified in <b>movingObjects</b> of GameTimeController.
+	 * @param cha the character to add to movingObjects of GameTimeController
 	 */
 	public void registerMovingObject(L2Character cha)
 	{
@@ -110,16 +121,12 @@ public class GameTimeController
 	}
 	
 	/**
-	 * Move all L2Characters contained in movingObjects of GameTimeController.<BR><BR>
-	 *
-	 * <B><U> Concept</U> :</B><BR><BR>
-	 * All L2Character in movement are identified in <B>movingObjects</B> of GameTimeController.<BR><BR>
-	 *
-	 * <B><U> Actions</U> :</B><BR><BR>
+	 * Move all characters contained in movingObjects of GameTimeController.<br>
+	 * All characters in movement are identified in <b>movingObjects</b> of GameTimeController.<br>
+	 * <b><u> Actions</u> :</b><br>
 	 * <li>Update the position of each L2Character </li>
 	 * <li>If movement is finished, the L2Character is removed from movingObjects </li>
-	 * <li>Create a task to update the _knownObject and _knowPlayers of each L2Character that finished its movement and of their already known L2Object then notify AI with EVT_ARRIVED </li><BR><BR>
-	 *
+	 * <li>Create a task to update the _knownObject and _knowPlayers of each L2Character that finished its movement and of their already known L2Object then notify AI with EVT_ARRIVED </li>
 	 */
 	protected void moveObjects()
 	{
@@ -150,6 +157,9 @@ public class GameTimeController
 		}
 	}
 	
+	/**
+	 * Stop timer.
+	 */
 	public void stopTimer()
 	{
 		_interruptRequest = true;
@@ -158,6 +168,9 @@ public class GameTimeController
 	
 	class TimerThread extends Thread
 	{
+		/**
+		 * Instantiates a new timer thread.
+		 */
 		public TimerThread()
 		{
 			super("GameTimeController");
@@ -210,12 +223,16 @@ public class GameTimeController
 	}
 	
 	/**
-	 * Update the _knownObject and _knowPlayers of each L2Character that finished its movement and of their already known L2Object then notify AI with EVT_ARRIVED.<BR><BR>
+	 * Update the _knownObject and _knowPlayers of each L2Character that finished its movement and of their already known L2Object then notify AI with EVT_ARRIVED.
 	 */
 	private static class MovingObjectArrived implements Runnable
 	{
 		private final L2Character _ended;
 		
+		/**
+		 * Instantiates a new moving object arrived.
+		 * @param ended the ended
+		 */
 		MovingObjectArrived(L2Character ended)
 		{
 			_ended = ended;
@@ -252,8 +269,9 @@ public class GameTimeController
 			tempIsNight = (h < 6);
 			
 			if (tempIsNight != _isNight)
-			{ // If diff day/night state
-				_isNight = tempIsNight; // Set current day/night varible to value of temp varible
+			{
+				// If diff day/night state
+				_isNight = tempIsNight; // Set current day/night variable to value of temp variable
 				DayNightSpawnManager.getInstance().notifyChangeMode();
 			}
 		}

L2J_Server_BETA/java/com/l2jserver/gameserver/GeoData.java

@@ -24,7 +24,7 @@ import com.l2jserver.gameserver.model.actor.instance.L2PcInstance;
 import com.l2jserver.gameserver.util.Point3D;
 
 /**
- * @author  -Nemesiss-
+ * @author -Nemesiss-
  */
 public class GeoData
 {
@@ -32,8 +32,13 @@ public class GeoData
 	
 	protected GeoData()
 	{
+		//
 	}
 	
+	/**
+	 * Instantiates a new geodata.
+	 * @param disabled the disabled
+	 */
 	protected GeoData(final boolean disabled)
 	{
 		if (disabled)
@@ -42,16 +47,20 @@ public class GeoData
 		}
 	}
 	
+	/**
+	 * Gets the single instance of GeoData.
+	 * @return single instance of GeoData
+	 */
 	public static GeoData getInstance()
 	{
 		return SingletonHolder._instance;
 	}
 	
-	// Public Methods
 	/**
-	 * @param x
-	 * @param y
-	 * @return Geo Block Type
+	 * Gets the type.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @return the geodata block type
 	 */
 	public short getType(int x, int y)
 	{
@@ -59,10 +68,11 @@ public class GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
-	 * @return Nearles Z
+	 * Gets the height.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @return the height
 	 */
 	public short getHeight(int x, int y, int z)
 	{
@@ -70,12 +80,13 @@ public class GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param zmin
-	 * @param zmax
-	 * @param spawn
-	 * @return
+	 * Gets the spawn height.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param zmin the minimum z coordinate
+	 * @param zmax the the maximum z coordinate
+	 * @param spawn the spawn
+	 * @return the spawn height
 	 */
 	public short getSpawnHeight(int x, int y, int zmin, int zmax, L2Spawn spawn)
 	{
@@ -83,9 +94,10 @@ public class GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @return
+	 * Geodata position.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @return the string
 	 */
 	public String geoPosition(int x, int y)
 	{
@@ -93,35 +105,53 @@ public class GeoData
 	}
 	
 	/**
-	 * @param cha
-	 * @param target
-	 * @return True if cha can see target (LOS)
+	 * Can see target.
+	 * @param cha the character
+	 * @param target the target
+	 * @return {@code true} if the character can see the target (LOS), {@code false} otherwise
 	 */
 	public boolean canSeeTarget(L2Object cha, L2Object target)
 	{
-		//If geo is off do simple check :]
-		//Don't allow casting on players on different dungeon lvls etc
+		//If geodata is off do simple check :]
+		//Don't allow casting on players on different dungeon levels etc
 		return (Math.abs(target.getZ() - cha.getZ()) < 1000);
 	}
 	
+	/**
+	 * Can see target.
+	 * @param cha the character
+	 * @param worldPosition the world position
+	 * @return {@code true} if the character can see the target at the given world position, {@code false} otherwise
+	 */
 	public boolean canSeeTarget(L2Object cha, Point3D worldPosition)
 	{
-		//If geo is off do simple check :]
-		//Don't allow casting on players on different dungeon lvls etc
+		// If geodata is off do simple check :]
+		// Don't allow casting on players on different dungeon levels etc
 		return Math.abs(worldPosition.getZ() - cha.getZ()) < 1000;
 	}
 	
+	/**
+	 * Can see target.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @return {@code true} if there is line of sight between the given coordinate sets, {@code false} otherwise
+	 */
 	public boolean canSeeTarget(int x, int y, int z, int tx, int ty, int tz)
 	{
-		// If geo is off do simple check :]
-		// Don't allow casting on players on different dungeon lvls etc
+		// If geodata is off do simple check :]
+		// Don't allow casting on players on different dungeon levels etc
 		return (Math.abs(z - tz) < 1000);
 	}
 	
 	/**
-	 * @param gm
-	 * @param target
-	 * @return True if cha can see target (LOS) and send useful info to PC
+	 * Can see target debug.
+	 * @param gm the Game Master
+	 * @param target the target
+	 * @return {@code true} if the Game Master can see the target target (LOS), {@code false} otherwise
 	 */
 	public boolean canSeeTargetDebug(L2PcInstance gm, L2Object target)
 	{
@@ -129,60 +159,98 @@ public class GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
-	 * @return Geo NSWE (0-15)
+	 * Gets the NSWE.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @return the geodata NSWE (0-15)
 	 */
 	public short getNSWE(int x, int y, int z)
 	{
 		return 15;
 	}
 	
+	/**
+	 * Gets the height and NSWE.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @return the height and NSWE
+	 */
 	public short getHeightAndNSWE(int x, int y, int z)
 	{
 		return (short)((z << 1) | 15);
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
-	 * @param tx
-	 * @param ty
-	 * @param tz
-	 * @param instanceId
-	 * @return Last Location (x,y,z) where player can walk - just before wall
+	 * Move check.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @param instanceId the instance id
+	 * @return the last Location (x,y,z) where player can walk - just before wall
 	 */
 	public Location moveCheck(int x, int y, int z, int tx, int ty, int tz, int instanceId)
 	{
 		return new Location(tx, ty, tz);
 	}
 	
+	/**
+	 * Can move from to target.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @param instanceId the instance id
+	 * @return {@code true} if the character at x,y,z can move to tx,ty,tz, {@code false} otherwise
+	 */
 	public boolean canMoveFromToTarget(int x, int y, int z, int tx, int ty, int tz, int instanceId)
 	{
 		return true;
 	}
 	
 	/**
-	 * @param gm
-	 * @param comment
+	 * Adds the geodata data bug.
+	 * @param gm the Game Master
+	 * @param comment the comment
 	 */
 	public void addGeoDataBug(L2PcInstance gm, String comment)
 	{
 		//Do Nothing
 	}
 	
+	/**
+	 * Unload geodata.
+	 * @param rx the region x coordinate
+	 * @param ry the region y coordinate
+	 */
 	public static void unloadGeodata(byte rx, byte ry)
 	{
 		
 	}
 	
+	/**
+	 * Load a geodata file.
+	 * @param rx the region x coordinate
+	 * @param ry the region y coordinate
+	 * @return {@code true} if successful, {@code false} otherwise
+	 */
 	public static boolean loadGeodataFile(byte rx, byte ry)
 	{
 		return false;
 	}
 	
+	/**
+	 * Checks for geodata.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @return {@code true} if there is geodata for the given coordinates, {@code false} otherwise
+	 */
 	public boolean hasGeo(int x, int y)
 	{
 		return false;

L2J_Server_BETA/java/com/l2jserver/gameserver/GeoEngine.java

@@ -59,17 +59,23 @@ public class GeoEngine extends GeoData
 	private static TShortObjectHashMap<IntBuffer> _geodataIndex = new TShortObjectHashMap<>();
 	private static BufferedOutputStream _geoBugsOut;
 	
+	/**
+	 * Gets the single instance of GeoEngine.
+	 * @return single instance of GeoEngine
+	 */
 	public static GeoEngine getInstance()
 	{
 		return SingletonHolder._instance;
 	}
 	
+	/**
+	 * Instantiates a new geo engine.
+	 */
 	protected GeoEngine()
 	{
 		nInitGeodata();
 	}
 	
-	//Public Methods
 	@Override
 	public short getType(int x, int y)
 	{
@@ -220,6 +226,16 @@ public class GeoEngine extends GeoData
 		return false;
 	}
 	
+	/**
+	 * Can see.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @return {@code true} if there is line of sight between the given coordinate sets, {@code false} otherwise
+	 */
 	private static boolean canSee(int x, int y, double z, int tx, int ty, int tz)
 	{
 		int dx = (tx - x);
@@ -336,13 +352,22 @@ public class GeoEngine extends GeoData
 	}
 	
 	/*
-	 * Debug function for checking if there's a line of sight between
-	 * two coordinates.
-	 *
-	 * Creates points for line of sight check (x,y,z towards target) and
-	 * in each point, layer and movement checks are made with NLOS function.
-	 *
-	 * Coordinates here are geodata x,y but z coordinate is world coordinate
+	 * Debug function for checking if there's a line of sight between two coordinates.<br>
+	 * Creates points for line of sight check (x,y,z towards target) and<br>
+	 * in each point, layer and movement checks are made with NLOS function.<br>
+	 * Coordinates here are geodata x,y but z coordinate is world coordinate<br>
+	 */
+	
+	/**
+	 * Can see debug.
+	 * @param gm the Game Master
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @return {@code true} if the Game Master can see the target target (LOS), {@code false} otherwise
 	 */
 	private static boolean canSeeDebug(L2PcInstance gm, int x, int y, double z, int tx, int ty, int tz)
 	{
@@ -463,16 +488,16 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 *  MoveCheck
-	 * @param startpoint 
-	 * @param destiny 
-	 * @param x 
-	 * @param y 
-	 * @param z 
-	 * @param tx 
-	 * @param ty 
-	 * @param tz 
-	 * @return 
+	 * Move check.
+	 * @param startpoint the start point location
+	 * @param destiny the destination location
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @return the location modified if it didn't pass the geodata checks
 	 */
 	private static Location moveCheck(Location startpoint, Location destiny, int x, int y, double z, int tx, int ty, int tz)
 	{
@@ -599,6 +624,11 @@ public class GeoEngine extends GeoData
 		return new Location(destiny.getX(), destiny.getY(), (int) z);
 	}
 	
+	/**
+	 * Sign of parameter x.
+	 * @param x the value to verify
+	 * @return +1 if x is positive, -1 otherwise
+	 */
 	private static byte sign(int x)
 	{
 		if (x >= 0)
@@ -608,6 +638,9 @@ public class GeoEngine extends GeoData
 	}
 	
 	//GeoEngine
+	/**
+	 * Initialize the geodata.
+	 */
 	private static void nInitGeodata()
 	{
 		final File file = new File("./data/geodata/geo_index.txt");
@@ -646,6 +679,11 @@ public class GeoEngine extends GeoData
 		}
 	}
 	
+	/**
+	 * Unload geodata.
+	 * @param rx the region x coordinate
+	 * @param ry the region y coordinate
+	 */
 	public static void unloadGeodata(byte rx, byte ry)
 	{
 		short regionoffset = (short) ((rx << 5) + ry);
@@ -653,6 +691,12 @@ public class GeoEngine extends GeoData
 		_geodata.remove(regionoffset);
 	}
 	
+	/**
+	 * Load geodata file.
+	 * @param rx the region x coordinate
+	 * @param ry the region y coordinate
+	 * @return {@code true} if successful, {@code false} otherwise
+	 */
 	public static boolean loadGeodataFile(byte rx, byte ry)
 	{
 		if (rx < Config.WORLD_X_MIN || rx > Config.WORLD_X_MAX || ry < Config.WORLD_Y_MIN || ry > Config.WORLD_Y_MAX)
@@ -721,9 +765,10 @@ public class GeoEngine extends GeoData
 	
 	//Geodata Methods
 	/**
-	 * @param x
-	 * @param y
-	 * @return Region Offset
+	 * Gets the region offset.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @return the region offset
 	 */
 	private static short getRegionOffset(int x, int y)
 	{
@@ -733,7 +778,8 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param geo_pos 
+	 * Gets the block.
+	 * @param geo_pos the geo-position
 	 * @return Block Index: 0-255
 	 */
 	private static int getBlock(int geo_pos)
@@ -742,7 +788,8 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param geo_pos 
+	 * Gets the cell.
+	 * @param geo_pos the geo-position
 	 * @return Cell Index: 0-7
 	 */
 	private static int getCell(int geo_pos)
@@ -753,8 +800,9 @@ public class GeoEngine extends GeoData
 	//Geodata Functions
 	
 	/**
-	 * @param x
-	 * @param y
+	 * Gets the type.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
 	 * @return Type of geo_block: 0-2
 	 */
 	private static short nGetType(int x, int y)
@@ -782,10 +830,11 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param geox 
-	 * @param geoy 
-	 * @param z
-	 * @return Nearest Z
+	 * Gets the height.
+	 * @param geox the geo-position x coordinate
+	 * @param geoy the geo-position y coordinate
+	 * @param z the z coordinate
+	 * @return the nearest z
 	 */
 	private static short nGetHeight(int geox, int geoy, int z)
 	{
@@ -860,10 +909,11 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param geox 
-	 * @param geoy 
-	 * @param z
-	 * @return One layer higher Z than parameter Z
+	 * Get upper height.
+	 * @param geox the geo-position x coordinate
+	 * @param geoy the geo-position y coordinate
+	 * @param z the z coordinate
+	 * @return One layer higher z than parameter z
 	 */
 	private static short nGetUpperHeight(int geox, int geoy, int z)
 	{
@@ -939,12 +989,13 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param geox 
-	 * @param geoy 
-	 * @param zmin
-	 * @param zmax
-	 * @param spawn 
-	 * @return Z between zmin and zmax
+	 * Get spawn height.
+	 * @param geox the geo-position x coordinate
+	 * @param geoy the geo-position y coordinate
+	 * @param zmin the minimum z coordinate
+	 * @param zmax the maximum z coordinate
+	 * @param spawn the spawn
+	 * @return a valid height between zmin and zmax
 	 */
 	private static short nGetSpawnHeight(int geox, int geoy, int zmin, int zmax, L2Spawn spawn)
 	{
@@ -1034,13 +1085,14 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
-	 * @param tx
-	 * @param ty
-	 * @param tz
-	 * @return True if char can move to (tx,ty,tz)
+	 * Can move next.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @param tz the target's z coordinate
+	 * @return {@code true} if character can move to (tx,ty,tz), {@code false} otherwise
 	 */
 	private static double nCanMoveNext(int x, int y, int z, int tx, int ty, int tz)
 	{
@@ -1130,15 +1182,16 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
-	 * @param inc_x
-	 * @param inc_y
-	 * @param inc_z 
-	 * @param tz
-	 * @param debug 
-	 * @return True if Char can see target
+	 * Line of sight.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
+	 * @param inc_x the inc_x
+	 * @param inc_y the inc_y
+	 * @param inc_z the inc_z
+	 * @param tz the tz
+	 * @param debug if {@code true} there will be debug logs
+	 * @return {@code true} if the character can see the target
 	 */
 	private static boolean nLOS(int x, int y, int z, int inc_x, int inc_y, double inc_z, int tz, boolean debug)
 	{
@@ -1245,7 +1298,7 @@ public class GeoEngine extends GeoData
 				_log.warning("z:" + z + " x: " + cellX + " y:" + cellY + " la " + layers + " lo:" + lowerHeight + " up:" + upperHeight);
 			// Check if LOS goes under a layer/floor
 			// clearly under layer but not too much under
-			// lowerheight here only for geodata bug checking, layers very close? maybe could be removed
+			// lower height here only for geodata bug checking, layers very close? maybe could be removed
 			if ((z - upperHeight) < -10 && (z - upperHeight) > inc_z - 20 && (z - lowerHeight) > 40)
 			{
 				if (debug)
@@ -1279,9 +1332,10 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
+	 * Get the NSWE for the given coordinates.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
 	 * @return NSWE: 0-15
 	 */
 	private static short nGetNSWE(int x, int y, int z)
@@ -1362,9 +1416,10 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
-	 * @param z
+	 * Gets the height and NSWE.
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param z the z coordinate
 	 * @return array [0] - height, [1] - NSWE
 	 */
 	@Override
@@ -1443,12 +1498,13 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param NSWE
-	 * @param x
-	 * @param y
-	 * @param tx
-	 * @param ty
-	 * @return True if NSWE dont block given direction
+	 * Check NSWE.
+	 * @param NSWE the NSWE
+	 * @param x the x coordinate
+	 * @param y the y coordinate
+	 * @param tx the target's x coordinate
+	 * @param ty the target's y coordinate
+	 * @return {@code true} if NSWE don't block given direction, {@code false} otherwise
 	 */
 	private static boolean checkNSWE(short NSWE, int x, int y, int tx, int ty)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/LoginServerThread.java

@@ -96,7 +96,7 @@ public class LoginServerThread extends Thread
 	 * "_;v.]05-31!|+-%xT!^[$\00"<br>
 	 * <br>
 	 * and then after handshake, with a new key sent by<br>
-	 * loginserver during the handshake. This new key is stored<br>
+	 * login server during the handshake. This new key is stored<br>
 	 * in {@link #_blowfishKey}
 	 */
 	private NewCrypt _blowfish;
@@ -114,6 +114,9 @@ public class LoginServerThread extends Thread
 	private final String[] _subnets;
 	private final String[] _hosts;
 	
+	/**
+	 * Instantiates a new login server thread.
+	 */
 	protected LoginServerThread()
 	{
 		super("LoginServerThread");
@@ -139,6 +142,10 @@ public class LoginServerThread extends Thread
 		_maxPlayer = Config.MAXIMUM_ONLINE_USERS;
 	}
 	
+	/**
+	 * Gets the single instance of LoginServerThread.
+	 * @return single instance of LoginServerThread
+	 */
 	public static LoginServerThread getInstance()
 	{
 		return SingletonHolder._instance;
@@ -240,7 +247,7 @@ public class LoginServerThread extends Thread
 							sendPacket(bfk);
 							if (Config.DEBUG)
 								_log.info("Sent new blowfish key");
-							//now, only accept paket with the new encryption
+							//now, only accept packet with the new encryption
 							_blowfish = new NewCrypt(_blowfishKey);
 							if (Config.DEBUG)
 								_log.info("Changed blowfish key");
@@ -392,6 +399,12 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Adds the waiting client and send request.
+	 * @param acc the account
+	 * @param client the game client
+	 * @param key the session key
+	 */
 	public void addWaitingClientAndSendRequest(String acc, L2GameClient client, SessionKey key)
 	{
 		if (Config.DEBUG)
@@ -414,6 +427,10 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Removes the waiting client.
+	 * @param client the client
+	 */
 	public void removeWaitingClient(L2GameClient client)
 	{
 		WaitingClient toRemove = null;
@@ -431,6 +448,10 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Send logout for the given account.
+	 * @param account the account
+	 */
 	public void sendLogout(String account)
 	{
 		if (account == null)
@@ -453,11 +474,21 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Adds the game server login.
+	 * @param account the account
+	 * @param client the client
+	 */
 	public void addGameServerLogin(String account, L2GameClient client)
 	{
 		_accountsInGameServer.put(account, client);
 	}
 	
+	/**
+	 * Send access level.
+	 * @param account the account
+	 * @param level the access level
+	 */
 	public void sendAccessLevel(String account, int level)
 	{
 		ChangeAccessLevel cal = new ChangeAccessLevel(account, level);
@@ -472,9 +503,14 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
-	public void sendClientTracert(String account, String[] adress)
+	/**
+	 * Send client tracert.
+	 * @param account the account
+	 * @param address the address
+	 */
+	public void sendClientTracert(String account, String[] address)
 	{
-		PlayerTracert ptc = new PlayerTracert(account, adress[0], adress[1], adress[2], adress[3], adress[4]);
+		PlayerTracert ptc = new PlayerTracert(account, address[0], address[1], address[2], address[3], address[4]);
 		try
 		{
 			sendPacket(ptc);
@@ -486,6 +522,12 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Send mail.
+	 * @param account the account
+	 * @param mailId the mail id
+	 * @param args the args
+	 */
 	public void sendMail(String account, String mailId, String... args)
 	{
 		SendMail sem = new SendMail(account, mailId, args);
@@ -500,6 +542,12 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Send temp ban.
+	 * @param account the account
+	 * @param ip the ip
+	 * @param time the time
+	 */
 	public void sendTempBan(String account, String ip, long time)
 	{
 		TempBan tbn = new TempBan(account, ip, time);
@@ -514,11 +562,20 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Hex to string.
+	 * @param hex the hex value
+	 * @return the hex value as string
+	 */
 	private String hexToString(byte[] hex)
 	{
 		return new BigInteger(hex).toString(16);
 	}
 	
+	/**
+	 * Kick player for the given account.
+	 * @param account the account
+	 */
 	public void doKickPlayer(String account)
 	{
 		L2GameClient client = _accountsInGameServer.get(account);
@@ -532,6 +589,10 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Gets the chars on server.
+	 * @param account the account
+	 */
 	private void getCharsOnServer(String account)
 	{
 		Connection con = null;
@@ -572,12 +633,12 @@ public class LoginServerThread extends Thread
 			if (Config.DEBUG)
 				_log.log(Level.WARNING, "", e);
 		}
-		
 	}
 	
 	/**
-	 * @param sl
-	 * @throws IOException
+	 * Send packet.
+	 * @param sl the sendable packet
+	 * @throws IOException Signals that an I/O exception has occurred.
 	 */
 	private void sendPacket(BaseSendablePacket sl) throws IOException
 	{
@@ -598,6 +659,7 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
+	 * Sets the max player.
 	 * @param maxPlayer The maxPlayer to set.
 	 */
 	public void setMaxPlayer(int maxPlayer)
@@ -607,6 +669,7 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
+	 * Gets the max player.
 	 * @return Returns the maxPlayer.
 	 */
 	public int getMaxPlayer()
@@ -615,8 +678,9 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * @param id
-	 * @param value
+	 * Send server status.
+	 * @param id the id
+	 * @param value the value
 	 */
 	public void sendServerStatus(int id, int value)
 	{
@@ -634,7 +698,7 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * Send Server Type Config to LS
+	 * Send Server Type Config to LS.
 	 */
 	public void sendServerType()
 	{
@@ -651,6 +715,13 @@ public class LoginServerThread extends Thread
 		}
 	}
 	
+	/**
+	 * Send change password.
+	 * @param accountName the account name
+	 * @param charName the char name
+	 * @param oldpass the old pass
+	 * @param newpass the new pass
+	 */
 	public void sendChangePassword(String accountName, String charName, String oldpass, String newpass)
 	{
 		ChangePassword cp = new ChangePassword(accountName, charName, oldpass, newpass);
@@ -666,7 +737,8 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * @return
+	 * Gets the status string.
+	 * @return the status string
 	 */
 	public String getStatusString()
 	{
@@ -674,7 +746,8 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * @return
+	 * Checks if is bracket shown.
+	 * @return true, if is bracket shown
 	 */
 	public boolean isBracketShown()
 	{
@@ -682,13 +755,18 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * @return Returns the serverName.
+	 * Gets the server name.
+	 * @return the server name.
 	 */
 	public String getServerName()
 	{
 		return _serverName;
 	}
 	
+	/**
+	 * Sets the server status.
+	 * @param status the new server status
+	 */
 	public void setServerStatus(int status)
 	{
 		switch (status)
@@ -729,6 +807,13 @@ public class LoginServerThread extends Thread
 		public int loginOkID1;
 		public int loginOkID2;
 		
+		/**
+		 * Instantiates a new session key.
+		 * @param loginOK1 the login o k1
+		 * @param loginOK2 the login o k2
+		 * @param playOK1 the play o k1
+		 * @param playOK2 the play o k2
+		 */
 		public SessionKey(int loginOK1, int loginOK2, int playOK1, int playOK2)
 		{
 			playOkID1 = playOK1;
@@ -751,6 +836,12 @@ public class LoginServerThread extends Thread
 		public L2GameClient gameClient;
 		public SessionKey session;
 		
+		/**
+		 * Instantiates a new waiting client.
+		 * @param acc the acc
+		 * @param client the client
+		 * @param key the key
+		 */
 		public WaitingClient(String acc, L2GameClient client, SessionKey key)
 		{
 			account = acc;

L2J_Server_BETA/java/com/l2jserver/gameserver/ai/Ctrl.java

@@ -58,42 +58,64 @@ import com.l2jserver.gameserver.model.actor.L2Character;
  */
 public interface Ctrl
 {
+	
 	/**
-	 * the character this AI serves 
-	 * @return
+	 * Gets the actor.
+	 * @return the actor
 	 */
 	L2Character getActor();
 	
 	/**
-	 * get current intention 
-	 * @return
+	 * Gets the intention.
+	 * @return the intention
 	 */
 	CtrlIntention getIntention();
 	
 	/**
-	 * get current ATTACK target 
-	 * @return
+	 * Gets the attack target.
+	 * @return the attack target
 	 */
 	L2Character getAttackTarget();
 	
 	/**
-	 * Set general state/intention for AI, with optional data 
-	 * @param intention
+	 * Set general state/intention for AI, with optional data.
+	 * @param intention the new intention
 	 */
 	void setIntention(CtrlIntention intention);
 	
+	/**
+	 * Sets the intention.
+	 * @param intention the intention
+	 * @param arg0 the arg0
+	 */
 	void setIntention(CtrlIntention intention, Object arg0);
 	
+	/**
+	 * Sets the intention.
+	 * @param intention the intention
+	 * @param arg0 the arg0
+	 * @param arg1 the arg1
+	 */
 	void setIntention(CtrlIntention intention, Object arg0, Object arg1);
 	
 	/**
-	 * Event, that notifies about previous step result, or user command,
-	 * that does not change current general intention 
-	 * @param evt
+	 * Event, that notifies about previous step result, or user command, that does not change current general intention.
+	 * @param evt the event
 	 */
 	void notifyEvent(CtrlEvent evt);
 	
+	/**
+	 * Notify an event.
+	 * @param evt the event
+	 * @param arg0 the arg0
+	 */
 	void notifyEvent(CtrlEvent evt, Object arg0);
 	
+	/**
+	 * Notify an event.
+	 * @param evt the event
+	 * @param arg0 the arg0
+	 * @param arg1 the arg1
+	 */
 	void notifyEvent(CtrlEvent evt, Object arg0, Object arg1);
 }

L2J_Server_BETA/java/com/l2jserver/gameserver/communitybbs/BB/Forum.java

@@ -212,7 +212,7 @@ public class Forum
 	}
 	
 	/**
-	 * @return
+	 * @return the forum Id
 	 */
 	public int getID()
 	{
@@ -232,8 +232,8 @@ public class Forum
 	}
 	
 	/**
-	 * @param name
-	 * @return
+	 * @param name the forum name
+	 * @return the forum for the given name
 	 */
 	public Forum getChildByName(String name)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/communitybbs/BB/Topic.java

@@ -108,7 +108,7 @@ public class Topic
 	}
 	
 	/**
-	 * @return
+	 * @return the topic Id
 	 */
 	public int getID()
 	{
@@ -121,7 +121,7 @@ public class Topic
 	}
 	
 	/**
-	 * @return
+	 * @return the topic name
 	 */
 	public String getName()
 	{
@@ -161,7 +161,7 @@ public class Topic
 	}
 	
 	/**
-	 * @return
+	 * @return the topic date
 	 */
 	public long getDate()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/communitybbs/Manager/AdminBBSManager.java

@@ -20,7 +20,8 @@ import com.l2jserver.gameserver.network.serverpackets.ShowBoard;
 public class AdminBBSManager extends BaseBBSManager
 {
 	/**
-	 * @return
+	 * Gets the single instance of AdminBBSManager.
+	 * @return single instance of AdminBBSManager
 	 */
 	public static AdminBBSManager getInstance()
 	{
@@ -41,7 +42,6 @@ public class AdminBBSManager extends BaseBBSManager
 		}
 		else
 		{
-			
 			ShowBoard sb = new ShowBoard("<html><body><br><br><center>the command: " + command
 					+ " is not implemented yet</center><br><br></body></html>", "101");
 			activeChar.sendPacket(sb);
@@ -51,10 +51,6 @@ public class AdminBBSManager extends BaseBBSManager
 		
 	}
 	
-	/**
-	 * 
-	 * @see com.l2jserver.gameserver.communitybbs.Manager.BaseBBSManager#parsewrite(java.lang.String, java.lang.String, java.lang.String, java.lang.String, java.lang.String, com.l2jserver.gameserver.model.actor.instance.L2PcInstance)
-	 */
 	@Override
 	public void parsewrite(String ar1, String ar2, String ar3, String ar4, String ar5, L2PcInstance activeChar)
 	{
@@ -62,7 +58,6 @@ public class AdminBBSManager extends BaseBBSManager
 		{
 			return;
 		}
-		
 	}
 	
 	private static class SingletonHolder

L2J_Server_BETA/java/com/l2jserver/gameserver/communitybbs/Manager/ForumsBBSManager.java

@@ -34,13 +34,17 @@ public class ForumsBBSManager extends BaseBBSManager
 	private int _lastid = 1;
 	
 	/**
-	 * @return
+	 * Gets the single instance of ForumsBBSManager.
+	 * @return single instance of ForumsBBSManager
 	 */
 	public static ForumsBBSManager getInstance()
 	{
 		return SingletonHolder._instance;
 	}
 	
+	/**
+	 * Instantiates a new forums bbs manager.
+	 */
 	protected ForumsBBSManager()
 	{
 		_table = new FastList<>();
@@ -70,6 +74,9 @@ public class ForumsBBSManager extends BaseBBSManager
 		}
 	}
 	
+	/**
+	 * Inits the root.
+	 */
 	public void initRoot()
 	{
 		for (Forum f : _table)
@@ -77,6 +84,10 @@ public class ForumsBBSManager extends BaseBBSManager
 		_log.info("Loaded " + _table.size() + " forums. Last forum id used: " + _lastid);
 	}
 	
+	/**
+	 * Adds the forum.
+	 * @param ff the forum
+	 */
 	public void addForum(Forum ff)
 	{
 		if (ff == null)
@@ -96,30 +107,30 @@ public class ForumsBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * 
-	 * @param Name
-	 * @return
+	 * Gets the forum by name.
+	 * @param name the forum name
+	 * @return the forum by name
 	 */
-	public Forum getForumByName(String Name)
+	public Forum getForumByName(String name)
 	{
 		for (Forum f : _table)
 		{
-			if (f.getName().equals(Name))
+			if (f.getName().equals(name))
 			{
 				return f;
 			}
 		}
-		
 		return null;
 	}
 	
 	/**
-	 * @param name
-	 * @param parent
-	 * @param type
-	 * @param perm
-	 * @param oid
-	 * @return
+	 * Creates the new forum.
+	 * @param name the forum name
+	 * @param parent the parent forum
+	 * @param type the forum type
+	 * @param perm the perm
+	 * @param oid the oid
+	 * @return the new forum
 	 */
 	public Forum createNewForum(String name, Forum parent, int type, int perm, int oid)
 	{
@@ -129,7 +140,8 @@ public class ForumsBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @return
+	 * Gets the a new Id.
+	 * @return the a new Id
 	 */
 	public int getANewID()
 	{
@@ -137,8 +149,9 @@ public class ForumsBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param idf
-	 * @return
+	 * Gets the forum by Id.
+	 * @param idf the the forum Id
+	 * @return the forum by Id
 	 */
 	public Forum getForumByID(int idf)
 	{
@@ -155,6 +168,7 @@ public class ForumsBBSManager extends BaseBBSManager
 	@Override
 	public void parsewrite(String ar1, String ar2, String ar3, String ar4, String ar5, L2PcInstance activeChar)
 	{
+		
 	}
 	
 	private static class SingletonHolder

L2J_Server_BETA/java/com/l2jserver/gameserver/communitybbs/Manager/RegionBBSManager.java

@@ -104,8 +104,9 @@ public class RegionBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param activeChar
-	 * @param name
+	 * Show old community player info.
+	 * @param activeChar the active char
+	 * @param name the player name
 	 */
 	private void showOldCommunityPI(L2PcInstance activeChar, String name)
 	{
@@ -174,8 +175,9 @@ public class RegionBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param activeChar
-	 * @param page 
+	 * Show old community.
+	 * @param activeChar the active char
+	 * @param page the page
 	 */
 	private void showOldCommunity(L2PcInstance activeChar, int page)
 	{
@@ -268,13 +270,17 @@ public class RegionBBSManager extends BaseBBSManager
 	private static FastMap<Integer, FastMap<String, String>> _communityPages = new FastMap<Integer, FastMap<String, String>>().shared();
 	
 	/**
-	 * @return
+	 * Gets the single instance of RegionBBSManager.
+	 * @return single instance of RegionBBSManager
 	 */
 	public static RegionBBSManager getInstance()
 	{
 		return SingletonHolder._instance;
 	}
 	
+	/**
+	 * Change community board.
+	 */
 	public void changeCommunityBoard()
 	{
 		final FastList<L2PcInstance> sortedPlayers = new FastList<>();
@@ -302,6 +308,10 @@ public class RegionBBSManager extends BaseBBSManager
 		writeCommunityPages();
 	}
 
+	/**
+	 * Adds the online player.
+	 * @param player the player
+	 */
 	private void addOnlinePlayer(L2PcInstance player)
 	{
 		boolean added = false;
@@ -341,6 +351,9 @@ public class RegionBBSManager extends BaseBBSManager
 		}
 	}
 	
+	/**
+	 * Write community pages.
+	 */
 	private void writeCommunityPages()
 	{
 		final StringBuilder htmlCode = new StringBuilder(2000);
@@ -543,6 +556,11 @@ public class RegionBBSManager extends BaseBBSManager
 		}
 	}
 	
+	/**
+	 * Gets the online count.
+	 * @param type the type, Game Masters or normal players
+	 * @return the online count
+	 */
 	private int getOnlineCount(String type)
 	{
 		if (type.equalsIgnoreCase("gm"))
@@ -551,11 +569,22 @@ public class RegionBBSManager extends BaseBBSManager
 		return _onlineCount;
 	}
 	
+	/**
+	 * Gets the online players.
+	 * @param page the page
+	 * @return the online players
+	 */
 	private FastList<L2PcInstance> getOnlinePlayers(int page)
 	{
 		return _onlinePlayers.get(page);
 	}
 	
+	/**
+	 * Gets the community page.
+	 * @param page the page
+	 * @param type the type
+	 * @return the community page
+	 */
 	public String getCommunityPage(int page, String type)
 	{
 		if (_communityPages.get(page) != null)

L2J_Server_BETA/java/com/l2jserver/gameserver/model/actor/L2Attackable.java

@@ -1154,7 +1154,7 @@ public class L2Attackable extends L2Npc
 			L2PcInstance act = (L2PcInstance) ai.getAttacker();
 			if (act.getAppearance().getInvisible() || ai.getAttacker().isInvul() || act.isSpawnProtected())
 			{
-				//Remove Object Should Use This Method and Can be Blocked While Interating
+				// Remove Object Should Use This Method and Can be Blocked While Interacting
 				getAggroList().remove(target);
 				return 0;
 			}

L2J_Server_BETA/java/com/l2jserver/gameserver/model/actor/instance/L2BlockInstance.java

@@ -96,8 +96,7 @@ public class L2BlockInstance extends L2MonsterInstance
 	}
 	
 	/**
-	 * Return if the block is red at this momment
-	 * @return
+	 * @return {@code true} if the block is red at this moment, {@code false} otherwise
 	 */
 	@Override
 	public int getColorEffect()

L2J_Server_BETA/java/com/l2jserver/gameserver/model/actor/knownlist/CharKnownList.java

@@ -55,9 +55,8 @@ public class CharKnownList extends ObjectKnownList
 	}
 	
 	/**
-	 * Return True if the L2PcInstance is in _knownPlayer of the L2Character.<BR><BR>
 	 * @param player The L2PcInstance to search in _knownPlayer
-	 * @return 
+	 * @return {@code true} if the player is in _knownPlayer of the character, {@code false} otherwise
 	 */
 	public final boolean knowsThePlayer(L2PcInstance player)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/actor/templates/L2CharTemplate.java

@@ -746,7 +746,7 @@ public class L2CharTemplate
 	
 	/**
 	 * Overridden in L2NpcTemplate
-	 * @return
+	 * @return the characters skills
 	 */
 	public Map<Integer, L2Skill> getSkills()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/actor/templates/L2NpcTemplate.java

@@ -188,7 +188,7 @@ public final class L2NpcTemplate extends L2CharTemplate
 	 * This is true if, and only if, obj is the same class represented by clazz, or a subclass of it or obj implements the interface represented by clazz.
 	 * @param obj
 	 * @param clazz
-	 * @return
+	 * @return {@code true} if the object can be assigned to the class, {@code false} otherwise
 	 */
 	public static boolean isAssignableTo(Object obj, Class<?> clazz)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/effects/EffectTemplate.java

@@ -129,10 +129,9 @@ public class EffectTemplate
 	
 	/**
 	 * Creates an L2Effect instance from an existing one and an Env object.
-	 * 
 	 * @param env
 	 * @param stolen
-	 * @return
+	 * @return the stolent effect
 	 */
 	public L2Effect getStolenEffect(Env env, L2Effect stolen)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/effects/L2Effect.java

@@ -654,8 +654,7 @@ public abstract class L2Effect implements IChanceSkillTrigger
 	}
 
 	/**
-	 * Return true if effect itself can be stolen
-	 * @return
+	 * @return {@code true} if effect itself can be stolen, {@code false} otherwise
 	 */
 	protected boolean effectCanBeStolen()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/quest/Quest.java

@@ -442,7 +442,7 @@ public class Quest extends ManagedScript
 	 * @param damage the damage dealt to the NPC by the player
 	 * @param isPet if {@code true}, the attack was actually made by the player's pet
 	 * @param skill the skill used to attack the NPC (can be null)
-	 * @return {@code true} or {@code false}
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyAttack(L2Npc npc, L2PcInstance attacker, int damage, boolean isPet, L2Skill skill)
 	{
@@ -481,7 +481,7 @@ public class Quest extends ManagedScript
 	/**
 	 * @param item
 	 * @param player
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyItemUse(L2Item item, L2PcInstance player)
 	{
@@ -501,7 +501,7 @@ public class Quest extends ManagedScript
 	 * @param instance
 	 * @param player
 	 * @param skill
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifySpellFinished(L2Npc instance, L2PcInstance player, L2Skill skill)
 	{
@@ -549,7 +549,7 @@ public class Quest extends ManagedScript
 	
 	/**
 	 * @param npc
-	 * @return
+	 * @return {@code true} if there was an error, {@code false} otherwise
 	 */
 	public final boolean notifySpawn(L2Npc npc)
 	{
@@ -569,7 +569,7 @@ public class Quest extends ManagedScript
 	 * @param event
 	 * @param npc
 	 * @param player
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyEvent(String event, L2Npc npc, L2PcInstance player)
 	{
@@ -587,7 +587,7 @@ public class Quest extends ManagedScript
 	
 	/**
 	 * @param player
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyEnterWorld(L2PcInstance player)
 	{
@@ -607,7 +607,7 @@ public class Quest extends ManagedScript
 	 * @param npc
 	 * @param killer
 	 * @param isPet
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyKill(L2Npc npc, L2PcInstance killer, boolean isPet)
 	{
@@ -626,7 +626,7 @@ public class Quest extends ManagedScript
 	/**
 	 * @param npc
 	 * @param qs
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyTalk(L2Npc npc, QuestState qs)
 	{
@@ -676,7 +676,7 @@ public class Quest extends ManagedScript
 	 * TODO: Remove and replace with listeners.
 	 * @param npc
 	 * @param player
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyAcquireSkillList(L2Npc npc, L2PcInstance player)
 	{
@@ -697,7 +697,7 @@ public class Quest extends ManagedScript
 	 * @param npc
 	 * @param player
 	 * @param skill
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyAcquireSkillInfo(L2Npc npc, L2PcInstance player, L2Skill skill)
 	{
@@ -718,7 +718,7 @@ public class Quest extends ManagedScript
 	 * @param npc
 	 * @param player
 	 * @param skill
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyAcquireSkill(L2Npc npc, L2PcInstance player, L2Skill skill)
 	{
@@ -785,7 +785,7 @@ public class Quest extends ManagedScript
 	 * @param skill
 	 * @param targets
 	 * @param isPet
-	 * @return
+	 * @return {@code true}
 	 */
 	public final boolean notifySkillSee(L2Npc npc, L2PcInstance caster, L2Skill skill, L2Object[] targets, boolean isPet)
 	{
@@ -798,7 +798,7 @@ public class Quest extends ManagedScript
 	 * @param caller
 	 * @param attacker
 	 * @param isPet
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyFactionCall(L2Npc npc, L2Npc caller, L2PcInstance attacker, boolean isPet)
 	{
@@ -848,7 +848,7 @@ public class Quest extends ManagedScript
 	 * @param npc
 	 * @param player
 	 * @param isPet
-	 * @return
+	 * @return {@code true}
 	 */
 	public final boolean notifyAggroRangeEnter(L2Npc npc, L2PcInstance player, boolean isPet)
 	{
@@ -859,7 +859,7 @@ public class Quest extends ManagedScript
 	/**
 	 * @param character
 	 * @param zone
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyEnterZone(L2Character character, L2ZoneType zone)
 	{
@@ -886,7 +886,7 @@ public class Quest extends ManagedScript
 	/**
 	 * @param character
 	 * @param zone
-	 * @return
+	 * @return {@code false} if there was an error or the message was sent, {@code true} otherwise
 	 */
 	public final boolean notifyExitZone(L2Character character, L2ZoneType zone)
 	{
@@ -912,7 +912,7 @@ public class Quest extends ManagedScript
 	
 	/**
 	 * @param winner
-	 * @param type
+	 * @param type {@code false} if there was an error, {@code true} otherwise
 	 */
 	public final void notifyOlympiadWin(L2PcInstance winner, CompetitionType type)
 	{
@@ -928,7 +928,7 @@ public class Quest extends ManagedScript
 	
 	/**
 	 * @param loser
-	 * @param type
+	 * @param type {@code false} if there was an error, {@code true} otherwise
 	 */
 	public final void notifyOlympiadLose(L2PcInstance loser, CompetitionType type)
 	{