BETA: Enabling project specific JavaDoc settings:

	* Fixed all malformed JavaDocs.
	* Added all missing JavaDoc tags.
'''Note 1:''' This doesn't force developers to write JavaDocs, only validates existing JavaDocs, anyway would be really good get all new code implementations well documented.
'''Note 2:''' In the future more restrictive JavaDoc settings could be enabled.

L2J_Server_BETA/.settings/org.eclipse.jdt.core.prefs

@@ -1,4 +1,4 @@
-#Sat Sep 24 23:36:52 UYT 2011
+#Sat Sep 24 23:47:07 UYT 2011
 eclipse.preferences.version=1
 org.eclipse.jdt.core.codeComplete.argumentPrefixes=
 org.eclipse.jdt.core.codeComplete.argumentSuffixes=
@@ -10,6 +10,7 @@ org.eclipse.jdt.core.codeComplete.staticFieldPrefixes=
 org.eclipse.jdt.core.codeComplete.staticFieldSuffixes=
 org.eclipse.jdt.core.codeComplete.staticFinalFieldPrefixes=
 org.eclipse.jdt.core.codeComplete.staticFinalFieldSuffixes=
+org.eclipse.jdt.core.compiler.doc.comment.support=enabled
 org.eclipse.jdt.core.compiler.problem.annotationSuperInterface=warning
 org.eclipse.jdt.core.compiler.problem.autoboxing=ignore
 org.eclipse.jdt.core.compiler.problem.comparingIdentical=warning
@@ -30,10 +31,23 @@ org.eclipse.jdt.core.compiler.problem.includeNullInfoFromAsserts=disabled
 org.eclipse.jdt.core.compiler.problem.incompatibleNonInheritedInterfaceMethod=warning
 org.eclipse.jdt.core.compiler.problem.incompleteEnumSwitch=ignore
 org.eclipse.jdt.core.compiler.problem.indirectStaticAccess=ignore
+org.eclipse.jdt.core.compiler.problem.invalidJavadoc=warning
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTags=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsDeprecatedRef=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsNotVisibleRef=enabled
+org.eclipse.jdt.core.compiler.problem.invalidJavadocTagsVisibility=private
 org.eclipse.jdt.core.compiler.problem.localVariableHiding=ignore
 org.eclipse.jdt.core.compiler.problem.methodWithConstructorName=warning
 org.eclipse.jdt.core.compiler.problem.missingDeprecatedAnnotation=ignore
 org.eclipse.jdt.core.compiler.problem.missingHashCodeMethod=ignore
+org.eclipse.jdt.core.compiler.problem.missingJavadocComments=ignore
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsOverriding=disabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocCommentsVisibility=public
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagDescription=no_tag
+org.eclipse.jdt.core.compiler.problem.missingJavadocTags=warning
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsMethodTypeParameters=enabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsOverriding=enabled
+org.eclipse.jdt.core.compiler.problem.missingJavadocTagsVisibility=private
 org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotation=ignore
 org.eclipse.jdt.core.compiler.problem.missingOverrideAnnotationForInterfaceMethodImplementation=enabled
 org.eclipse.jdt.core.compiler.problem.missingSerialVersion=warning

L2J_Server_BETA/java/com/l2jserver/Config.java

@@ -1091,7 +1091,7 @@ public final class Config
 	/**
 	 * This class initializes all global variables for configuration.<br>
 	 * If the key doesn't appear in properties file, a default value is set by this class.
-	 * @see CONFIGURATION_FILE (properties file) for configuring your server.
+	 * {@link #CONFIGURATION_FILE} (properties file) for configuring your server.
 	 */
 	public static void load()
 	{
@@ -3048,7 +3048,6 @@ public final class Config
 	 * @param pName (String) : name of the parameter to change
 	 * @param pValue (String) : new value of the parameter
 	 * @return boolean : true if modification has been made
-	 * @link useAdminCommand
 	 */
 	public static boolean setParameterValue(String pName, String pValue)
 	{
@@ -3446,10 +3445,9 @@ public final class Config
 	
 	/**
 	 * Save hexadecimal ID of the server in the L2Properties file.
-	 * @param string (String) : hexadecimal ID of the server to store
-	 * @see HEXID_FILE
-	 * @see saveHexid(String string, String fileName)
-	 * @link LoginServerThread
+	 * {@link #HEXID_FILE}
+	 * @param serverId
+	 * @param string hexadecimal ID of the server to store
 	 */
 	public static void saveHexid(int serverId, String string)
 	{
@@ -3458,6 +3456,7 @@ public final class Config
 	
 	/**
 	 * Save hexadecimal ID of the server in the L2Properties file.
+	 * @param serverId 
 	 * @param hexId (String) : hexadecimal ID of the server to store
 	 * @param fileName (String) : name of the L2Properties file
 	 */
@@ -3484,6 +3483,7 @@ public final class Config
 	
 	/**
 	 * Loads flood protector configurations.
+	 * @param properties 
 	 */
 	private static void loadFloodProtectorConfigs(final L2Properties properties)
 	{
@@ -3653,6 +3653,8 @@ public final class Config
 	/**
 	 * itemId1,itemNumber1;itemId2,itemNumber2...
 	 * to the int[n][2] = [itemId1][itemNumber1],[itemId2][itemNumber2]...
+	 * @param line 
+	 * @return 
 	 */
 	private static int[][] parseItemsList(String line)
 	{

L2J_Server_BETA/java/com/l2jserver/configurator/ConfigUserInterface.java

@@ -316,6 +316,7 @@ public class ConfigUserInterface extends JFrame implements ActionListener
 	
 	/**
 	 * @param value
+	 * @return 
 	 */
 	private Object parseValue(String value)
 	{
@@ -743,10 +744,10 @@ public class ConfigUserInterface extends JFrame implements ActionListener
 	}
 	
 	/**
+	 * @param keyName 
 	 * @return Returns the configuration setting name in a
 	 * human readable form.
 	 */
-	
 	public static String unCamelize(final String keyName) {
 		Pattern p = Pattern.compile("\\p{Lu}");
 		Matcher m = p.matcher(keyName);

L2J_Server_BETA/java/com/l2jserver/dbinstaller/util/swing/SpringUtilities.java

@@ -45,6 +45,7 @@ public class SpringUtilities {
     /**
      * A debugging utility that prints to stdout the component's
      * minimum, preferred, and maximum sizes.
+     * @param c 
      */
     public static void printSizes(Component c) {
         System.out.println("minimumSize = " + c.getMinimumSize());
@@ -58,7 +59,7 @@ public class SpringUtilities {
      * a grid. Each component is as big as the maximum
      * preferred width and height of the components.
      * The parent is made just big enough to fit them all.
-     *
+     * @param parent 
      * @param rows number of rows
      * @param cols number of columns
      * @param initialX x location to start the grid at
@@ -161,7 +162,7 @@ public class SpringUtilities {
      * preferred width of the components in that column;
      * height is similarly determined for each row.
      * The parent is made just big enough to fit them all.
-     *
+     * @param parent 
      * @param rows number of rows
      * @param cols number of columns
      * @param initialX x location to start the grid at

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

@@ -50,6 +50,7 @@ public class GameTimeController
 	
 	/**
 	 * one ingame day is 240 real minutes
+	 * @return 
 	 */
 	public static GameTimeController getInstance()
 	{
@@ -237,9 +238,6 @@ public class GameTimeController
 		}
 	}
 	
-	/**
-	 * @param rise
-	 */
 	class BroadcastSunState implements Runnable
 	{
 		int h;

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

@@ -76,7 +76,7 @@ public class GeoData
 	 * @param y
 	 * @param zmin
 	 * @param zmax
-	 * @param spawnid
+	 * @param spawn
 	 * @return
 	 */
 	public short getSpawnHeight(int x, int y, int zmin, int zmax, L2Spawn spawn)
@@ -121,9 +121,9 @@ public class GeoData
 	}
 	
 	/**
-	 * @param cha
+	 * @param gm
 	 * @param target
-	 * @return True if cha can see target (LOS) and send usful info to PC
+	 * @return True if cha can see target (LOS) and send useful info to PC
 	 */
 	public boolean canSeeTargetDebug(L2PcInstance gm, L2Object target)
 	{

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

@@ -88,9 +88,6 @@ public class GeoEngine extends GeoData
 		return nGetHeight((x - L2World.MAP_MIN_X) >> 4, (y - L2World.MAP_MIN_Y) >> 4, z);
 	}
 	
-	/**
-	 * @see com.l2jserver.gameserver.GeoData#getSpawnHeight(int, int, int, int, int)
-	 */
 	@Override
 	public short getSpawnHeight(int x, int y, int zmin, int zmax, L2Spawn spawn)
 	{
@@ -784,7 +781,7 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param pos
+	 * @param geo_pos 
 	 * @return Block Index: 0-255
 	 */
 	private static int getBlock(int geo_pos)
@@ -793,7 +790,7 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param pos
+	 * @param geo_pos 
 	 * @return Cell Index: 0-7
 	 */
 	private static int getCell(int geo_pos)
@@ -833,8 +830,8 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
+	 * @param geox 
+	 * @param geoy 
 	 * @param z
 	 * @return Nearest Z
 	 */
@@ -911,8 +908,8 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
+	 * @param geox 
+	 * @param geoy 
 	 * @param z
 	 * @return One layer higher Z than parameter Z
 	 */
@@ -990,11 +987,12 @@ public class GeoEngine extends GeoData
 	}
 	
 	/**
-	 * @param x
-	 * @param y
+	 * @param geox 
+	 * @param geoy 
 	 * @param zmin
 	 * @param zmax
-	 * @return Z betwen zmin and zmax
+	 * @param spawn 
+	 * @return Z between zmin and zmax
 	 */
 	private static short nGetSpawnHeight(int geox, int geoy, int zmin, int zmax, L2Spawn spawn)
 	{
@@ -1185,7 +1183,9 @@ public class GeoEngine extends GeoData
 	 * @param z
 	 * @param inc_x
 	 * @param inc_y
+	 * @param inc_z 
 	 * @param tz
+	 * @param debug 
 	 * @return True if Char can see target
 	 */
 	private static boolean nLOS(int x, int y, int z, int inc_x, int inc_y, double inc_z, int tz, boolean debug)

L2J_Server_BETA/java/com/l2jserver/gameserver/GmListTable.java

@@ -73,6 +73,8 @@ public class GmListTable
 	
 	/**
 	 * Add a L2PcInstance player to the Set _gmList
+	 * @param player 
+	 * @param hidden 
 	 */
 	public void addGm(L2PcInstance player, boolean hidden)
 	{

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

@@ -79,7 +79,9 @@ public class LoginServerThread extends Thread
 	protected static final Logger _log = Logger.getLogger(LoginServerThread.class.getName());
 	protected static final Logger _logAccounting = Logger.getLogger("accounting");
 	
-	/** {@see com.l2jserver.loginserver.LoginServer#PROTOCOL_REV } */
+	/**
+	 * @see com.l2jserver.loginserver.L2LoginServer#PROTOCOL_REV
+	 */
 	private static final int REVISION = 0x0106;
 	private RSAPublicKey _publicKey;
 	private final String _hostname;
@@ -614,7 +616,8 @@ public class LoginServerThread extends Thread
 	}
 	
 	/**
-	 * @param server_gm_only
+	 * @param id
+	 * @param value
 	 */
 	public void sendServerStatus(int id, int value)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/SevenSigns.java

@@ -559,6 +559,7 @@ public class SevenSigns
 	/**
 	 * returns true if the given date is in Seal Validation or in Quest Event Results period
 	 * @param date
+	 * @return 
 	 */
 	public boolean isDateInSealValidPeriod(Calendar date)
 	{
@@ -741,8 +742,6 @@ public class SevenSigns
 	
 	/**
 	 * Restores all Seven Signs data and settings, usually called at server startup.
-	 *
-	 * @throws Exception
 	 */
 	protected void restoreSevenSignsData()
 	{
@@ -822,13 +821,8 @@ public class SevenSigns
 	}
 	
 	/**
-	 * Saves all Seven Signs player data.
+	 * Saves all Seven Signs player data.<br>
 	 * Should be called on period change and shutdown only.
-	 * <BR>
-	 *
-	 * @param player
-	 * @param updateSettings
-	 * @throws Exception
 	 */
 	public void saveSevenSignsData()
 	{
@@ -973,16 +967,12 @@ public class SevenSigns
 	}
 	
 	/**
-	 * Used to specify cabal-related details for the specified player. This method
-	 * checks to see if the player has registered before and will update the database
-	 * if necessary.
-	 * <BR>
-	 * Returns the cabal ID the player has joined.
-	 *
-	 * @param player
+	 * Used to specify cabal-related details for the specified player.<br>
+	 * This method checks to see if the player has registered before and will update the database if necessary.
+	 * @param objectId
 	 * @param chosenCabal
 	 * @param chosenSeal
-	 * @return int cabal
+	 * @return the cabal ID the player has joined.
 	 */
 	public int setPlayerInfo(int objectId, int chosenCabal, int chosenSeal)
 	{
@@ -1053,10 +1043,9 @@ public class SevenSigns
 	 * Returns the amount of ancient adena the specified player can claim, if any.<BR>
 	 * If removeReward = True, all the ancient adena owed to them is removed, then
 	 * DB is updated.
-	 *
-	 * @param player
+	 * @param objectId
 	 * @param removeReward
-	 * @return int rewardAmount
+	 * @return
 	 */
 	public int getAncientAdenaReward(int objectId, boolean removeReward)
 	{
@@ -1084,15 +1073,14 @@ public class SevenSigns
 	/**
 	 * Used to add the specified player's seal stone contribution points
 	 * to the current total for their cabal. Returns the point score the
-	 * contribution was worth.
+	 * contribution was worth.<br>
 	 *
 	 * Each stone count <B>must be</B> broken down and specified by the stone's color.
-	 *
-	 * @param player
+	 * @param objectId
 	 * @param blueCount
 	 * @param greenCount
 	 * @param redCount
-	 * @return int contribScore
+	 * @return
 	 */
 	public long addPlayerStoneContrib(int objectId, long blueCount, long greenCount, long redCount)
 	{
@@ -1376,6 +1364,7 @@ public class SevenSigns
 	 * necropolises, who belong to the losing cabal.
 	 * <BR><BR>
 	 * Should only ever called at the beginning of Seal Validation.
+	 * @param compWinner 
 	 */
 	protected void teleLosingCabalFromDungeons(String compWinner)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/SevenSignsFestival.java

@@ -814,7 +814,7 @@ public class SevenSignsFestival implements SpawnListener
 	/**
 	 * Returns the associated name (level range) to a given festival ID.
 	 *
-	 * @param int festivalID
+	 * @param festivalID
 	 * @return String festivalName
 	 */
 	public static final String getFestivalName(int festivalID)
@@ -929,8 +929,6 @@ public class SevenSignsFestival implements SpawnListener
 	/**
 	 * Restores saved festival data, basic settings from the properties file
 	 * and past high score data from the database.
-	 *
-	 * @throws Exception
 	 */
 	protected void restoreFestivalData()
 	{
@@ -1029,7 +1027,6 @@ public class SevenSignsFestival implements SpawnListener
 	 * If updateSettings = true, then all Seven Signs data is updated in the database.
 	 *
 	 * @param updateSettings
-	 * @throws Exception
 	 */
 	public void saveFestivalData(boolean updateSettings)
 	{
@@ -1202,6 +1199,7 @@ public class SevenSignsFestival implements SpawnListener
 	
 	/**
 	 * Used to reset all festival data at the beginning of a new quest event period.
+	 * @param updateSettings 
 	 */
 	protected void resetFestivalData(boolean updateSettings)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/Shutdown.java

@@ -66,11 +66,9 @@ public class Shutdown extends Thread
 	private static final String[] MODE_TEXT = { "SIGTERM", "shutting down", "restarting", "aborting" };
 	
 	/**
-	 * This function starts a shutdown countdown from Telnet (Copied from Function startShutdown())
+	 * This function starts a shutdown count down from Telnet (Copied from Function startShutdown())
 	 *
-	 * @param ip IP Which Issued shutdown command
-	 * @param seconds seconds untill shutdown
-	 * @param restart true if the server will restart after shutdown
+	 * @param seconds seconds until shutdown
 	 */
 	private void SendServerQuit(int seconds)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/ThreadPoolManager.java

@@ -48,9 +48,9 @@ import com.l2jserver.util.StringUtil;
  * </ul>
  * or
  * <ul>
- * <li>{@link #scheduleEffectAtFixedRate(Runnable, long, long)(Runnable, long)} : for effects Tasks that needs to be executed periodicaly.</li>
- * <li>{@link #scheduleGeneralAtFixedRate(Runnable, long, long)(Runnable, long)} : for scheduled Tasks that needs to be executed periodicaly.</li>
- * <li>{@link #scheduleAiAtFixedRate(Runnable, long, long)(Runnable, long)} : for AI Tasks that needs to be executed periodicaly</li>
+ * <li>{@link #scheduleEffectAtFixedRate(Runnable, long, long)} : for effects Tasks that needs to be executed periodicaly.</li>
+ * <li>{@link #scheduleGeneralAtFixedRate(Runnable, long, long)} : for scheduled Tasks that needs to be executed periodicaly.</li>
+ * <li>{@link #scheduleAiAtFixedRate(Runnable, long, long)} : for AI Tasks that needs to be executed periodicaly</li>
  * </ul></p>
  *
  * <p>For all Tasks that should be executed with no delay asynchronously in a ThreadPool there also are usual {@link ThreadPoolExecutor ThreadPools}

L2J_Server_BETA/java/com/l2jserver/gameserver/TradeController.java

@@ -44,7 +44,9 @@ public class TradeController
 	private int _nextListId;
 	private Map<Integer, L2TradeList> _lists = new FastMap<Integer, L2TradeList>();
 	
-	/** Task launching the function for restore count of Item (Clan Hall) */
+	//**
+	// * Task launching the function for restore count of Item (Clan Hall) 
+	// */
 	/*public class RestoreCount implements Runnable
 	{
 		private int _timer;

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

@@ -153,7 +153,7 @@ abstract class AbstractAI implements Ctrl
 	}
 	
 	/**
-	 * Return the L2Character managed by this Accessor AI.<BR><BR>
+	 * @return the L2Character managed by this Accessor AI.
 	 */
 	public L2Character getActor()
 	{
@@ -161,7 +161,7 @@ abstract class AbstractAI implements Ctrl
 	}
 	
 	/**
-	 * Return the current Intention.<BR><BR>
+	 * @return the current Intention.
 	 */
 	public CtrlIntention getIntention()
 	{
@@ -174,7 +174,7 @@ abstract class AbstractAI implements Ctrl
 	}
 	
 	/**
-	 * Return the current cast target.<BR><BR>
+	 * @return the current cast target.
 	 */
 	public L2Character getCastTarget()
 	{
@@ -187,7 +187,7 @@ abstract class AbstractAI implements Ctrl
 	}
 	
 	/**
-	 * Return current attack target.<BR><BR>
+	 * @return current attack target.
 	 */
 	public L2Character getAttackTarget()
 	{
@@ -550,7 +550,8 @@ abstract class AbstractAI implements Ctrl
 	 * Move the actor to Pawn server side AND client side by sending Server->Client packet MoveToPawn <I>(broadcast)</I>.<BR><BR>
 	 *
 	 * <FONT COLOR=#FF0000><B> <U>Caution</U> : Low level function, used by AI subclasses</B></FONT><BR><BR>
-	 *
+	 * @param pawn 
+	 * @param offset 
 	 */
 	protected void moveToPawn(L2Object pawn, int offset)
 	{
@@ -622,7 +623,9 @@ abstract class AbstractAI implements Ctrl
 	 * Move the actor to Location (x,y,z) server side AND client side by sending Server->Client packet CharMoveToLocation <I>(broadcast)</I>.<BR><BR>
 	 *
 	 * <FONT COLOR=#FF0000><B> <U>Caution</U> : Low level function, used by AI subclasses</B></FONT><BR><BR>
-	 *
+	 * @param x 
+	 * @param y 
+	 * @param z 
 	 */
 	protected void moveTo(int x, int y, int z)
 	{
@@ -651,7 +654,7 @@ abstract class AbstractAI implements Ctrl
 	 * Stop the actor movement server side AND client side by sending Server->Client packet StopMove/StopRotation <I>(broadcast)</I>.<BR><BR>
 	 *
 	 * <FONT COLOR=#FF0000><B> <U>Caution</U> : Low level function, used by AI subclasses</B></FONT><BR><BR>
-	 *
+	 * @param pos 
 	 */
 	protected void clientStopMoving(L2CharPosition pos)
 	{
@@ -838,7 +841,7 @@ abstract class AbstractAI implements Ctrl
 	 * Create and Launch an AI Follow Task to execute every 0.5s, following at specified range.<BR><BR>
 	 *
 	 * @param target The L2Character to follow
-	 *
+	 * @param range 
 	 */
 	public synchronized void startFollow(L2Character target, int range)
 	{

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

@@ -60,24 +60,39 @@ import com.l2jserver.gameserver.model.actor.L2Character;
 public interface Ctrl
 {
 	
-	/** the character this AI serves */
+	/**
+	 * the character this AI serves 
+	 * @return
+	 */
 	L2Character getActor();
 	
-	/** get current intention */
+	/**
+	 * get current intention 
+	 * @return
+	 */
 	CtrlIntention getIntention();
 	
-	/** get current ATTACK target */
+	/**
+	 * get current ATTACK target 
+	 * @return
+	 */
 	L2Character getAttackTarget();
 	
-	/** Set general state/intention for AI, with optional data */
+	/**
+	 * Set general state/intention for AI, with optional data 
+	 * @param intention
+	 */
 	void setIntention(CtrlIntention intention);
 	
 	void setIntention(CtrlIntention intention, Object arg0);
 	
 	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 */
+	/**
+	 * Event, that notifies about previous step result, or user command,
+	 * that does not change current general intention 
+	 * @param evt
+	 */
 	void notifyEvent(CtrlEvent evt);
 	
 	void notifyEvent(CtrlEvent evt, Object arg0);

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

@@ -107,8 +107,6 @@ public class L2AttackableAI extends L2CharacterAI implements Runnable
 	}
 	
 	/**
-	 * Return True if the target is autoattackable (depends on the actor type).<BR><BR>
-	 *
 	 * <B><U> Actor is a L2GuardInstance</U> :</B><BR><BR>
 	 * <li>The target isn't a Folk or a Door</li>
 	 * <li>The target isn't dead, isn't invulnerable, isn't in silent moving mode AND too far (>100)</li>
@@ -136,7 +134,7 @@ public class L2AttackableAI extends L2CharacterAI implements Runnable
 	 * <li>The actor is Aggressive</li><BR><BR>
 	 *
 	 * @param target The targeted L2Object
-	 *
+	 * @return True if the target is autoattackable (depends on the actor type).
 	 */
 	private boolean autoAttackCondition(L2Character target)
 	{
@@ -2474,7 +2472,6 @@ public class L2AttackableAI extends L2CharacterAI implements Runnable
 	 * <li>Add the target to the actor _aggroList or update hate if already present </li>
 	 * <li>Set the actor Intention to AI_INTENTION_ATTACK (if actor is L2GuardInstance check if it isn't too far from its home location)</li><BR><BR>
 	 *
-	 * @param attacker The L2Character that attacks
 	 * @param aggro The value of hate to add to the actor against the target
 	 *
 	 */
@@ -2526,7 +2523,7 @@ public class L2AttackableAI extends L2CharacterAI implements Runnable
 	}
 	
 	/**
-	 * @param timepass The timepass to set.
+	 * @param TP The timepass to set.
 	 */
 	public void setTimepass(int TP)
 	{

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

@@ -92,8 +92,6 @@ public class L2FortSiegeGuardAI extends L2CharacterAI implements Runnable
 	}
 	
 	/**
-	 * Return True if the target is autoattackable (depends on the actor type).<BR><BR>
-	 *
 	 * <B><U> Actor is a L2GuardInstance</U> :</B><BR><BR>
 	 * <li>The target isn't a Folk or a Door</li>
 	 * <li>The target isn't dead, isn't invulnerable, isn't in silent moving mode AND too far (>100)</li>
@@ -121,7 +119,7 @@ public class L2FortSiegeGuardAI extends L2CharacterAI implements Runnable
 	 * <li>The actor is Aggressive</li><BR><BR>
 	 *
 	 * @param target The targeted L2Object
-	 *
+	 * @return True if the target is autoattackable (depends on the actor type).
 	 */
 	private boolean autoAttackCondition(L2Character target)
 	{
@@ -783,7 +781,6 @@ public class L2FortSiegeGuardAI extends L2CharacterAI implements Runnable
 	 * <li>Add the target to the actor _aggroList or update hate if already present </li>
 	 * <li>Set the actor Intention to AI_INTENTION_ATTACK (if actor is L2GuardInstance check if it isn't too far from its home location)</li><BR><BR>
 	 *
-	 * @param attacker The L2Character that attacks
 	 * @param aggro The value of hate to add to the actor against the target
 	 *
 	 */

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

@@ -90,8 +90,6 @@ public class L2SiegeGuardAI extends L2CharacterAI implements Runnable
 	}
 	
 	/**
-	 * Return True if the target is autoattackable (depends on the actor type).<BR><BR>
-	 *
 	 * <B><U> Actor is a L2GuardInstance</U> :</B><BR><BR>
 	 * <li>The target isn't a Folk or a Door</li>
 	 * <li>The target isn't dead, isn't invulnerable, isn't in silent moving mode AND too far (>100)</li>
@@ -119,7 +117,7 @@ public class L2SiegeGuardAI extends L2CharacterAI implements Runnable
 	 * <li>The actor is Aggressive</li><BR><BR>
 	 *
 	 * @param target The targeted L2Object
-	 *
+	 * @return True if the target is autoattackable (depends on the actor type).
 	 */
 	protected boolean autoAttackCondition(L2Character target)
 	{
@@ -764,7 +762,6 @@ public class L2SiegeGuardAI extends L2CharacterAI implements Runnable
 	 * <li>Add the target to the actor _aggroList or update hate if already present </li>
 	 * <li>Set the actor Intention to AI_INTENTION_ATTACK (if actor is L2GuardInstance check if it isn't too far from its home location)</li><BR><BR>
 	 *
-	 * @param attacker The L2Character that attacks
 	 * @param aggro The value of hate to add to the actor against the target
 	 *
 	 */

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

@@ -19,13 +19,14 @@ import com.l2jserver.gameserver.model.L2Skill;
 import com.l2jserver.gameserver.model.actor.L2Character;
 import com.l2jserver.gameserver.model.actor.L2Vehicle;
 
+/**
+ * @author DS
+ */
 public abstract class L2VehicleAI extends L2CharacterAI
 {
 	/**
-	 * 
-	 * @author DS
 	 * Simple AI for vehicles
-	 * 
+	 * @param accessor 
 	 */
 	public L2VehicleAI(L2Vehicle.AIAccessor accessor)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/cache/FastMRUCache.java

@@ -23,7 +23,8 @@ import javolution.util.FastMap;
 
 /**
  * @author Layane
- *
+ * @param <K> 
+ * @param <V> 
  */
 @SuppressWarnings({ "rawtypes" })
 public class FastMRUCache<K, V> extends FastCollection implements Reusable
@@ -80,8 +81,6 @@ public class FastMRUCache<K, V> extends FastCollection implements Reusable
 	
 	/**
 	 * Returns a set allocated from the stack when executing in a
-	 * {@link javolution.realtime.PoolContext PoolContext}).
-	 *
 	 * @return a new, pre-allocated or recycled set instance.
 	 */
 	public static FastMRUCache newInstance()

L2J_Server_BETA/java/com/l2jserver/gameserver/cache/HtmCache.java

@@ -229,8 +229,8 @@ public class HtmCache
 	
 	/**
 	 * Check if an HTM exists and can be loaded
-	 * @param
-	 * path The path to the HTM
+	 * @param path The path to the HTM
+	 * @return 
 	 * */
 	public boolean isLoadable(String path)
 	{

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

@@ -61,8 +61,8 @@ public class Forum
 	 * {@link com.l2jserver.gameserver.communitybbs.Manager.ForumsBBSManager#
 	 * addForum(com.l2jserver.gameserver.communitybbs.BB.Forum)} to add forum
 	 * to the forums manager.
-	 *
-	 * @param i
+	 * @param Forumid
+	 * @param FParent
 	 */
 	public Forum(int Forumid, Forum FParent)
 	{
@@ -80,6 +80,7 @@ public class Forum
 	 * @param parent
 	 * @param type
 	 * @param perm
+	 * @param OwnerID 
 	 */
 	public Forum(String name, Forum parent, int type, int perm, int OwnerID)
 	{

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

@@ -44,12 +44,18 @@ public class Post
 		public int postForumId;
 		public String postTxt;
 	}
+	
 	private List<CPost> _post;
+	//public enum ConstructorType {REPLY, CREATE };
+	
 	/**
-	 * @param restore
-	 * @param t
+	 * @param _PostOwner
+	 * @param _PostOwnerID
+	 * @param date
+	 * @param tid
+	 * @param _PostForumID
+	 * @param txt
 	 */
-	//public enum ConstructorType {REPLY, CREATE };
 	public Post(String _PostOwner,int _PostOwnerID,long date,int tid,int _PostForumID,String txt)
 	{
 		_post = new FastList<CPost>();

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

@@ -40,15 +40,15 @@ public class Topic
 	private int _cReply;
 	
 	/**
-	 * @param restaure
-	 * @param i
-	 * @param j
-	 * @param string
-	 * @param k
-	 * @param string2
-	 * @param l
-	 * @param m
-	 * @param n
+	 * @param ct
+	 * @param id
+	 * @param fid
+	 * @param name
+	 * @param date
+	 * @param oname
+	 * @param oid
+	 * @param type
+	 * @param Creply
 	 */
 	public Topic(ConstructorType ct, int id, int fid, String name, long date, String oname, int oid, int type, int Creply)
 	{
@@ -135,7 +135,7 @@ public class Topic
 	}
 	
 	/**
-	 *
+	 * @param f 
 	 */
 	public void deleteme(Forum f)
 	{

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

@@ -56,6 +56,7 @@ public abstract class BaseBBSManager
 	
 	/**
 	 * @param html
+	 * @param acha 
 	 */
 	protected void send1001(String html, L2PcInstance acha)
 	{
@@ -66,7 +67,7 @@ public abstract class BaseBBSManager
 	}
 	
 	/**
-	 * @param i
+	 * @param acha 
 	 */
 	protected void send1002(L2PcInstance acha)
 	{
@@ -77,6 +78,7 @@ public abstract class BaseBBSManager
 	 * @param activeChar
 	 * @param string
 	 * @param string2
+	 * @param string3 
 	 */
 	protected void send1002(L2PcInstance activeChar, String string, String string2, String string3)
 	{

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

@@ -96,7 +96,8 @@ public class ForumsBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param string
+	 * 
+	 * @param Name
 	 * @return
 	 */
 	public Forum getForumByName(String Name)
@@ -114,7 +115,10 @@ public class ForumsBBSManager extends BaseBBSManager
 	
 	/**
 	 * @param name
-	 * @param forumByName
+	 * @param parent
+	 * @param type
+	 * @param perm
+	 * @param oid
 	 * @return
 	 */
 	public Forum createNewForum(String name, Forum parent, int type, int perm, int oid)

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

@@ -136,7 +136,7 @@ public class PostBBSManager extends BaseBBSManager
 	
 	/**
 	 * @param topic
-	 * @param forumByID
+	 * @param forum 
 	 * @param activeChar
 	 * @param idp
 	 */
@@ -157,11 +157,10 @@ public class PostBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param posttByTopic
-	 * @param forumByID
+	 * @param topic 
+	 * @param forum 
 	 * @param activeChar
 	 * @param ind
-	 * @param idf
 	 */
 	private void showPost(Topic topic, Forum forum, L2PcInstance activeChar, int ind)
 	{

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

@@ -166,6 +166,7 @@ public class RegionBBSManager extends BaseBBSManager
 	
 	/**
 	 * @param activeChar
+	 * @param page 
 	 */
 	private void showOldCommunity(L2PcInstance activeChar, int page)
 	{

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

@@ -255,7 +255,7 @@ public class TopicBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param forumByID
+	 * @param forum 
 	 * @param activeChar
 	 * @param idf
 	 */
@@ -325,7 +325,10 @@ public class TopicBBSManager extends BaseBBSManager
 	}
 	
 	/**
-	 * @param memo
+	 * @param forum 
+	 * @param activeChar 
+	 * @param index 
+	 * @param idf 
 	 */
 	private void showTopics(Forum forum, L2PcInstance activeChar, int index, int idf)
 	{
@@ -354,6 +357,7 @@ public class TopicBBSManager extends BaseBBSManager
 	/**
 	 * @param forum
 	 * @param activeChar
+	 * @param index 
 	 */
 	private void showMemoTopics(Forum forum, L2PcInstance activeChar, int index)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/AccessLevels.java

@@ -189,6 +189,9 @@ public class AccessLevels
 		return accessLevel;
 	}
 	
+	/**
+	 * @param accessLevel
+	 */
 	public void addBanAccessLevel(int accessLevel)
 	{
 		synchronized (_accessLevels)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/AugmentationData.java

@@ -510,9 +510,8 @@ public class AugmentationData
 	
 	/**
 	 * Generate a new random augmentation
-	 * @param item
 	 * @param lifeStoneLevel
-	 * @param lifeSoneGrade
+	 * @param lifeStoneGrade
 	 * @param bodyPart
 	 * @return L2Augmentation
 	 */

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/ClanTable.java

@@ -146,6 +146,7 @@ public class ClanTable
 	 * Creates a new clan and store clan info to database
 	 *
 	 * @param player
+	 * @param clanName 
 	 * @return NULL if clan with same name already exists
 	 */
 	public L2Clan createClan(L2PcInstance player, String clanName)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/EnchantGroupsTable.java

@@ -218,6 +218,7 @@ public class EnchantGroupsTable
 	}
 	
 	/**
+	 * @param id 
 	 * @return L2EnchantSkillGroup
 	 */
 	public L2EnchantSkillGroup getEnchantSkillGroupById(int id)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/EventDroplist.java

@@ -92,7 +92,7 @@ public class EventDroplist
 	}
 	
 	/**
-	 * Return all DateDrop of EventDroplist allNpcDateDrops within the date range.<BR><BR>
+	 * @return all DateDrop of EventDroplist allNpcDateDrops within the date range.
 	 */
 	public List<DateDrop> getAllDrops()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/FishTable.java

@@ -100,9 +100,9 @@ public class FishTable
 	}
 	
 	/**
-	 * @param Fish - lvl
-	 * @param Fish - type
-	 * @param Fish - group
+	 * @param lvl
+	 * @param type
+	 * @param group
 	 * @return List of Fish that can be fished
 	 */
 	public List<FishData> getfish(int lvl, int type, int group)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/HelperBuffTable.java

@@ -99,6 +99,8 @@ public class HelperBuffTable
 	
 	/**
 	 * Load the Newbie Helper Buff list from SQL Table helper_buff_list
+	 * @param HelperBuffData 
+	 * @throws Exception 
 	 */
 	private void fillHelperBuffTable(ResultSet HelperBuffData) throws Exception
 	{
@@ -151,7 +153,7 @@ public class HelperBuffTable
 	}
 	
 	/**
-	 * Return the Helper Buff List
+	 * @return the Helper Buff List
 	 */
 	public List<L2HelperBuff> getHelperBuffTable()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/ItemTable.java

@@ -171,8 +171,7 @@ public class ItemTable
 	}
 	
 	/**
-	 * Returns a new object Item
-	 * @return
+	 * @return a new object Item
 	 */
 	public Item newItem()
 	{
@@ -212,6 +211,7 @@ public class ItemTable
 	
 	/**
 	 * Builds a variable in which all items are putting in in function of their ID.
+	 * @param size 
 	 */
 	private void buildFastLookupTable(int size)
 	{
@@ -362,12 +362,11 @@ public class ItemTable
 	 * <B><U> Actions</U> :</B><BR><BR>
 	 * <li>Sets L2ItemInstance parameters to be unusable </li>
 	 * <li>Removes the L2ItemInstance object to _allObjects of L2world </li>
-	 * <li>Logs Item delettion according to log settings</li><BR><BR>
-	 *
-	 * @param process : String Identifier of process triggering this action
-	 * @param itemId : int Item Identifier of the item to be created
-	 * @param actor : L2PcInstance Player requesting the item destroy
-	 * @param reference : Object Object referencing current action like NPC selling item or previous item in transformation
+	 * <li>Logs Item deletion according to log settings</li><BR><BR>
+	 * @param process a string identifier of process triggering this action.
+	 * @param item the item instance to be destroyed.
+	 * @param actor the player requesting the item destroy.
+	 * @param reference the object referencing current action like NPC selling item or previous item in transformation.
 	 */
 	public void destroyItem(String process, L2ItemInstance item, L2PcInstance actor, Object reference)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/LevelUpData.java

@@ -101,7 +101,7 @@ public class LevelUpData
 	}
 	
 	/**
-	 * @param template id
+	 * @param classId
 	 * @return
 	 */
 	public L2LvlupData getTemplate(int classId)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/MultiSell.java

@@ -94,6 +94,12 @@ public class MultiSell
 	 * 		  be at +0
 	 * 3) apply taxes: Uses the "taxIngredient" entry in order to add a certain amount of adena to the ingredients
 	 * 4) additional product and ingredient multipliers
+	 * @param listId 
+	 * @param player 
+	 * @param npc 
+	 * @param inventoryOnly 
+	 * @param productMultiplier 
+	 * @param ingredientMultiplier 
 	 */
 	public final void separateAndSend(int listId, L2PcInstance player, L2Npc npc, boolean inventoryOnly, double productMultiplier, double ingredientMultiplier)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/NpcTable.java

@@ -901,9 +901,9 @@ public class NpcTable
 		
 		return list.toArray(new L2NpcTemplate[list.size()]);
 	}
-	
+
 	/**
-	 * @param class1
+	 * @param clazz
 	 * @return
 	 */
 	public Set<Integer> getAllNpcOfL2jClass(Class<?> clazz)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/PetDataTable.java

@@ -290,8 +290,9 @@ public class PetDataTable
 	}
 	
 	/**
+	 * @param npcId 
+	 * @return 
 	 * @see L2PetData#getFood()
-	 * 
 	 */
 	@Deprecated
 	public static int[] getFoodItemId(int npcId)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/SkillTable.java

@@ -136,7 +136,9 @@ public class SkillTable
 	}
 	
 	/**
-	 * Returns an array with siege skills. If addNoble == true, will add also Advanced headquarters.
+	 * @param addNoble 
+	 * @param hasCastle 
+	 * @return an array with siege skills. If addNoble == true, will add also Advanced headquarters.
 	 */
 	public L2Skill[] getSiegeSkills(boolean addNoble, boolean hasCastle)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/SkillTreesData.java

@@ -753,7 +753,7 @@ public final class SkillTreesData
 	
 	/**
 	 * Some transformations are not available for some races.
-	 * @param playerthe transformation skill learning player.
+	 * @param player the transformation skill learning player.
 	 * @return all the available Transformation skills for a given {@code player}.
 	 */
 	public FastList<L2SkillLearn> getAvailableTransformSkills(L2PcInstance player)

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/SpawnTable.java

@@ -309,9 +309,10 @@ public class SpawnTable
 	
 	/**
 	 * Get all the spawn of a NPC<BR><BR>
-	 *
-	 * @param npcId : ID of the NPC to find.
-	 * @return
+	 * @param activeChar
+	 * @param npcId
+	 * @param teleportIndex
+	 * @param showposition
 	 */
 	public void findNPCInstances(L2PcInstance activeChar, int npcId, int teleportIndex, boolean showposition)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/SummonEffectsTable.java

@@ -87,7 +87,9 @@ public class SummonEffectsTable
 		}
 	}
 	
-	/** Common **/
+	/**
+	 * @return
+	 */
 	public static SummonEffectsTable getInstance()
 	{
 		return SingletonHolder._instance;

L2J_Server_BETA/java/com/l2jserver/gameserver/datatables/TeleportLocationTable.java

@@ -128,7 +128,7 @@ public class TeleportLocationTable
 	}
 	
 	/**
-	 * @param template id
+	 * @param id
 	 * @return
 	 */
 	public L2TeleportLocation getTemplate(int id)

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/IChatHandler.java

@@ -28,6 +28,7 @@ public interface IChatHandler
 	 * @param type
 	 * @param activeChar
 	 * @param target
+	 * @param text 
 	 */
 	public void handleChat(int type, L2PcInstance activeChar, String target, String text);
 	

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/IItemHandler.java

@@ -33,9 +33,9 @@ public interface IItemHandler
 	
 	/**
 	 * Launch task associated to the item.
-	 * @param playable : L2PlayableInstance designating the player
-	 * @param item : L2ItemInstance designating the item to use
-	 * @param forceUse: ctrl hold on item use
+	 * @param playable L2PlayableInstance designating the player
+	 * @param item L2ItemInstance designating the item to use
+	 * @param forceUse ctrl hold on item use
 	 */
 	public void useItem(L2Playable playable, L2ItemInstance item, boolean forceUse);
 }

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/ISkillHandler.java

@@ -34,9 +34,8 @@ public interface ISkillHandler
 	/**
 	 * this is the worker method that is called when using an item.
 	 * @param activeChar
-	 * @param item
-	 * @param target
-	 * @return count reduction after usage
+	 * @param skill
+	 * @param targets
 	 */
 	public void useSkill(L2Character activeChar, L2Skill skill, L2Object[] targets);
 	

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/IUserCommandHandler.java

@@ -29,8 +29,8 @@ public interface IUserCommandHandler
 	public static Logger _log = Logger.getLogger(IUserCommandHandler.class.getName());
 	/**
 	 * this is the worker method that is called when someone uses an admin command.
+	 * @param id
 	 * @param activeChar
-	 * @param command
 	 * @return command success
 	 */
 	public boolean useUserCommand(int id, L2PcInstance activeChar);

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/IVoicedCommandHandler.java

@@ -31,6 +31,7 @@ public interface IVoicedCommandHandler
 	 * this is the worker method that is called when someone uses an admin command.
 	 * @param activeChar
 	 * @param command
+	 * @param params 
 	 * @return command success
 	 */
 	public boolean useVoicedCommand(String command, L2PcInstance activeChar, String params);

L2J_Server_BETA/java/com/l2jserver/gameserver/handler/ItemHandler.java

@@ -67,7 +67,7 @@ public class ItemHandler
 	
 	/**
 	 * Returns the handler of the item
-	 * @param itemId : int designating the itemID
+	 * @param item
 	 * @return IItemHandler
 	 */
 	public IItemHandler getItemHandler(L2EtcItem item)

L2J_Server_BETA/java/com/l2jserver/gameserver/idfactory/IdFactory.java

@@ -316,8 +316,8 @@ public abstract class IdFactory
 	}
 	
 	/**
-	 * @param con
 	 * @return
+	 * @throws Exception 
 	 * @throws SQLException
 	 */
 	protected final int[] extractUsedObjectIDTable() throws Exception
@@ -404,8 +404,7 @@ public abstract class IdFactory
 	/**
 	 * return a used Object ID back to the pool
 	 * 
-	 * @param object
-	 *            ID
+	 * @param id
 	 */
 	public abstract void releaseId(int id);
 	

L2J_Server_BETA/java/com/l2jserver/gameserver/idfactory/StackIDFactory.java

@@ -145,7 +145,7 @@ public class StackIDFactory extends IdFactory
 	
 	/**
 	 * return a used Object ID back to the pool
-	 * @param object ID
+	 * @param id
 	 */
 	@Override
 	public synchronized void releaseId(int id)

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/AntiFeedManager.java

@@ -119,14 +119,13 @@ public class AntiFeedManager
 	}
 
 	/**
-	 * If number of all simultaneous connections from player's IP address lower than max
-	 * then increment connection count and return true.
-	 * Returns false if number of all simultaneous connections from player's IP address
-	 * higher than max.
+	 * 
 	 * @param eventId
 	 * @param player
 	 * @param max
-	 * @return
+	 * @return If number of all simultaneous connections from player's IP address lower than max
+	 * then increment connection count and return true.<br>
+	 * False if number of all simultaneous connections from player's IP address higher than max.
 	 */
 	public final boolean tryAddPlayer(int eventId, L2PcInstance player, int max)
 	{
@@ -134,14 +133,12 @@ public class AntiFeedManager
 	}
 
 	/**
-	 * If number of all simultaneous connections from player's IP address lower than max
-	 * then increment connection count and return true.
-	 * Returns false if number of all simultaneous connections from player's IP address
-	 * higher than max.
+	 * 
 	 * @param eventId
-	 * @param player
+	 * @param client
 	 * @param max
-	 * @return
+	 * @return If number of all simultaneous connections from player's IP address lower than max then increment connection count and return true.<br>
+	 * False if number of all simultaneous connections from player's IP address higher than max.
 	 */
 	public final boolean tryAddClient(int eventId, L2GameClient client, int max)
 	{
@@ -172,10 +169,9 @@ public class AntiFeedManager
 
 	/**
 	 * Decreasing number of active connection from player's IP address
-	 * Returns true if success and false if any problem detected.
 	 * @param eventId
 	 * @param player
-	 * @return
+	 * @return true if success and false if any problem detected.
 	 */
 	public final boolean removePlayer(int eventId, L2PcInstance player)
 	{
@@ -203,7 +199,7 @@ public class AntiFeedManager
 
 	/**
 	 * Remove player connection IP address from all registered events lists.
-	 * @param player
+	 * @param client
 	 */
 	public final void onDisconnect(L2GameClient client)
 	{
@@ -226,10 +222,9 @@ public class AntiFeedManager
 	}
 
 	/**
-	 * Returns maximum number of allowed connections (whitelist + max)
 	 * @param player
 	 * @param max
-	 * @return
+	 * @return maximum number of allowed connections (whitelist + max)
 	 */
 	public final int getLimit(L2PcInstance player, int max)
 	{
@@ -237,10 +232,9 @@ public class AntiFeedManager
 	}
 
 	/**
-	 * Returns maximum number of allowed connections (whitelist + max)
 	 * @param client
 	 * @param max
-	 * @return
+	 * @return maximum number of allowed connections (whitelist + max)
 	 */
 	public final int getLimit(L2GameClient client, int max)
 	{
@@ -257,8 +251,9 @@ public class AntiFeedManager
 		private int _num = 0;
 
 		/**
-		 * Returns true if successfully incremented number of connections
 		 * and false if maximum number is reached.
+		 * @param max 
+		 * @return true if successfully incremented number of connections
 		 */
 		public final synchronized boolean testAndIncrement(int max)
 		{
@@ -271,7 +266,7 @@ public class AntiFeedManager
 		}
 
 		/**
-		 * Returns true if all connections are removed
+		 * @return true if all connections are removed
 		 */
 		public final synchronized boolean testAndDecrement()
 		{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/AuctionManager.java

@@ -146,7 +146,10 @@ public class AuctionManager
 		return _auctions;
 	}
 	
-	/** Init Clan NPC aution */
+	/**
+	 * Init Clan NPC aution 
+	 * @param id
+	 */
 	public void initNPC(int id)
 	{
 		int i;

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/BoatManager.java

@@ -150,6 +150,9 @@ public class BoatManager
 	
 	/**
 	 * Broadcast one packet in both path points
+	 * @param point1 
+	 * @param point2 
+	 * @param packet 
 	 */
 	public void broadcastPacket(VehiclePathPoint point1, VehiclePathPoint point2, L2GameServerPacket packet)
 	{
@@ -159,6 +162,9 @@ public class BoatManager
 	
 	/**
 	 * Broadcast several packets in both path points
+	 * @param point1 
+	 * @param point2 
+	 * @param packets 
 	 */
 	public void broadcastPackets(VehiclePathPoint point1, VehiclePathPoint point2, L2GameServerPacket... packets)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/ClanHallManager.java

@@ -139,19 +139,25 @@ public final class ClanHallManager
 		return _allClanHalls;
 	}
 
-	/** Get Map with all FreeClanHalls */
+	/**
+	 * @return all FreeClanHalls
+	 */
 	public final Map<Integer, AuctionableHall> getFreeClanHalls()
 	{
 		return _freeClanHall;
 	}
 	
-	/** Get Map with all ClanHalls that have owner*/
+	/**
+	 * @return all ClanHalls that have owner
+	 */
 	public final Map<Integer, AuctionableHall> getClanHalls()
 	{
 		return _clanHall;
 	}
 	
-	/** Get Map with all ClanHalls*/
+	/**
+	 * @return all ClanHalls
+	 */
 	public final Map<Integer, AuctionableHall> getAllAuctionableClanHalls()
 	{
 		return _allAuctionableClanHalls;
@@ -162,7 +168,10 @@ public final class ClanHallManager
 		_allClanHalls.put(hall.getId(), hall);
 	}
 
-	/** Check is free ClanHall */
+	/**
+	 * @param chId 
+	 * @return true is free ClanHall
+	 */
 	public final boolean isFree(int chId)
 	{
 		if (_freeClanHall.containsKey(chId))
@@ -170,7 +179,10 @@ public final class ClanHallManager
 		return false;
 	}
 	
-	/** Free a ClanHall */
+	/**
+	 * Free a ClanHall 
+	 * @param chId
+	 */
 	public final synchronized void setFree(int chId)
 	{
 		_freeClanHall.put(chId, _clanHall.get(chId));
@@ -179,7 +191,11 @@ public final class ClanHallManager
 		_clanHall.remove(chId);
 	}
 	
-	/** Set ClanHallOwner */
+	/**
+	 * Set ClanHallOwner
+	 * @param chId 
+	 * @param clan
+	 */
 	public final synchronized void setOwner(int chId, L2Clan clan)
 	{
 		if (!_clanHall.containsKey(chId))
@@ -193,7 +209,10 @@ public final class ClanHallManager
 		_clanHall.get(chId).setOwner(clan);
 	}
 	
-	/** Get Clan Hall by Id */
+	/**
+	 * @param clanHallId 
+	 * @return Clan Hall by Id
+	 */
 	public final ClanHall getClanHallById(int clanHallId)
 	{
 		return _allClanHalls.get(clanHallId);
@@ -203,9 +222,13 @@ public final class ClanHallManager
 	{
 		return _allAuctionableClanHalls.get(clanHallId);
 	}
-
 	
-	/** Get Clan Hall by x,y,z */
+	/**
+	 * @param x 
+	 * @param y 
+	 * @param z 
+	 * @return Clan Hall by x,y,z 
+	 */
 	public final ClanHall getClanHall(int x, int y, int z)
 	{
 		for (ClanHall temp : getAllClanHalls().values())
@@ -252,7 +275,10 @@ public final class ClanHallManager
 		return null;
 	}
 	
-	/** Get Clan Hall by Owner */
+	/**
+	 * @param clan 
+	 * @return Clan Hall by Owner
+	 */
 	public final AuctionableHall getClanHallByOwner(L2Clan clan)
 	{
 		for (Map.Entry<Integer, AuctionableHall> ch : _clanHall.entrySet())

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/DayNightSpawnManager.java

@@ -83,10 +83,10 @@ public class DayNightSpawnManager
 	
 	/**
 	 * Manage Spawn/Respawn
-	 * Arg 1 : List with spawns must be unspawned
-	 * Arg 2 : List with spawns must be spawned
-	 * Arg 3 : String for log info for unspawned L2NpcInstance
-	 * Arg 4 : String for log info for spawned L2NpcInstance
+	 * @param unSpawnCreatures List with spawns must be unspawned
+	 * @param spawnCreatures List with spawns must be spawned
+	 * @param UnspawnLogInfo String for log info for unspawned L2NpcInstance
+	 * @param SpawnLogInfo String for log info for spawned L2NpcInstance
 	 */
 	private void spawnCreatures(List<L2Spawn> unSpawnCreatures, List<L2Spawn> spawnCreatures, String UnspawnLogInfo,
 			String SpawnLogInfo)

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/DuelManager.java

@@ -153,9 +153,9 @@ public class DuelManager
 	}
 	
 	/**
-	 * Registers a debuff which will be removed if the duel ends
+	 * Registers a buff which will be removed if the duel ends
 	 * @param player
-	 * @param debuff
+	 * @param buff
 	 */
 	public void onBuff(L2PcInstance player, L2Effect buff)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/FortSiegeManager.java

@@ -84,8 +84,9 @@ public class FortSiegeManager
 	}
 	
 	/**
-	 * Return true if character summon<BR><BR>
 	 * @param activeChar The L2Character of the character can summon
+	 * @param isCheckOnly 
+	 * @return true if character summon
 	 */
 	public final boolean checkIfOkToSummon(L2Character activeChar, boolean isCheckOnly)
 	{
@@ -111,8 +112,9 @@ public class FortSiegeManager
 	}
 	
 	/**
-	 * Return true if the clan is registered or owner of a fort<BR><BR>
 	 * @param clan The L2Clan of the player
+	 * @param fortid 
+	 * @return true if the clan is registered or owner of a fort
 	 */
 	public final boolean checkIsRegistered(L2Clan clan, int fortid)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/GrandBossManager.java

@@ -296,6 +296,7 @@ public class GrandBossManager
 	
 	/**
 	 * Adds a L2GrandBossInstance to the list of bosses.
+	 * @param boss 
 	 */
 	public void addBoss(L2GrandBossInstance boss)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/HandysBlockCheckerManager.java

@@ -147,6 +147,7 @@ public final class HandysBlockCheckerManager
 	 * arena manager) and send the needed server ->  client packets
 	 * @param player
 	 * @param arenaId
+	 * @return 
 	 */
 	public boolean addPlayerToArena(L2PcInstance player, int arenaId)
 	{
@@ -224,6 +225,7 @@ public final class HandysBlockCheckerManager
 	 * his team mates / enemy team mates
 	 * @param player
 	 * @param arenaId
+	 * @param team 
 	 */
 	public void removePlayer(L2PcInstance player, int arenaId, int team)
 	{
@@ -327,7 +329,7 @@ public final class HandysBlockCheckerManager
 	/**
 	 * Called when played logs out while participating
 	 * in Block Checker Event
-	 * @param L2PcInstance player
+	 * @param player
 	 */
 	public void onDisconnect(L2PcInstance player)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/HellboundManager.java

@@ -37,6 +37,7 @@ public class HellboundManager
 	
 	/**
 	 * Returns true if Hellbound is locked
+	 * @return 
 	 */
 	public boolean isLocked()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/MapRegionManager.java

@@ -256,8 +256,8 @@ public class MapRegionManager
 	
 	/**
 	 * Get town name by character position
-	 * @param townId
-	 * @return String
+	 * @param activeChar
+	 * @return
 	 */
 	public String getClosestTownName(L2Character activeChar)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/MercTicketManager.java

@@ -216,6 +216,8 @@ public class MercTicketManager
 	/**
 	 * Checks if the passed item has reached the limit of number of dropped
 	 * tickets that this SPECIFIC item may have in its castle
+	 * @param itemId 
+	 * @return 
 	 */
 	public boolean isAtTypeLimit(int itemId)
 	{
@@ -245,6 +247,8 @@ public class MercTicketManager
 	/**
 	 * Checks if the passed item belongs to a castle which has reached its limit
 	 * of number of dropped tickets.
+	 * @param itemId 
+	 * @return 
 	 */
 	public boolean isAtCasleLimit(int itemId)
 	{
@@ -292,6 +296,10 @@ public class MercTicketManager
 	 * 3) create a copy of the item to drop in the world
 	 * returns the id of the mercenary npc that was added to the spawn
 	 * returns -1 if this fails.
+	 * @param itemId 
+	 * @param activeChar 
+	 * @param messages 
+	 * @return 
 	 */
 	public int addTicket(int itemId, L2PcInstance activeChar, String[] messages)
 	{
@@ -349,6 +357,7 @@ public class MercTicketManager
 	/**
 	 * Delete all tickets from a castle;
 	 * remove the items from the world and remove references to them from this class
+	 * @param castleId 
 	 */
 	public void deleteTickets(int castleId)
 	{
@@ -368,6 +377,7 @@ public class MercTicketManager
 	/**
 	 * remove a single ticket and its associated spawn from the world
 	 * (used when the castle lord picks up a ticket, for example)
+	 * @param item 
 	 */
 	public void removeTicket(L2ItemInstance item)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/SiegeGuardManager.java

@@ -50,7 +50,9 @@ public class SiegeGuardManager
 	// =========================================================
 	// Method - Public
 	/**
-	 * Add guard.<BR><BR>
+	 * Add guard.
+	 * @param activeChar 
+	 * @param npcId 
 	 */
 	public void addSiegeGuard(L2PcInstance activeChar, int npcId)
 	{
@@ -60,7 +62,12 @@ public class SiegeGuardManager
 	}
 	
 	/**
-	 * Add guard.<BR><BR>
+	 * Add guard.
+	 * @param x 
+	 * @param y 
+	 * @param z 
+	 * @param heading 
+	 * @param npcId 
 	 */
 	public void addSiegeGuard(int x, int y, int z, int heading, int npcId)
 	{
@@ -68,7 +75,9 @@ public class SiegeGuardManager
 	}
 	
 	/**
-	 * Hire merc.<BR><BR>
+	 * Hire merc.
+	 * @param activeChar 
+	 * @param npcId 
 	 */
 	public void hireMerc(L2PcInstance activeChar, int npcId)
 	{
@@ -78,7 +87,12 @@ public class SiegeGuardManager
 	}
 	
 	/**
-	 * Hire merc.<BR><BR>
+	 * Hire merc.
+	 * @param x 
+	 * @param y 
+	 * @param z 
+	 * @param heading 
+	 * @param npcId 
 	 */
 	public void hireMerc(int x, int y, int z, int heading, int npcId)
 	{
@@ -88,6 +102,10 @@ public class SiegeGuardManager
 	/**
 	 * Remove a single mercenary, identified by the npcId and location.
 	 * Presumably, this is used when a castle lord picks up a previously dropped ticket
+	 * @param npcId 
+	 * @param x 
+	 * @param y 
+	 * @param z 
 	 */
 	public void removeMerc(int npcId, int x, int y, int z)
 	{
@@ -242,6 +260,12 @@ public class SiegeGuardManager
 	
 	/**
 	 * Save guards.<BR><BR>
+	 * @param x 
+	 * @param y 
+	 * @param z 
+	 * @param heading 
+	 * @param npcId 
+	 * @param isHire 
 	 */
 	private void saveSiegeGuard(int x, int y, int z, int heading, int npcId, int isHire)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/SiegeManager.java

@@ -86,8 +86,9 @@ public class SiegeManager
 	}
 	
 	/**
-	 * Return true if character summon<BR><BR>
 	 * @param activeChar The L2Character of the character can summon
+	 * @param isCheckOnly 
+	 * @return true if character summon
 	 */
 	public final boolean checkIfOkToSummon(L2Character activeChar, boolean isCheckOnly)
 	{
@@ -113,8 +114,9 @@ public class SiegeManager
 	}
 	
 	/**
-	 * Return true if the clan is registered or owner of a castle<BR><BR>
 	 * @param clan The L2Clan of the player
+	 * @param castleid 
+	 * @return true if the clan is registered or owner of a castle
 	 */
 	public final boolean checkIsRegistered(L2Clan clan, int castleid)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/TerritoryWarManager.java

@@ -166,8 +166,9 @@ public class TerritoryWarManager implements Siegable
 	}
 	
 	/**
-	 * Return true if the clan is registered<BR><BR>
+	 * @param castleId 
 	 * @param clan The L2Clan of the player
+	 * @return true if the clan is registered
 	 */
 	public final boolean checkIsRegistered(int castleId, L2Clan clan)
 	{
@@ -188,8 +189,9 @@ public class TerritoryWarManager implements Siegable
 	}
 	
 	/**
-	 * Return true if the player is registered<BR><BR>
-	 * @param integer The objectId of the player
+	 * @param castleId
+	 * @param objId
+	 * @return true if the player is registered
 	 */
 	public final boolean checkIsRegistered(int castleId, int objId)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/instancemanager/ZoneManager.java

@@ -65,6 +65,9 @@ public class ZoneManager
 	private int _lastDynamicId = 300000;
 	private List<L2ItemInstance> _debugItems;
 	
+	/**
+	 * @return
+	 */
 	public static final ZoneManager getInstance()
 	{
 		return SingletonHolder._instance;
@@ -446,7 +449,8 @@ public class ZoneManager
 	
 	/**
 	 * Add new zone
-	 *
+	 * @param <T> 
+	 * @param id 
 	 * @param zone
 	 */
 	@SuppressWarnings("unchecked")
@@ -511,6 +515,7 @@ public class ZoneManager
 	
 	/**
 	 * Get zone by ID and zone class
+	 * @param <T> 
 	 * @param id
 	 * @param zoneType
 	 * @return zone
@@ -533,11 +538,10 @@ public class ZoneManager
 	}
 	
 	/**
-	 * Returns zone from where the object is located by type
-	 *
+	 * @param <T> 
 	 * @param object
 	 * @param type
-	 * @return zone
+	 * @return zone from where the object is located by type
 	 */
 	public <T extends L2ZoneType> T getZone(L2Object object, Class<T> type)
 	{
@@ -586,13 +590,12 @@ public class ZoneManager
 	}
 	
 	/**
-	 * Returns zone from given coordinates
-	 * 
+	 * @param <T> 
 	 * @param x
 	 * @param y
 	 * @param z
 	 * @param type
-	 * @return zone
+	 * @return zone from given coordinates
 	 */
 	@SuppressWarnings("unchecked")
 	public <T extends L2ZoneType> T getZone(int x, int y, int z, Class<T> type)

L2J_Server_BETA/java/com/l2jserver/gameserver/model/AutoChatHandler.java

@@ -146,9 +146,9 @@ public class AutoChatHandler implements SpawnListener
 	 * <BR>
 	 * Returns the associated auto chat instance.
 	 *
-	 * @param int npcId
-	 * @param String[] chatTexts
-	 * @param int chatDelay (-1 = default delay)
+	 * @param npcId
+	 * @param chatTexts
+	 * @param chatDelay (-1 = default delay)
 	 * @return AutoChatInstance chatInst
 	 */
 	public AutoChatInstance registerGlobalChat(int npcId, String[] chatTexts, long chatDelay)
@@ -163,9 +163,9 @@ public class AutoChatHandler implements SpawnListener
 	 * <BR>
 	 * Returns the associated auto chat instance.
 	 *
-	 * @param L2Npc npcInst
-	 * @param String[] chatTexts
-	 * @param int chatDelay (-1 = default delay)
+	 * @param npcInst
+	 * @param chatTexts
+	 * @param chatDelay (-1 = default delay)
 	 * @return AutoChatInstance chatInst
 	 */
 	public AutoChatInstance registerChat(L2Npc npcInst, String[] chatTexts, long chatDelay)
@@ -197,8 +197,8 @@ public class AutoChatHandler implements SpawnListener
 	 * Removes and cancels ALL auto chat definition for the given NPC ID,
 	 * and removes its chat instance if it exists.
 	 *
-	 * @param int npcId
-	 * @return boolean removedSuccessfully
+	 * @param npcId
+	 * @return removedSuccessfully
 	 */
 	public boolean removeChat(int npcId)
 	{
@@ -209,8 +209,8 @@ public class AutoChatHandler implements SpawnListener
 	/**
 	 * Removes and cancels ALL auto chats for the given chat instance.
 	 *
-	 * @param AutoChatInstance chatInst
-	 * @return boolean removedSuccessfully
+	 * @param chatInst
+	 * @return {@code true} if the chat is removed successfully.
 	 */
 	public boolean removeChat(AutoChatInstance chatInst)
 	{
@@ -230,8 +230,8 @@ public class AutoChatHandler implements SpawnListener
 	 * Returns the associated auto chat instance either by the given NPC ID
 	 * or object ID.
 	 *
-	 * @param int id
-	 * @param boolean byObjectId
+	 * @param id
+	 * @param byObjectId
 	 * @return AutoChatInstance chatInst
 	 */
 	public AutoChatInstance getAutoChatInstance(int id, boolean byObjectId)
@@ -250,7 +250,7 @@ public class AutoChatHandler implements SpawnListener
 	 * Sets the active state of all auto chat instances to that specified,
 	 * and cancels the scheduled chat task if necessary.
 	 *
-	 * @param boolean isActive
+	 * @param isActive
 	 */
 	public void setAutoChatActive(boolean isActive)
 	{
@@ -264,6 +264,7 @@ public class AutoChatHandler implements SpawnListener
 	 * <BR><BR>
 	 * If an auto chat instance is set to be "global", all instances matching the registered
 	 * NPC ID will be added to that chat instance.
+	 * @param npc 
 	 */
 	public void npcSpawned(L2Npc npc)
 	{
@@ -335,9 +336,8 @@ public class AutoChatHandler implements SpawnListener
 		 * to the created chat definition.
 		 * <BR>
 		 * <B>Note</B>: Uses pre-defined default values for texts and chat delays from the chat instance.
-		 *
-		 * @param L2Npc npcInst
-		 * @return int objectId
+		 * @param npcInst
+		 * @return
 		 */
 		public int addChatDefinition(L2Npc npcInst)
 		{
@@ -351,10 +351,10 @@ public class AutoChatHandler implements SpawnListener
 		 * Returns the object ID for the NPC instance, with which to refer
 		 * to the created chat definition.
 		 *
-		 * @param L2Npc npcInst
-		 * @param String[] chatTexts
-		 * @param int chatDelay
-		 * @return int objectId
+		 * @param npcInst
+		 * @param chatTexts
+		 * @param chatDelay
+		 * @return objectId
 		 */
 		public int addChatDefinition(L2Npc npcInst, String[] chatTexts, long chatDelay)
 		{
@@ -369,8 +369,8 @@ public class AutoChatHandler implements SpawnListener
 		/**
 		 * Removes a chat definition specified by the given object ID.
 		 *
-		 * @param int objectId
-		 * @return boolean removedSuccessfully
+		 * @param objectId
+		 * @return {@code true} if the chat is removed successfully.
 		 */
 		public boolean removeChatDefinition(int objectId)
 		{
@@ -418,7 +418,7 @@ public class AutoChatHandler implements SpawnListener
 		
 		/**
 		 * Tests if the auto chat definition given by its object ID is set to be random.
-		 *
+		 * @param objectId 
 		 * @return boolean isRandom
 		 */
 		public boolean isRandomChat(int objectId)
@@ -466,6 +466,7 @@ public class AutoChatHandler implements SpawnListener
 		
 		/**
 		 * A series of methods used to get and set default values for new chat definitions.
+		 * @return 
 		 */
 		public long getDefaultDelay()
 		{
@@ -495,8 +496,8 @@ public class AutoChatHandler implements SpawnListener
 		/**
 		 * Sets a specific chat delay for the specified auto chat definition given by its object ID.
 		 *
-		 * @param int objectId
-		 * @param long delayValue
+		 * @param objectId
+		 * @param delayValue
 		 */
 		public void setChatDelay(int objectId, long delayValue)
 		{
@@ -509,8 +510,8 @@ public class AutoChatHandler implements SpawnListener
 		/**
 		 * Sets a specific set of chat texts for the specified auto chat definition given by its object ID.
 		 *
-		 * @param int objectId
-		 * @param String[] textsValue
+		 * @param objectId
+		 * @param textsValue
 		 */
 		public void setChatTexts(int objectId, String[] textsValue)
 		{
@@ -523,8 +524,8 @@ public class AutoChatHandler implements SpawnListener
 		/**
 		 * Sets specifically to use random chat order for the auto chat definition given by its object ID.
 		 *
-		 * @param int objectId
-		 * @param boolean randValue
+		 * @param objectId
+		 * @param randValue
 		 */
 		public void setRandomChat(int objectId, boolean randValue)
 		{
@@ -537,7 +538,7 @@ public class AutoChatHandler implements SpawnListener
 		/**
 		 * Sets the activity of ALL auto chat definitions handled by this chat instance.
 		 *
-		 * @param boolean isActive
+		 * @param activeValue
 		 */
 		public void setActive(boolean activeValue)
 		{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/AutoSpawnHandler.java

@@ -183,11 +183,11 @@ public class AutoSpawnHandler
 	 * it as active. Returns a AutoSpawnInstance containing info about the
 	 * spawn.
 	 *
-	 * @param int npcId
-	 * @param int[][] spawnPoints
-	 * @param int initialDelay (If < 0 = default value)
-	 * @param int respawnDelay (If < 0 = default value)
-	 * @param int despawnDelay (If < 0 = default value or if = 0, function disabled)
+	 * @param npcId
+	 * @param spawnPoints
+	 * @param initialDelay (If < 0 = default value)
+	 * @param respawnDelay (If < 0 = default value)
+	 * @param despawnDelay (If < 0 = default value or if = 0, function disabled)
 	 * @return AutoSpawnInstance spawnInst
 	 */
 	public AutoSpawnInstance registerSpawn(int npcId, int[][] spawnPoints, int initialDelay, int respawnDelay, int despawnDelay)
@@ -226,10 +226,10 @@ public class AutoSpawnHandler
 	 * <B>Warning:</B> Spawn locations must be specified separately using
 	 * addSpawnLocation().
 	 *
-	 * @param int npcId
-	 * @param int initialDelay (If < 0 = default value)
-	 * @param int respawnDelay (If < 0 = default value)
-	 * @param int despawnDelay (If < 0 = default value or if = 0, function disabled)
+	 * @param npcId
+	 * @param initialDelay (If < 0 = default value)
+	 * @param respawnDelay (If < 0 = default value)
+	 * @param despawnDelay (If < 0 = default value or if = 0, function disabled)
 	 * @return AutoSpawnInstance spawnInst
 	 */
 	public AutoSpawnInstance registerSpawn(int npcId, int initialDelay, int respawnDelay, int despawnDelay)
@@ -241,7 +241,7 @@ public class AutoSpawnHandler
 	 * Remove a registered spawn from the list, specified by the given spawn
 	 * instance.
 	 *
-	 * @param AutoSpawnInstance spawnInst
+	 * @param spawnInst
 	 * @return boolean removedSuccessfully
 	 */
 	public boolean removeSpawn(AutoSpawnInstance spawnInst)
@@ -275,8 +275,7 @@ public class AutoSpawnHandler
 	 * Remove a registered spawn from the list, specified by the given spawn
 	 * object ID.
 	 *
-	 * @param int objectId
-	 * @return boolean removedSuccessfully
+	 * @param objectId
 	 */
 	public void removeSpawn(int objectId)
 	{
@@ -286,8 +285,8 @@ public class AutoSpawnHandler
 	/**
 	 * Sets the active state of the specified spawn.
 	 *
-	 * @param AutoSpawnInstance spawnInst
-	 * @param boolean isActive
+	 * @param spawnInst
+	 * @param isActive
 	 */
 	public void setSpawnActive(AutoSpawnInstance spawnInst, boolean isActive)
 	{
@@ -330,8 +329,7 @@ public class AutoSpawnHandler
 	 * Sets the active state of all auto spawn instances to that specified, and
 	 * cancels the scheduled spawn task if necessary.
 	 *
-	 * @param boolean
-	 *			isActive
+	 * @param isActive
 	 */
 	public void setAllActive(boolean isActive)
 	{
@@ -345,11 +343,10 @@ public class AutoSpawnHandler
 	}
 	
 	/**
-	 * Returns the number of milliseconds until the next occurrance of the given
-	 * spawn.
+	 * Returns the number of milliseconds until the next occurrence of the given spawn.
 	 *
-	 * @param AutoSpawnInstance spawnInst
-	 * @param long milliRemaining
+	 * @param spawnInst
+	 * @return 
 	 */
 	public final long getTimeToNextSpawn(AutoSpawnInstance spawnInst)
 	{
@@ -367,8 +364,8 @@ public class AutoSpawnHandler
 	 * Note: If isObjectId == false, returns first instance for the specified
 	 * NPC ID.
 	 *
-	 * @param int id
-	 * @param boolean isObjectId
+	 * @param id
+	 * @param isObjectId
 	 * @return AutoSpawnInstance spawnInst
 	 */
 	public final AutoSpawnInstance getAutoSpawnInstance(int id, boolean isObjectId)
@@ -401,8 +398,8 @@ public class AutoSpawnHandler
 	/**
 	 * Tests if the specified object ID is assigned to an auto spawn.
 	 *
-	 * @param int objectId
-	 * @return boolean isAssigned
+	 * @param objectId
+	 * @return isAssigned
 	 */
 	public final boolean isSpawnRegistered(int objectId)
 	{
@@ -412,7 +409,7 @@ public class AutoSpawnHandler
 	/**
 	 * Tests if the specified spawn instance is assigned to an auto spawn.
 	 *
-	 * @param AutoSpawnInstance spawnInst
+	 * @param spawnInst
 	 * @return boolean isAssigned
 	 */
 	public final boolean isSpawnRegistered(AutoSpawnInstance spawnInst)

L2J_Server_BETA/java/com/l2jserver/gameserver/model/CharEffectList.java

@@ -231,7 +231,7 @@ public class CharEffectList
 	
 	/**
 	 * Returns the first effect matching the given skillId
-	 * @param index
+	 * @param skillId
 	 * @return
 	 */
 	public final L2Effect getFirstEffect(int skillId)
@@ -469,6 +469,7 @@ public class CharEffectList
 	/**
 	 * Exits all effects created by a specific skill type
 	 * @param skillType skill type
+	 * @param negateLvl 
 	 */
 	public final void stopSkillEffects(L2SkillType skillType, int negateLvl)
 	{
@@ -998,6 +999,7 @@ public class CharEffectList
 	
 	/**
 	 * Remove all passive effects held by this <b>skillId</b>.
+	 * @param skillId 
 	 */
 	public void removePassiveEffects(int skillId)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/IChanceSkillTrigger.java

@@ -24,21 +24,25 @@ public interface IChanceSkillTrigger
 {
 	/**
 	 * Just a flag
+	 * @return 
 	 */
 	public boolean triggersChanceSkill();
 	
 	/**
 	 * Triggered Id
+	 * @return 
 	 */
 	public int getTriggeredChanceId();
 	
 	/**
 	 * Triggered level
+	 * @return 
 	 */
 	public int getTriggeredChanceLevel();
 	
 	/**
 	 * Chance condition object
+	 * @return 
 	 */
 	public ChanceCondition getTriggeredChanceCondition();
 }

L2J_Server_BETA/java/com/l2jserver/gameserver/model/ItemInfo.java

@@ -62,7 +62,7 @@ public class ItemInfo
 	
 	/**
 	 * Get all information from L2ItemInstance to generate ItemInfo.<BR><BR>
-	 *
+	 * @param item 
 	 */
 	public ItemInfo(L2ItemInstance item)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2AccessLevel.java

@@ -154,8 +154,7 @@ public class L2AccessLevel
 	
 	/**
 	 * Retruns if the access level is allowed to use fixed res or not<br><br>
-	 * 
-	 * @return: true if the access level is allowed to use fixed res, otherwise false<br>
+	 * @return true if the access level is allowed to use fixed res, otherwise false<br>
 	 */
 	public boolean allowFixedRes()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2AdminCommandAccessRight.java

@@ -32,6 +32,7 @@ public class L2AdminCommandAccessRight
 	 * 
 	 * @param adminCommand as String
 	 * @param accessLevels as String
+	 * @param confirm 
 	 */
 	public L2AdminCommandAccessRight(String adminCommand, String accessLevels, boolean confirm)
 	{
@@ -69,7 +70,7 @@ public class L2AdminCommandAccessRight
 	/**
 	 * Checks if the given characterAccessLevel is allowed to use the admin command which belongs to this access right<br><br>
 	 * 
-	 * @param characterAccessLevel<br><br>
+	 * @param characterAccessLevel
 	 * 
 	 * @return boolean: true if characterAccessLevel is allowed to use the admin command which belongs to this access right, otherwise false<br>
 	 */

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2ArmorSet.java

@@ -163,10 +163,8 @@ public final class L2ArmorSet
 	}
 	
 	/**
-	 * Checks if all parts of set are enchanted to +6 or more
-	 * 
 	 * @param player
-	 * @return
+	 * @return true if all parts of set are enchanted to +6 or more
 	 */
 	public boolean isEnchanted6(L2PcInstance player)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2CharPosition.java

@@ -23,7 +23,11 @@ public final class L2CharPosition
 	public final int x, y, z, heading;
 	
 	/**
-	 * Constructor of L2CharPosition.<BR><BR>
+	 * Constructor of L2CharPosition.
+	 * @param pX 
+	 * @param pY 
+	 * @param pZ 
+	 * @param pHeading 
 	 */
 	public L2CharPosition(int pX, int pY, int pZ, int pHeading)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Clan.java

@@ -234,7 +234,7 @@ public class L2Clan
 	}
 	
 	/**
-	 * @param leaderId The leaderId to set.
+	 * @param leader the leader to set.
 	 */
 	public void setLeader(L2ClanMember leader)
 	{
@@ -1182,7 +1182,9 @@ public class L2Clan
 		}
 	}
 	
-	/** used to retrieve all skills */
+	/**
+	 * @return all the clan skills.
+	 */
 	public final L2Skill[] getAllSkills()
 	{
 		if (_skills == null)
@@ -1191,7 +1193,11 @@ public class L2Clan
 		return _skills.values().toArray(new L2Skill[_skills.values().size()]);
 	}
 	
-	/** used to add a skill to skill list of this L2Clan */
+	/** 
+	 * Used to add a skill to skill list of this L2Clan 
+	 * @param newSkill 
+	 * @return
+	 */
 	public L2Skill addSkill(L2Skill newSkill)
 	{
 		L2Skill oldSkill = null;
@@ -1210,7 +1216,12 @@ public class L2Clan
 		return addNewSkill(newSkill, -2);
 	}
 	
-	/** used to add a new skill to the list, send a packet to all online clan members, update their stats and store it in db*/
+	/**
+	 * Used to add a new skill to the list, send a packet to all online clan members, update their stats and store it in db
+	 * @param newSkill 
+	 * @param subType 
+	 * @return
+	 */
 	public L2Skill addNewSkill(L2Skill newSkill, int subType)
 	{
 		L2Skill oldSkill = null;
@@ -1641,7 +1652,11 @@ public class L2Clan
 		}
 	}
 	
-	/** used to retrieve subPledge by type */
+	/**
+	 * used to retrieve subPledge by type 
+	 * @param pledgeType 
+	 * @return
+	 */
 	public final SubPledge getSubPledge(int pledgeType)
 	{
 		if (_subPledges == null)
@@ -1650,7 +1665,11 @@ public class L2Clan
 		return _subPledges.get(pledgeType);
 	}
 	
-	/** used to retrieve subPledge by type */
+	/**
+	 * Used to retrieve subPledge by type 
+	 * @param pledgeName 
+	 * @return
+	 */
 	public final SubPledge getSubPledge(String pledgeName)
 	{
 		if (_subPledges == null)
@@ -1666,7 +1685,10 @@ public class L2Clan
 		return null;
 	}
 	
-	/** used to retrieve all subPledges */
+	/**
+	 * Used to retrieve all subPledges 
+	 * @return
+	 */
 	public final SubPledge[] getAllSubPledges()
 	{
 		if (_subPledges == null)
@@ -2706,6 +2728,7 @@ public class L2Clan
 	/**
 	 * Change the ally crest. If crest id is 0, crest is removed. New crest id is saved to database.
 	 * @param crestId if 0, crest is removed, else new crest id is set and saved to database
+	 * @param onlyThisClan 
 	 */
 	public void changeAllyCrest(int crestId, boolean onlyThisClan)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2CommandChannel.java

@@ -42,8 +42,7 @@ public class L2CommandChannel
 	/**
 	 * Creates a New Command Channel and Add the Leaders party to the CC
 	 *
-	 * @param CommandChannelLeader
-	 *
+	 * @param leader
 	 */
 	public L2CommandChannel(L2PcInstance leader)
 	{
@@ -58,7 +57,7 @@ public class L2CommandChannel
 	
 	/**
 	 * Adds a Party to the Command Channel
-	 * @param Party
+	 * @param party
 	 */
 	public void addParty(L2Party party)
 	{
@@ -77,7 +76,7 @@ public class L2CommandChannel
 	
 	/**
 	 * Removes a Party from the Command Channel
-	 * @param Party
+	 * @param party
 	 */
 	public void removeParty(L2Party party)
 	{
@@ -136,8 +135,8 @@ public class L2CommandChannel
 	}
 	
 	/**
-	 * Broadcast packet to every channelmember
-	 * @param L2GameServerPacket
+	 * Broadcast packet to every channel member
+	 * @param gsp
 	 */
 	public void broadcastToChannelMembers(L2GameServerPacket gsp)
 	{
@@ -192,7 +191,7 @@ public class L2CommandChannel
 	public int getLevel() { return _channelLvl; }
 	
 	/**
-	 * @param sets the leader of the Command Channel
+	 * @param leader the leader of the Command Channel
 	 */
 	public void setChannelLeader(L2PcInstance leader)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2DropCategory.java

@@ -185,6 +185,7 @@ public class L2DropCategory
 	 * This change doesn't affect calculation when drop_chance * RATE_DROP_ITEMS < 100 %,
 	 * meaning there are no big changes for low rate servers and no changes at all for 1x
 	 * servers.
+	 * @param raid 
 	 *
 	 * @return selected drop from category, or null if nothing is dropped.
 	 */

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2DropData.java

@@ -135,7 +135,7 @@ public class L2DropData
 	
 	/**
 	 * Sets the questID
-	 * @param String designating the questID to set.
+	 * @param questID the quest Id to set.
 	 */
 	public void setQuestID(String questID)
 	{

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

@@ -149,8 +149,10 @@ public abstract class L2Effect
 	private L2SkillType _effectSkillType;
 	
 	/**
-	 * <font color="FF0000"><b>WARNING: scheduleEffect nolonger inside constructor</b></font><br>
+	 * <font color="FF0000"><b>WARNING: scheduleEffect no longer inside constructor</b></font><br>
 	 * So you must call it explicitly
+	 * @param env 
+	 * @param template 
 	 */
 	protected L2Effect(Env env, EffectTemplate template)
 	{
@@ -429,10 +431,15 @@ public abstract class L2Effect
 		}
 	}
 	
-	/** returns effect type */
+	/**
+	 * @return effect type
+	 */
 	public abstract L2EffectType getEffectType();
 	
-	/** Notify started */
+	/**
+	 * Notify started 
+	 * @return
+	 */
 	public boolean onStart()
 	{
 		if (_abnormalEffect != AbnormalEffect.NULL)
@@ -457,7 +464,9 @@ public abstract class L2Effect
 			getEffected().getActingPlayer().stopEventEffect(_eventEffect);
 	}
 	
-	/** Return true for continuation of this effect */
+	/**
+	 * @return true for continuation of this effect
+	 */
 	public abstract boolean onActionTime();
 	
 	public final void scheduleEffect()

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2ItemInstance.java

@@ -699,8 +699,8 @@ public final class L2ItemInstance extends L2Object
 	}
 	
 	/**
-	 * Returns if item can be deposited in warehouse or freight
-	 * @return boolean
+	 * @param isPrivateWareHouse 
+	 * @return if item can be deposited in warehouse or freight
 	 */
 	public boolean isDepositable(boolean isPrivateWareHouse)
 	{
@@ -761,8 +761,10 @@ public final class L2ItemInstance extends L2Object
 	}
 	
 	/**
-	 * Returns if item is available for manipulation
-	 * @return boolean
+	 * @param player 
+	 * @param allowAdena 
+	 * @param allowNonTradeable 
+	 * @return if item is available for manipulation
 	 */
 	public boolean isAvailable(L2PcInstance player, boolean allowAdena, boolean allowNonTradeable)
 	{
@@ -790,8 +792,7 @@ public final class L2ItemInstance extends L2Object
 	}
 	
 	/**
-	 * Sets the level of enchantment of the item
-	 * @param int
+	 * @param enchantLevel the enchant value to set
 	 */
 	public void setEnchantLevel(int enchantLevel)
 	{
@@ -1197,7 +1198,7 @@ public final class L2ItemInstance extends L2Object
 	 * sends a inventory update
 	 * schedules a new consumption task if non is running
 	 * optionally one could force a new task
-	 * @param forces a new consumption task if item is equipped
+	 * @param resetConsumingMana if true forces a new consumption task if item is equipped
 	 */
 	public void decreaseMana(boolean resetConsumingMana)
 	{
@@ -1209,7 +1210,7 @@ public final class L2ItemInstance extends L2Object
 	 * sends a inventory update
 	 * schedules a new consumption task if non is running
 	 * optionally one could force a new task
-	 * @param forces a new consumption task if item is equipped
+	 * @param resetConsumingMana if forces a new consumption task if item is equipped
 	 * @param count how much mana decrease
 	 */
 	public void decreaseMana(boolean resetConsumingMana, int count)
@@ -1438,7 +1439,8 @@ public final class L2ItemInstance extends L2Object
 	
 	/**
 	 * Returns a L2ItemInstance stored in database from its objectID
-	 * @param objectId : int designating the objectID of the item
+	 * @param ownerId
+	 * @param rs 
 	 * @return L2ItemInstance
 	 */
 	public static L2ItemInstance restoreFromDb(int ownerId, ResultSet rs)

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2LvlupData.java

@@ -95,7 +95,7 @@ public class L2LvlupData
 	}
 	
 	/**
-	 * @param hpAdd The _classCpAdd to set.
+	 * @param cpAdd The _classCpAdd to set.
 	 */
 	public void setClassCpAdd(float cpAdd)
 	{
@@ -112,7 +112,7 @@ public class L2LvlupData
 	}
 	
 	/**
-	 * @param hpBase The _classCpBase to set.
+	 * @param cpBase The _classCpBase to set.
 	 */
 	public void setClassCpBase(float cpBase)
 	{
@@ -145,7 +145,7 @@ public class L2LvlupData
 	}
 	
 	/**
-	 * @param _classid The _classid to set.
+	 * @param pClassid The _classid to set.
 	 */
 	public void setClassid(int pClassid)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Macro.java

@@ -51,7 +51,12 @@ public class L2Macro
 	}
 	
 	/**
-	 *
+	 * @param pId 
+	 * @param pIcon 
+	 * @param pName 
+	 * @param pDescr 
+	 * @param pAcronym 
+	 * @param pCommands 
 	 */
 	public L2Macro(int pId, int pIcon, String pName, String pDescr, String pAcronym, L2MacroCmd[] pCommands)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2ManufactureList.java

@@ -51,7 +51,7 @@ public class L2ManufactureList
 	}
 	
 	/**
-	 * @param _manufactureStoreName The _manufactureStoreName to set.
+	 * @param manufactureStoreName The _manufactureStoreName to set.
 	 */
 	public void setStoreName(String manufactureStoreName)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2MinionData.java

@@ -39,8 +39,7 @@ public class L2MinionData
 	/**
 	 * Set the Identifier of the Minion to spawn.<BR><BR>
 	 *
-	 * @param if The L2Character Identifier to spawn
-	 *
+	 * @param id The L2Character Identifier to spawn
 	 */
 	public void setMinionId(int id)
 	{
@@ -48,7 +47,7 @@ public class L2MinionData
 	}
 	
 	/**
-	 * Return the Identifier of the Minion to spawn.<BR><BR>
+	 * @return the Identifier of the Minion to spawn.
 	 */
 	public int getMinionId()
 	{
@@ -89,7 +88,7 @@ public class L2MinionData
 	}
 	
 	/**
-	 * Return the amount of this Minion type to spawn.<BR><BR>
+	 * @return the amount of this Minion type to spawn.
 	 */
 	public int getAmount()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Object.java

@@ -566,6 +566,7 @@ public abstract class L2Object
 	 *
 	 * <B><U> Concept</U> :</B><BR><BR>
 	 * A L2Object is visble if <B>__IsVisible</B>=true and <B>_worldregion</B>!=null <BR><BR>
+	 * @return 
 	 */
 	public final boolean isVisible()
 	{
@@ -641,6 +642,7 @@ public abstract class L2Object
 	
 	/**
 	 * returns reference to region this object is in
+	 * @return 
 	 */
 	public L2WorldRegion getWorldRegion()
 	{
@@ -665,6 +667,7 @@ public abstract class L2Object
 	 * <li>L2Summon</li>
 	 * <li>L2Trap</li>
 	 * <li>L2ItemInstance</li>
+	 * @param activeChar 
 	 */
 	public void sendInfo(L2PcInstance activeChar)
 	{
@@ -682,6 +685,7 @@ public abstract class L2Object
 	 *
 	 * <B><U> Overridden in </U> :</B><BR><BR>
 	 * <li> L2PcInstance</li><BR><BR>
+	 * @param mov 
 	 */
 	public void sendPacket(L2GameServerPacket mov)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Party.java

@@ -106,7 +106,7 @@ public class L2Party
 	/**
 	 * constructor ensures party has always one member - leader
 	 * @param leader
-	 * @param itemDistributionMode
+	 * @param itemDistribution
 	 */
 	public L2Party(L2PcInstance leader, int itemDistribution)
 	{
@@ -131,6 +131,7 @@ public class L2Party
 	/**
 	 * set invitation process flag and store time for expiration
 	 * happens when: player join party or player decline to join
+	 * @param val 
 	 */
 	public void setPendingInvitation(boolean val)
 	{
@@ -159,6 +160,8 @@ public class L2Party
 	
 	/**
 	 * get random member from party
+	 * @param ItemId 
+	 * @param target 
 	 * @return
 	 */
 	private L2PcInstance getCheckedRandomMember(int ItemId, L2Character target)
@@ -176,6 +179,8 @@ public class L2Party
 	
 	/**
 	 * get next item looter
+	 * @param ItemId 
+	 * @param target 
 	 * @return
 	 */
 	private L2PcInstance getCheckedNextLooter(int ItemId, L2Character target)
@@ -202,6 +207,10 @@ public class L2Party
 	
 	/**
 	 * get next item looter
+	 * @param player 
+	 * @param ItemId 
+	 * @param spoil 
+	 * @param target 
 	 * @return
 	 */
 	private L2PcInstance getActualLooter(L2PcInstance player, int ItemId, boolean spoil, L2Character target)
@@ -289,6 +298,8 @@ public class L2Party
 	
 	/**
 	 * Send a Server->Client packet to all other L2PcInstance of the Party.<BR><BR>
+	 * @param player 
+	 * @param msg 
 	 */
 	public void broadcastToPartyMembers(L2PcInstance player, L2GameServerPacket msg)
 	{
@@ -648,6 +659,8 @@ public class L2Party
 	 * distribute item(s) to party members
 	 * @param player
 	 * @param item
+	 * @param spoil 
+	 * @param target 
 	 */
 	public void distributeItem(L2PcInstance player, L2Attackable.RewardItem item, boolean spoil, L2Attackable target)
 	{
@@ -685,7 +698,9 @@ public class L2Party
 	
 	/**
 	 * distribute adena to party members
+	 * @param player 
 	 * @param adena
+	 * @param target 
 	 */
 	public void distributeAdena(L2PcInstance player, long adena, L2Character target)
 	{
@@ -729,7 +744,9 @@ public class L2Party
 	 * @param xpReward The Experience reward to distribute
 	 * @param spReward The SP reward to distribute
 	 * @param rewardedMembers The list of L2PcInstance to reward
-	 *
+	 * @param topLvl 
+	 * @param partyDmg 
+	 * @param target 
 	 */
 	public void distributeXpAndSp(long xpReward, int spReward, List<L2Playable> rewardedMembers, int topLvl, int partyDmg, L2Attackable target)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2RecipeInstance.java

@@ -28,6 +28,8 @@ public class L2RecipeInstance
 	
 	/**
 	 * Constructor of L2RecipeInstance (create a new line in a RecipeList).<BR><BR>
+	 * @param itemId 
+	 * @param quantity 
 	 */
 	public L2RecipeInstance(int itemId, int quantity)
 	{
@@ -36,7 +38,7 @@ public class L2RecipeInstance
 	}
 	
 	/**
-	 * Return the Identifier of the L2RecipeInstance Item needed.<BR><BR>
+	 * @return the Identifier of the L2RecipeInstance Item needed.
 	 */
 	public int getItemId()
 	{
@@ -44,7 +46,7 @@ public class L2RecipeInstance
 	}
 	
 	/**
-	 * Return the Item quantity needed of the L2RecipeInstance.<BR><BR>
+	 * @return the Item quantity needed of the L2RecipeInstance.
 	 */
 	public int getQuantity()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2RecipeList.java

@@ -67,6 +67,8 @@ public class L2RecipeList
 	
 	/**
 	 * Constructor of L2RecipeList (create a new Recipe).<BR><BR>
+	 * @param set 
+	 * @param haveRare 
 	 */
 	public L2RecipeList(StatsSet set, boolean haveRare)
 	{
@@ -91,6 +93,7 @@ public class L2RecipeList
 	
 	/**
 	 * Add a L2RecipeInstance to the L2RecipeList (add a line Item-Quantity needed to the Recipe).<BR><BR>
+	 * @param recipe 
 	 */
 	public void addRecipe(L2RecipeInstance recipe)
 	{
@@ -103,6 +106,7 @@ public class L2RecipeList
 	
 	/**
 	 * Add a L2RecipeStatInstance of the statUse parameter to the L2RecipeList.<BR><BR>
+	 * @param statUse 
 	 */
 	public void addStatUse(L2RecipeStatInstance statUse)
 	{
@@ -115,6 +119,7 @@ public class L2RecipeList
 	
 	/**
 	 * Add a L2RecipeStatInstance of the altStatChange parameter to the L2RecipeList.<BR><BR>
+	 * @param statChange 
 	 */
 	public void addAltStatChange(L2RecipeStatInstance statChange)
 	{
@@ -126,7 +131,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the Identifier of the Instance.<BR><BR>
+	 * @return the Identifier of the Instance.
 	 */
 	public int getId()
 	{
@@ -134,7 +139,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the crafting level needed to use this L2RecipeList.<BR><BR>
+	 * @return the crafting level needed to use this L2RecipeList.
 	 */
 	public int getLevel()
 	{
@@ -142,7 +147,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the Identifier of the L2RecipeList.<BR><BR>
+	 * @return the Identifier of the L2RecipeList.
 	 */
 	public int getRecipeId()
 	{
@@ -150,7 +155,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the name of the L2RecipeList.<BR><BR>
+	 * @return the name of the L2RecipeList.
 	 */
 	public String getRecipeName()
 	{
@@ -158,7 +163,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the crafting success rate when using the L2RecipeList.<BR><BR>
+	 * @return the crafting success rate when using the L2RecipeList.
 	 */
 	public int getSuccessRate()
 	{
@@ -166,7 +171,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return rue if the Item crafted with this L2RecipeList is consumable (shot, arrow,...).<BR><BR>
+	 * @return true if the Item crafted with this L2RecipeList is consumable (shot, arrow,...).
 	 */
 	public boolean isConsumable()
 	{
@@ -178,7 +183,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the Identifier of the Item crafted with this L2RecipeList.<BR><BR>
+	 * @return the Identifier of the Item crafted with this L2RecipeList.
 	 */
 	public int getItemId()
 	{
@@ -186,7 +191,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the quantity of Item crafted when using this L2RecipeList.<BR><BR>
+	 * @return the quantity of Item crafted when using this L2RecipeList.
 	 */
 	public int getCount()
 	{
@@ -194,7 +199,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the Identifier of the Rare Item crafted with this L2RecipeList.<BR><BR>
+	 * @return the Identifier of the Rare Item crafted with this L2RecipeList.
 	 */
 	public int getRareItemId()
 	{
@@ -202,7 +207,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the quantity of Rare Item crafted when using this L2RecipeList.<BR><BR>
+	 * @return the quantity of Rare Item crafted when using this L2RecipeList.
 	 */
 	public int getRareCount()
 	{
@@ -210,7 +215,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the chance of Rare Item crafted when using this L2RecipeList.<BR><BR>
+	 * @return the chance of Rare Item crafted when using this L2RecipeList.
 	 */
 	public int getRarity()
 	{
@@ -218,7 +223,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return <B>true</B> if this a Dwarven recipe or <B>false</B> if its a Common recipe
+	 * @return {@code true} if this a Dwarven recipe or {@code false} if its a Common recipe
 	 */
 	public boolean isDwarvenRecipe()
 	{
@@ -226,7 +231,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the table containing all L2RecipeInstance (1 line of the recipe : Item-Quantity needed) of the L2RecipeList.<BR><BR>
+	 * @return the table containing all L2RecipeInstance (1 line of the recipe : Item-Quantity needed) of the L2RecipeList.
 	 */
 	public L2RecipeInstance[] getRecipes()
 	{
@@ -234,7 +239,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the table containing all L2RecipeStatInstance of the statUse parameter of the L2RecipeList.<BR><BR>
+	 * @return the table containing all L2RecipeStatInstance of the statUse parameter of the L2RecipeList.
 	 */
 	public L2RecipeStatInstance[] getStatUse()
 	{
@@ -242,7 +247,7 @@ public class L2RecipeList
 	}
 	
 	/**
-	 * Return the table containing all L2RecipeStatInstance of the AltStatChange parameter of the L2RecipeList.<BR><BR>
+	 * @return the table containing all L2RecipeStatInstance of the AltStatChange parameter of the L2RecipeList.
 	 */
 	public L2RecipeStatInstance[] getAltStatChange()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2RecipeStatInstance.java

@@ -39,6 +39,8 @@ public class L2RecipeStatInstance
 	
 	/**
 	 * Constructor of L2RecipeStatInstance.<BR><BR>
+	 * @param type 
+	 * @param value 
 	 */
 	public L2RecipeStatInstance(String type, int value)
 	{
@@ -51,7 +53,7 @@ public class L2RecipeStatInstance
 	}
 	
 	/**
-	 * Return the the type of the L2RecipeStatInstance.<BR><BR>
+	 * @return the the type of the L2RecipeStatInstance.
 	 */
 	public StatType getType()
 	{
@@ -59,7 +61,7 @@ public class L2RecipeStatInstance
 	}
 	
 	/**
-	 * Return the value of the L2RecipeStatInstance.<BR><BR>
+	 * @return the value of the L2RecipeStatInstance.
 	 */
 	public int getValue()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Request.java

@@ -50,6 +50,7 @@ public class L2Request
 	
 	/**
 	 * Set the L2PcInstance member of a transaction (ex : FriendInvite, JoinAlly, JoinParty...).<BR><BR>
+	 * @param partner 
 	 */
 	private synchronized void setPartner(L2PcInstance partner)
 	{
@@ -57,7 +58,7 @@ public class L2Request
 	}
 	
 	/**
-	 * Return the L2PcInstance member of a transaction (ex : FriendInvite, JoinAlly, JoinParty...).<BR><BR>
+	 * @return the L2PcInstance member of a transaction (ex : FriendInvite, JoinAlly, JoinParty...).
 	 */
 	public L2PcInstance getPartner()
 	{
@@ -65,7 +66,8 @@ public class L2Request
 	}
 	
 	/**
-	 * Set the packet incomed from requestor.<BR><BR>
+	 * Set the packet incomed from requester.<BR><BR>
+	 * @param packet 
 	 */
 	private synchronized void setRequestPacket(L2GameClientPacket packet)
 	{
@@ -73,7 +75,8 @@ public class L2Request
 	}
 	
 	/**
-	 * Return the packet originally incomed from requestor.<BR><BR>
+	 * Return the packet originally incomed from requester.<BR><BR>
+	 * @return 
 	 */
 	public L2GameClientPacket getRequestPacket()
 	{
@@ -82,6 +85,9 @@ public class L2Request
 	
 	/**
 	 * Checks if request can be made and in success case puts both PC on request state.<BR><BR>
+	 * @param partner 
+	 * @param packet 
+	 * @return 
 	 */
 	public synchronized boolean setRequest(L2PcInstance partner, L2GameClientPacket packet)
 	{
@@ -140,7 +146,7 @@ public class L2Request
 	}
 	
 	/**
-	 * Return True if a transaction is in progress.<BR><BR>
+	 * @return {@code true} if a transaction is in progress.
 	 */
 	public boolean isProcessingRequest()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Skill.java

@@ -634,7 +634,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return the target type of the skill : SELF, PARTY, CLAN, PET...<BR><BR>
-	 *
+	 * @return 
 	 */
 	public final SkillTargetType getTargetType()
 	{
@@ -665,7 +665,12 @@ public abstract class L2Skill implements IChanceSkillTrigger
 		return isPassive();
 	}
 	/**
-	 * Return the power of the skill.<BR><BR>
+	 * Return the power of the skill.
+	 * @param activeChar 
+	 * @param target 
+	 * @param isPvP 
+	 * @param isPvE 
+	 * @return 
 	 */
 	public final double getPower(L2Character activeChar, L2Character target, boolean isPvP, boolean isPvE)
 	{
@@ -748,6 +753,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return true if skill should ignore all resistances
+	 * @return 
 	 */
 	public final boolean ignoreResists()
 	{
@@ -756,6 +762,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return minimum skill/effect land rate (default is 1).
+	 * @return 
 	 */
 	public final int getMinChance()
 	{
@@ -764,6 +771,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return maximum skill/effect land rate (default is 99).
+	 * @return 
 	 */
 	public final int getMaxChance()
 	{
@@ -772,6 +780,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return true if skill effects should be removed on any action except movement
+	 * @return 
 	 */
 	public final boolean isRemovedOnAnyActionExceptMove()
 	{
@@ -780,6 +789,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return true if skill effects should be removed on damage
+	 * @return 
 	 */
 	public final boolean isRemovedOnDamage()
 	{
@@ -788,6 +798,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return the additional effect Id.<BR><BR>
+	 * @return 
 	 */
 	public final int getEffectId()
 	{
@@ -795,6 +806,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	}
 	/**
 	 * Return the additional effect level.<BR><BR>
+	 * @return 
 	 */
 	public final int getEffectLvl()
 	{
@@ -808,6 +820,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return the additional effect skill type (ex : STUN, PARALYZE,...).<BR><BR>
+	 * @return 
 	 */
 	public final L2SkillType getEffectType()
 	{
@@ -816,6 +829,7 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Return true if character should attack target after skill
+	 * @return 
 	 */
 	public final boolean nextActionIsAttack()
 	{
@@ -904,7 +918,8 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	}
 	
 	/**
-	 * Return the skill type (ex : BLEED, SLEEP, WATER...).<BR><BR>
+	 * Return the skill type (ex : BLEED, SLEEP, WATER...).
+	 * @return 
 	 */
 	public final Stats getStat()
 	{
@@ -912,7 +927,8 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	}
 	
 	/**
-	 * Return skill saveVs base stat (STR, INT ...).<BR><BR>
+	 * Return skill saveVs base stat (STR, INT ...).
+	 * @return 
 	 */
 	public final BaseStats getSaveVs()
 	{
@@ -1443,6 +1459,9 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	 * <li>ITEM</li><BR><BR>
 	 *
 	 * @param activeChar The L2Character who use the skill
+	 * @param onlyFirst 
+	 * @param target 
+	 * @return 
 	 *
 	 */
 	public final L2Object[] getTargetList(L2Character activeChar, boolean onlyFirst, L2Character target)
@@ -2593,7 +2612,9 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Env is used to pass parameters for secondary effects (shield and ss/bss/bsss)
-	 * 
+	 * @param effector 
+	 * @param effected 
+	 * @param env 
 	 * @return an array with the effects that have been added to effector
 	 */
 	public final L2Effect[] getEffects(L2Character effector, L2Character effected, Env env)
@@ -2663,7 +2684,9 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	
 	/**
 	 * Warning: this method doesn't consider modifier (shield, ss, sps, bss) for secondary effects
-	 * 
+	 * @param effector 
+	 * @param effected 
+	 * @return 
 	 */
 	public final L2Effect[] getEffects(L2Character effector, L2Character effected)
 	{
@@ -2680,6 +2703,10 @@ public abstract class L2Skill implements IChanceSkillTrigger
 	 * This new feature works following next rules:
 	 * <li> To enable feature, effectPower must be over -1 (check DocumentSkill#attachEffect for further information)</li>
 	 * <li> If main skill fails, secondary effect always fail</li>
+	 * @param effector 
+	 * @param effected 
+	 * @param env 
+	 * @return 
 	 */
 	public final L2Effect[] getEffects(L2CubicInstance effector, L2Character effected, Env env)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Spawn.java

@@ -148,7 +148,9 @@ public class L2Spawn
 	 * <li>Create the generic constructor of L2NpcInstance managed by this L2Spawn</li><BR><BR>
 	 *
 	 * @param mobTemplate The L2NpcTemplate to link to this L2Spawn
-	 *
+	 * @throws SecurityException 
+	 * @throws ClassNotFoundException 
+	 * @throws NoSuchMethodException 
 	 */
 	public L2Spawn(L2NpcTemplate mobTemplate) throws SecurityException, ClassNotFoundException, NoSuchMethodException
 	{
@@ -164,7 +166,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the maximum number of L2NpcInstance that this L2Spawn can manage.<BR><BR>
+	 * @return the maximum number of L2NpcInstance that this L2Spawn can manage.
 	 */
 	public int getAmount()
 	{
@@ -172,7 +174,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the Identifier of the location area where L2NpcInstance can be spwaned.<BR><BR>
+	 * @return the Identifier of the location area where L2NpcInstance can be spwaned.
 	 */
 	public int getLocation()
 	{
@@ -180,7 +182,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the X position of the spwan point.<BR><BR>
+	 * @return the X position of the spwan point.
 	 */
 	public int getLocx()
 	{
@@ -188,7 +190,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the Y position of the spwan point.<BR><BR>
+	 * @return the Y position of the spwan point.
 	 */
 	public int getLocy()
 	{
@@ -196,7 +198,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the Z position of the spwan point.<BR><BR>
+	 * @return the Z position of the spwan point.
 	 */
 	public int getLocz()
 	{
@@ -204,7 +206,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the Itdentifier of the L2NpcInstance manage by this L2Spwan contained in the L2NpcTemplate.<BR><BR>
+	 * @return the Identifier of the L2NpcInstance manage by this L2Spwan contained in the L2NpcTemplate.
 	 */
 	public int getNpcid()
 	{
@@ -212,7 +214,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the heading of L2NpcInstance when they are spawned.<BR><BR>
+	 * @return the heading of L2NpcInstance when they are spawned.
 	 */
 	public int getHeading()
 	{
@@ -220,21 +222,21 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return the delay between a L2NpcInstance remove and its re-spawn.<BR><BR>
+	 * @return the delay between a L2NpcInstance remove and its re-spawn.
 	 */
 	public int getRespawnDelay()
 	{
 		return _respawnDelay;
 	}
 	/**
-	 * Return Min RaidBoss Spawn delay.<BR><BR>
+	 * @return Min RaidBoss Spawn delay.
 	 */
 	public int getRespawnMinDelay()
 	{
 		return _respawnMinDelay;
 	}
 	/**
-	 * Return Max RaidBoss Spawn delay.<BR><BR>
+	 * @return Max RaidBoss Spawn delay.
 	 */
 	public int getRespawnMaxDelay()
 	{
@@ -243,6 +245,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the maximum number of L2NpcInstance that this L2Spawn can manage.<BR><BR>
+	 * @param amount 
 	 */
 	public void setAmount(int amount)
 	{
@@ -251,6 +254,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the Identifier of the location area where L2NpcInstance can be spwaned.<BR><BR>
+	 * @param location 
 	 */
 	public void setLocation(int location)
 	{
@@ -258,6 +262,7 @@ public class L2Spawn
 	}
 	/**
 	 * Set Minimum Respawn Delay.<BR><BR>
+	 * @param date 
 	 */
 	public void setRespawnMinDelay(int date)
 	{
@@ -265,6 +270,7 @@ public class L2Spawn
 	}
 	/**
 	 * Set Maximum Respawn Delay.<BR><BR>
+	 * @param date 
 	 */
 	public void setRespawnMaxDelay(int date)
 	{
@@ -272,6 +278,7 @@ public class L2Spawn
 	}
 	/**
 	 * Set the X position of the spwan point.<BR><BR>
+	 * @param locx 
 	 */
 	public void setLocx(int locx)
 	{
@@ -280,6 +287,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the Y position of the spwan point.<BR><BR>
+	 * @param locy 
 	 */
 	public void setLocy(int locy)
 	{
@@ -288,6 +296,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the Z position of the spwan point.<BR><BR>
+	 * @param locz 
 	 */
 	public void setLocz(int locz)
 	{
@@ -296,6 +305,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the heading of L2NpcInstance when they are spawned.<BR><BR>
+	 * @param heading 
 	 */
 	public void setHeading(int heading)
 	{
@@ -304,6 +314,7 @@ public class L2Spawn
 	
 	/**
 	 * Set the spawn as custom.<BR>
+	 * @param custom 
 	 */
 	public void setCustom(boolean custom)
 	{
@@ -311,8 +322,7 @@ public class L2Spawn
 	}
 	
 	/**
-	 * Return type of spawn.<BR>
-	 * <BR>
+	 * @return type of spawn.
 	 */
 	public boolean isCustom()
 	{
@@ -329,7 +339,7 @@ public class L2Spawn
 	 * <li>Create a new SpawnTask to launch after the respawn Delay </li><BR><BR>
 	 *
 	 * <FONT COLOR=#FF0000><B> <U>Caution</U> : A respawn is possible ONLY if _doRespawn=True and _scheduledCount + _currentCount < _maximumCount</B></FONT><BR><BR>
-	 *
+	 * @param oldNpc 
 	 */
 	public void decreaseCount(/*int npcId*/L2Npc oldNpc)
 	{
@@ -370,6 +380,8 @@ public class L2Spawn
 	
 	/**
 	 * Create a L2NpcInstance in this L2Spawn.<BR><BR>
+	 * @param val 
+	 * @return 
 	 */
 	public L2Npc spawnOne(boolean val)
 	{
@@ -377,7 +389,7 @@ public class L2Spawn
 	}
 
 	/**
-	 * Return true if respawn enabled
+	 * @return true if respawn enabled
 	 */
 	public boolean isRespawnEnabled()
 	{
@@ -385,7 +397,7 @@ public class L2Spawn
 	}
 
 	/**
-	 * Set _doRespawn to False to stop respawn in thios L2Spawn.<BR><BR>
+	 * Set _doRespawn to False to stop respawn in this L2Spawn.<BR><BR>
 	 */
 	public void stopRespawn()
 	{
@@ -422,7 +434,8 @@ public class L2Spawn
 	 * <li>Init other values of the L2NpcInstance (ex : from its L2CharTemplate for INT, STR, DEX...) and add it in the world </li>
 	 * <li>Lauch the action OnSpawn fo the L2NpcInstance </li><BR><BR>
 	 * <li>Increase the current number of L2NpcInstance managed by this L2Spawn  </li><BR><BR>
-	 *
+	 * @param isSummonSpawn 
+	 * @return 
 	 */
 	public L2Npc doSpawn(boolean isSummonSpawn)
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2Transformation.java

@@ -182,7 +182,7 @@ public abstract class L2Transformation implements Cloneable, Runnable
 	}
 	
 	/**
-	 * Returns true if transformation can do melee attack
+	 * @return true if transformation can do melee attack
 	 */
 	public boolean canDoMeleeAttack()
 	{
@@ -190,7 +190,7 @@ public abstract class L2Transformation implements Cloneable, Runnable
 	}
 	
 	/**
-	 * Returns true if transformation can start follow target when trying to cast an skill out of range
+	 * @return true if transformation can start follow target when trying to cast an skill out of range
 	 */
 	public boolean canStartFollowToCast()
 	{

L2J_Server_BETA/java/com/l2jserver/gameserver/model/L2World.java

@@ -101,7 +101,7 @@ public final class L2World
 	}
 	
 	/**
-	 * Return the current instance of L2World.<BR><BR>
+	 * @return the current instance of L2World.
 	 */
 	public static L2World getInstance()
 	{
@@ -114,6 +114,7 @@ public final class L2World
 	 * <B><U> Example of use </U> :</B><BR><BR>
 	 * <li> Withdraw an item from the warehouse, create an item</li>
 	 * <li> Spawn a L2Character (PC, NPC, Pet)</li><BR>
+	 * @param object 
 	 */
 	public void storeObject(L2Object object)
 	{
@@ -181,12 +182,11 @@ public final class L2World
 	}
 	
 	/**
-	 * Return the L2Object object that belongs to an ID or null if no object found.<BR><BR>
-	 *
 	 * <B><U> Example of use </U> :</B><BR><BR>
 	 * <li> Client packets : Action, AttackRequest, RequestJoinParty, RequestJoinPledge...</li><BR>
 	 *
 	 * @param oID Identifier of the L2Object
+	 * @return the L2Object object that belongs to an ID or null if no object found.
 	 */
 	public L2Object findObject(int oID)
 	{
@@ -203,11 +203,8 @@ public final class L2World
 	
 	/**
 	 * Added by Tempy - 08 Aug 05
-	 * Allows easy retrevial of all visible objects in world.
-	 * 
-	 * -- do not use that fucntion, its unsafe!
-	 * 
-	 * @deprecated
+	 * @return retrieval of all visible objects in world.
+	 * @deprecated do not use that function, its unsafe!
 	 */
 	@Deprecated
 	public final TIntObjectHashMap<L2Object> getAllVisibleObjects()
@@ -236,8 +233,7 @@ public final class L2World
 	}
 	
 	/**
-	 * Return a table containing all GMs.<BR><BR>
-	 * 
+	 * @return a table containing all GMs.
 	 */
 	public FastList<L2PcInstance> getAllGMs()
 	{
@@ -270,10 +266,9 @@ public final class L2World
 	}
 	
 	/**
-	 * Return the player instance corresponding to the given name.<BR><BR>
 	 * <B>If you have access to player objectId use {@link #getPlayer(int playerObjId)}</B>
-	 * <BR>
 	 * @param name Name of the player to get Instance
+	 * @return the player instance corresponding to the given name.
 	 */
 	public L2PcInstance getPlayer(String name)
 	{
@@ -281,9 +276,8 @@ public final class L2World
 	}
 	
 	/**
-	 * Return the player instance corresponding to the given object ID.<BR><BR>
-	 *
 	 * @param playerObjId Object ID of the player to get Instance
+	 * @return the player instance corresponding to the given object ID.
 	 */
 	public L2PcInstance getPlayer(int playerObjId)
 	{
@@ -291,9 +285,8 @@ public final class L2World
 	}
 	
 	/**
-	 * Return the pet instance from the given ownerId.<BR><BR>
-	 *
 	 * @param ownerId ID of the owner
+	 * @return the pet instance from the given ownerId.
 	 */
 	public L2PetInstance getPet(int ownerId)
 	{
@@ -305,6 +298,7 @@ public final class L2World
 	 *
 	 * @param ownerId ID of the owner
 	 * @param pet L2PetInstance of the pet
+	 * @return 
 	 */
 	public L2PetInstance addPet(int ownerId, L2PetInstance pet)
 	{
@@ -357,9 +351,7 @@ public final class L2World
 	 * <li> Apply Death Penalty of a L2PcInstance </li><BR><BR>
 	 *
 	 * @param object L2object to add in the world
-	 * @param newregion L2WorldRegion in wich the object will be add (not used)
-	 * @param dropper L2Character who has dropped the object (if necessary)
-	 *
+	 * @param newRegion L2WorldRegion in wich the object will be add (not used)
 	 */
 	public void addVisibleObject(L2Object object, L2WorldRegion newRegion)
 	{
@@ -413,7 +405,7 @@ public final class L2World
 	
 	/**
 	 * Add the L2PcInstance to _allPlayers of L2World.<BR><BR>
-	 * 
+	 * @param cha 
 	 */
 	public void addToAllPlayers(L2PcInstance cha)
 	{
@@ -424,8 +416,8 @@ public final class L2World
 	 * Remove the L2PcInstance from _allPlayers of L2World.<BR><BR>
 	 *
 	 * <B><U> Example of use </U> :</B><BR><BR>
-	 * <li> Remove a player fom the visible objects </li><BR>
-	 *
+	 * <li> Remove a player from the visible objects </li><BR>
+	 * @param cha 
 	 */
 	public void removeFromAllPlayers(L2PcInstance cha)
 	{
@@ -457,7 +449,7 @@ public final class L2World
 	 * <li> Decay a L2Character</li><BR><BR>
 	 *
 	 * @param object L2object to remove from the world
-	 * @param oldregion L2WorldRegion in wich the object was before removing
+	 * @param oldRegion L2WorldRegion in which the object was before removing
 	 *
 	 */
 	public void removeVisibleObject(L2Object object, L2WorldRegion oldRegion)
@@ -517,7 +509,7 @@ public final class L2World
 	 * <li> Find Close Objects for L2Character </li><BR>
 	 *
 	 * @param object L2object that determine the current L2WorldRegion
-	 *
+	 * @return 
 	 */
 	public List<L2Object> getVisibleObjects(L2Object object)
 	{
@@ -561,7 +553,7 @@ public final class L2World
 	 *
 	 * @param object L2object that determine the center of the circular area
 	 * @param radius Radius of the circular area
-	 *
+	 * @return 
 	 */
 	public List<L2Object> getVisibleObjects(L2Object object, int radius)
 	{
@@ -612,7 +604,7 @@ public final class L2World
 	 *
 	 * @param object L2object that determine the center of the circular area
 	 * @param radius Radius of the spheric area
-	 *
+	 * @return 
 	 */
 	public List<L2Object> getVisibleObjects3D(L2Object object, int radius)
 	{
@@ -653,8 +645,6 @@ public final class L2World
 	}
 	
 	/**
-	 * Return all visible players of the L2WorldRegion object's and of its surrounding L2WorldRegion.<BR><BR>
-	 *
 	 * <B><U> Concept</U> :</B><BR><BR>
 	 * All visible object are identified in <B>_visibleObjects</B> of their current L2WorldRegion <BR>
 	 * All surrounding L2WorldRegion are identified in <B>_surroundingRegions</B> of the selected L2WorldRegion in order to scan a large area around a L2Object<BR><BR>
@@ -663,7 +653,7 @@ public final class L2World
 	 * <li> Find Close Objects for L2Character </li><BR>
 	 *
 	 * @param object L2object that determine the current L2WorldRegion
-	 *
+	 * @return all visible players of the L2WorldRegion object's and of its surrounding L2WorldRegion.
 	 */
 	public List<L2Playable> getVisiblePlayable(L2Object object)
 	{
@@ -702,9 +692,10 @@ public final class L2World
 	 *
 	 * <B><U> Example of use </U> :</B><BR><BR>
 	 * <li> Set position of a new L2Object (drop, spawn...) </li>
-	 * <li> Update position of a L2Object after a mouvement </li><BR>
+	 * <li> Update position of a L2Object after a movement </li><BR>
 	 *
-	 * @param Point3D point position of the object
+	 * @param point position of the object
+	 * @return 
 	 */
 	public L2WorldRegion getRegion(Point3D point)
 	{