Author: fschumacher
Date: Sun Dec 28 12:47:46 2014
New Revision: 1648193

URL: http://svn.apache.org/r1648193
Log:
Bug 57193: Add param and return tags to javadoc
Bugzilla Id: 57193

Modified:
    jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellInterpreter.java
    jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellServer.java
    jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java
    jmeter/trunk/src/core/org/apache/jmeter/util/ColorHelper.java
    jmeter/trunk/src/core/org/apache/jmeter/util/Document.java
    jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
    jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java
    jmeter/trunk/src/core/org/apache/jmeter/util/NameUpdater.java
    jmeter/trunk/src/core/org/apache/jmeter/util/SSLManager.java
    jmeter/trunk/src/core/org/apache/jmeter/util/SlowSocket.java
    jmeter/trunk/src/core/org/apache/jmeter/util/XPathUtil.java
    jmeter/trunk/src/core/org/apache/jmeter/util/keystore/JmeterKeyStore.java
    jmeter/trunk/src/core/org/apache/jmeter/visualizers/Sample.java
    jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java
    jmeter/trunk/src/core/org/apache/jmeter/visualizers/Visualizer.java
    jmeter/trunk/src/functions/org/apache/jmeter/functions/FileWrapper.java

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellInterpreter.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellInterpreter.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellInterpreter.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellInterpreter.java Sun 
Dec 28 12:47:46 2014
@@ -101,6 +101,7 @@ public class BeanShellInterpreter {
      *
      * @param init initialisation file
      * @param _log logger to pass to interpreter
+     * @throws ClassNotFoundException when beanshell can not be instantiated
      */
     public BeanShellInterpreter(String init, Logger _log)  throws 
ClassNotFoundException {
         initFile = init;

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellServer.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellServer.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellServer.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/BeanShellServer.java Sun Dec 
28 12:47:46 2014
@@ -41,7 +41,14 @@ public class BeanShellServer implements
     private final String serverfile;
 
     /**
-     *
+     * Constructor which sets the port for this server and the path to an
+     * optional init file
+     * 
+     * @param port
+     *            the port for the server to use
+     * @param file
+     *            the path to an init file, or an empty string, if no init file
+     *            should be used
      */
     public BeanShellServer(int port, String file) {
         super();

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/Calculator.java Sun Dec 28 
12:47:46 2014
@@ -22,10 +22,12 @@ import org.apache.jmeter.samplers.Sample
 
 /**
  * Class to calculate various items that don't require all previous results to 
be saved:
- * - mean = average
- * - standard deviation
- * - minimum
- * - maximum
+ * <ul>
+ *   <li>mean = average</li>
+ *   <li>standard deviation</li>
+ *   <li>minimum</li>
+ *   <li>maximum</li>
+ * </ul>
  */
 public class Calculator {
 
@@ -70,7 +72,7 @@ public class Calculator {
     /**
      * Add the value for a single sample.
      * 
-     * @param newValue
+     * @param newValue the value for the new sample
      * 
      * @see #addValue(long, int)
      * @deprecated Use {@link #addSample(SampleResult)} instead
@@ -189,6 +191,8 @@ public class Calculator {
      * May be slightly skewed because it takes the timestamps of the first and
      * last samples as the total time passed, and the test may actually have
      * started before that start time and ended after that end time.
+     *
+     * @return throughput associated to this sampler in requests per second
      */
     public double getRate() {
         if (elapsedTime == 0) {

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/ColorHelper.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/ColorHelper.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/ColorHelper.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/ColorHelper.java Sun Dec 28 
12:47:46 2014
@@ -38,6 +38,8 @@ public final class ColorHelper {
      * If all the components are the same value increment in the order of red,
      * green and blue.
      *
+     * @param col
+     *            {@link Color} to start with
      * @param inc
      *            value to increment the color components
      * @return the color after change

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/Document.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/Document.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/Document.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/Document.java Sun Dec 28 
12:47:46 2014
@@ -41,9 +41,11 @@ public class Document {
         JMeterUtils.getPropDefault("document.max_size", 10 * 1024 * 1024); // 
$NON-NLS-1$
 
     /**
-     * Convert to text plain a lot of kind of document
-     * (like odt, ods, odp, doc(x), xls(x), ppt(x), pdf, mp3, mp4, etc.) with 
Apache Tika
-     * @param document 
+     * Convert to text plain a lot of kind of document (like odt, ods, odp,
+     * doc(x), xls(x), ppt(x), pdf, mp3, mp4, etc.) with Apache Tika
+     *
+     * @param document
+     *            binary representation of the document
      * @return text from document without format
      */
     public static String getTextFromDocument(byte[] document) {

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/JMeterUtils.java Sun Dec 28 
12:47:46 2014
@@ -173,7 +173,9 @@ public class JMeterUtils implements Unit
      * Load the JMeter properties file; if not found, then
      * default to "org/apache/jmeter/jmeter.properties" from the classpath
      *
+     * <p>
      * c.f. loadProperties
+     *
      * @param file Name of the file from which the JMeter properties should be 
loaded
      */
     public static void loadJMeterProperties(String file) {
@@ -259,7 +261,7 @@ public class JMeterUtils implements Unit
     /**
      * Get a compiled expression from the pattern cache (READ_ONLY).
      *
-     * @param expression
+     * @param expression regular expression to be looked up
      * @return compiled pattern
      *
      * @throws MalformedCachePatternException (Runtime)
@@ -274,7 +276,7 @@ public class JMeterUtils implements Unit
      * Get a compiled expression from the pattern cache.
      *
      * @param expression RE
-     * @param options e.g. READ_ONLY_MASK
+     * @param options e.g. {@link Perl5Compiler#READ_ONLY_MASK READ_ONLY_MASK}
      * @return compiled pattern
      *
      * @throws MalformedCachePatternException (Runtime)
@@ -714,8 +716,14 @@ public class JMeterUtils implements Unit
 
     /**
      * Creates the vector of alias strings.
+     * <p>
+     * The properties will be filtered by all values starting with
+     * <code>alias.</code>. The matching entries will be used for the new
+     * {@link Hashtable} while the prefix <code>alias.</code> will be stripped
+     * of the keys.
      *
      * @param properties
+     *            the input values
      * @return The Alias value
      */
     public static Hashtable<String, String> getAlias(Properties properties) {
@@ -747,10 +755,22 @@ public class JMeterUtils implements Unit
     /**
      * Creates a table of strings for all the properties that start with a
      * common prefix.
+     * <p>
+     * So if you have {@link Properties} <code>prop</code> with two entries, 
say
+     * <ul>
+     * <li>this.test</li>
+     * <li>that.something</li>
+     * </ul>
+     * And would call this method with a <code>prefix</code> <em>this</em>, the
+     * result would be a new {@link Hashtable} with one entry, which key would
+     * be <em>test</em>.
      *
-     * @param properties input to search
-     * @param prefix to match against properties
-     * @return a Hashtable where the keys are the original keys with the 
prefix removed
+     * @param properties
+     *            input to search
+     * @param prefix
+     *            to match against properties
+     * @return a Hashtable where the keys are the original matching keys with
+     *         the prefix removed
      */
     public static Hashtable<String, String> getHashtable(Properties 
properties, String prefix) {
         Hashtable<String, String> t = new Hashtable<String, String>();
@@ -1205,7 +1225,7 @@ public class JMeterUtils implements Unit
     /**
      * Find a file in the current directory or in the JMeter bin directory.
      *
-     * @param fileName
+     * @param fileName the name of the file to find
      * @return File object
      */
     public static File findFile(String fileName){

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/JsseSSLManager.java Sun Dec 28 
12:47:46 2014
@@ -173,11 +173,12 @@ public class JsseSSLManager extends SSLM
     }
 
     /**
-     * Returns the SSLContext we are using.
-     * This is either a context per thread,
+     * Returns the SSLContext we are using. This is either a context per 
thread,
      * or, for backwards compatibility, a single shared context.
      *
      * @return The Context value
+     * @throws GeneralSecurityException
+     *             when constructing the context fails
      */
     public SSLContext getContext() throws GeneralSecurityException {
         if (SHARED_SESSION_CONTEXT) {
@@ -211,11 +212,15 @@ public class JsseSSLManager extends SSLM
             this.threadlocal.set(null);
         }
     }
+    
     /*
-     *
+     * 
      * Creates new SSL context
+     * 
      * @return SSL context
-     * @throws GeneralSecurityException
+     * 
+     * @throws GeneralSecurityException when the algorithm for the context can
+     * not be found or the keys have problems
      */
     private SSLContext createContext() throws GeneralSecurityException {
         SSLContext context;

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/NameUpdater.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/NameUpdater.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/NameUpdater.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/NameUpdater.java Sun Dec 28 
12:47:46 2014
@@ -181,7 +181,7 @@ public final class NameUpdater {
      * {@link org.apache.jmeter.save.SaveService#checkClasses() 
SaveService#checkClasses()}
      * only.
      * 
-     * @param key
+     * @param key name of the key to check
      * @return true if the key is in the map
      */
     public static boolean isMapped(String key) {

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/SSLManager.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/SSLManager.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/SSLManager.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/SSLManager.java Sun Dec 28 
12:47:46 2014
@@ -242,6 +242,8 @@ public abstract class SSLManager {
 
     /**
      * Static accessor for the SSLManager object. The SSLManager is a 
singleton.
+     *
+     * @return the singleton {@link SSLManager}
      */
     public static final synchronized SSLManager getInstance() {
         if (null == SSLManager.manager) {
@@ -267,6 +269,8 @@ public abstract class SSLManager {
 
     /**
      * Test whether SSL is supported or not.
+     *
+     * @return flag whether SSL is supported
      */
     public static final boolean isSSLSupported() {
         return SSLManager.isSSLSupported;
@@ -274,10 +278,17 @@ public abstract class SSLManager {
 
     /**
      * Configure Keystore
-     * @param preload 
-     * @param startIndex 
-     * @param endIndex 
-     * @param clientCertAliasVarName 
+     * 
+     * @param preload
+     *            flag whether the keystore should be opened within this 
method,
+     *            or the opening should be delayed
+     * @param startIndex
+     *            first index to consider for a key
+     * @param endIndex
+     *            last index to consider for a key
+     * @param clientCertAliasVarName
+     *            name of the default key, if empty the first key will be used
+     *            as default key
      */
     public void configureKeystore(boolean preload, int startIndex, int 
endIndex, String clientCertAliasVarName) {
         this.keystoreAliasStartIndex = startIndex;

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/SlowSocket.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/SlowSocket.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/SlowSocket.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/SlowSocket.java Sun Dec 28 
12:47:46 2014
@@ -50,14 +50,23 @@ public class SlowSocket extends Socket {
 
     /**
      *
-     * @param cps characters per second
-     * @param host hostname
-     * @param port port
-     * @param localAddr local address
-     * @param localPort local port
+     * @param cps
+     *            characters per second
+     * @param host
+     *            hostname
+     * @param port
+     *            port
+     * @param localAddr
+     *            local address
+     * @param localPort
+     *            local port
      *
      * @throws IOException
-     * @throws IllegalArgumentException if cps &lt;= 0
+     *             if an I/O error occurs during initialization
+     * @throws IllegalArgumentException
+     *             if cps &lt;= 0, or if the <code>port</code> or
+     *             <code>localPort</code> values lie outside of the allowed
+     *             range between <code>0</code> and <code>65535</code>
      */
     public SlowSocket(int cps, String host, int port, InetAddress localAddr, 
int localPort) throws IOException {
         super(host, port, localAddr, localPort);
@@ -69,13 +78,21 @@ public class SlowSocket extends Socket {
 
     /**
      *
-     * @param cps characters per second
-     * @param host hostname
-     * @param port port
+     * @param cps
+     *            characters per second
+     * @param host
+     *            hostname
+     * @param port
+     *            port
      *
      * @throws UnknownHostException
+     *             if the name of the host can not be determined automatically
      * @throws IOException
-     * @throws IllegalArgumentException if cps &lt;= 0
+     *             if an I/O error occurs during initialization
+     * @throws IllegalArgumentException
+     *             if cps &lt;= 0, or if the <code>port</code> or
+     *             <code>localPort</code> values lie outside of the allowed
+     *             range between <code>0</code> and <code>65535</code>
      */
     public SlowSocket(int cps, String host, int port) throws 
UnknownHostException, IOException {
         super(host, port);
@@ -88,7 +105,7 @@ public class SlowSocket extends Socket {
     /**
      * Added for use by SlowHC4SocketFactory.
      * 
-     * @param cps
+     * @param cps characters per second
      */
     public SlowSocket(int cps) {
         super();

Modified: jmeter/trunk/src/core/org/apache/jmeter/util/XPathUtil.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/XPathUtil.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/XPathUtil.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/XPathUtil.java Sun Dec 28 
12:47:46 2014
@@ -101,7 +101,7 @@ public class XPathUtil {
      * @param namespace should the parser be namespace aware?
      * @param downloadDTDs if true, parser should attempt to resolve external 
entities
      * @return document builder
-     * @throws ParserConfigurationException
+     * @throws ParserConfigurationException if {@link DocumentBuilder} can not 
be created for the wanted configuration
      */
     public static DocumentBuilder makeDocumentBuilder(boolean validate, 
boolean whitespace, boolean namespace, boolean downloadDTDs)
             throws ParserConfigurationException {
@@ -134,10 +134,10 @@ public class XPathUtil {
      * @param isXml - is document already XML (Tidy only)
      * @param downloadDTDs - if true, try to download external DTDs
      * @return document
-     * @throws ParserConfigurationException
-     * @throws SAXException
-     * @throws IOException
-     * @throws TidyException
+     * @throws ParserConfigurationException when no {@link DocumentBuilder} 
can be constructed for the wanted configuration
+     * @throws SAXException if parsing fails
+     * @throws IOException if an I/O error occurs while parsing
+     * @throws TidyException if a ParseError is detected and 
<code>report_errors</code> is <code>true</code>
      */
     public static Document makeDocument(InputStream stream, boolean validate, 
boolean whitespace, boolean namespace,
             boolean tolerant, boolean quiet, boolean showWarnings, boolean 
report_errors, boolean isXml, boolean downloadDTDs)
@@ -161,10 +161,10 @@ public class XPathUtil {
      * @param downloadDTDs - if true, try to download external DTDs
      * @param tidyOut OutputStream for Tidy pretty-printing
      * @return document
-     * @throws ParserConfigurationException
-     * @throws SAXException
-     * @throws IOException
-     * @throws TidyException
+     * @throws ParserConfigurationException if {@link DocumentBuilder} can not 
be created for the wanted configuration
+     * @throws SAXException if parsing fails
+     * @throws IOException if I/O error occurs while parsing
+     * @throws TidyException if a ParseError is detected and 
<code>report_errors</code> is <code>true</code>
      */
     public static Document makeDocument(InputStream stream, boolean validate, 
boolean whitespace, boolean namespace,
             boolean tolerant, boolean quiet, boolean showWarnings, boolean 
report_errors, boolean isXml, boolean downloadDTDs, 
@@ -289,7 +289,7 @@ public class XPathUtil {
      * @param document {@link Document}
      * @param xPathExpression XPath expression
      * @return {@link NodeList}
-     * @throws TransformerException 
+     * @throws TransformerException when the internally used xpath engine fails
      */
     public static NodeList selectNodeList(Document document, String 
xPathExpression) throws TransformerException {
         XObject xObject = XPathAPI.eval(document, xPathExpression, 
getPrefixResolver(document));
@@ -302,7 +302,7 @@ public class XPathUtil {
      * @param xPathQuery XPath Query
      * @param matchStrings List of strings that will be filled
      * @param fragment return fragment
-     * @throws TransformerException
+     * @throws TransformerException when the internally used xpath engine fails
      */
     public static void putValuesForXPathInList(Document document, 
             String xPathQuery,
@@ -374,7 +374,7 @@ public class XPathUtil {
      * @param result {@link AssertionResult}
      * @param doc XML Document
      * @param xPathExpression XPath expression
-     * @param isNegated
+     * @param isNegated flag whether a non-match should be considered a success
      */
     public static void computeAssertionResult(AssertionResult result,
             Document doc, 
@@ -430,7 +430,7 @@ public class XPathUtil {
     
     /**
      * Formats XML
-     * @param xml
+     * @param xml string to format
      * @return String formatted XML
      */
     public static final String formatXml(String xml){

Modified: 
jmeter/trunk/src/core/org/apache/jmeter/util/keystore/JmeterKeyStore.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/util/keystore/JmeterKeyStore.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/util/keystore/JmeterKeyStore.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/util/keystore/JmeterKeyStore.java 
Sun Dec 28 12:47:46 2014
@@ -48,8 +48,14 @@ public final class JmeterKeyStore {
     private static final Logger LOG = LoggingManager.getLoggerForClass();
 
     private final KeyStore store;
+
+    /** first index to consider for a key */
     private final int startIndex;
+
+    /** last index to consider for a key */
     private final int endIndex;
+
+    /** name of the default alias */
     private String clientCertAliasVarName;
 
     private String[] names = new String[0]; // default empty array to prevent 
NPEs
@@ -60,6 +66,18 @@ public final class JmeterKeyStore {
     private int last_user;
 
 
+    /**
+     * @param type
+     *            type of the {@link KeyStore}
+     * @param startIndex which keys should be considered starting from 
<code>0</code>
+     * @param endIndex which keys should be considered up to <code>count - 
1</code>
+     * @param clientCertAliasVarName name for the default key, if empty use 
the first key available
+     * @throws KeyStoreException
+     *             when the type of the keystore is not supported
+     * @throws IllegalArgumentException
+     *             when <code>startIndex</code> &lt; 0, <code>endIndex</code>
+     *             &lt; 0 or <code>endIndex</code> &lt; 
</code>startIndex</code>
+     */
     private JmeterKeyStore(String type, int startIndex, int endIndex, String 
clientCertAliasVarName) throws KeyStoreException {
         if (startIndex < 0 || endIndex < 0 || endIndex < startIndex) {
             throw new IllegalArgumentException("Invalid index(es). 
Start="+startIndex+", end="+endIndex);
@@ -71,12 +89,26 @@ public final class JmeterKeyStore {
     }
 
     /**
-     * Process the input stream
-     * @throws IOException 
-     * @throws CertificateException 
-     * @throws NoSuchAlgorithmException 
-     * @throws KeyStoreException 
-     * @throws UnrecoverableKeyException 
+     * Process the input stream and try to read the keys from the store
+     *
+     * @param is
+     *            {@link InputStream} from which the store should be loaded
+     * @param pword
+     *            the password used to check the integrity of the store
+     * @throws IOException
+     *             if there is a problem decoding or reading the store. A bad
+     *             password might be the cause for this, or an empty store
+     * @throws CertificateException
+     *             if any of the certificated in the store can not be loaded
+     * @throws NoSuchAlgorithmException
+     *             if the algorithm to check the integrity of the store can not
+     *             be found
+     * @throws KeyStoreException
+     *             if the store has not been initialized (should not happen
+     *             here)
+     * @throws UnrecoverableKeyException
+     *             if the key can not be recovered from the store (should not
+     *             happen here, either)
      */
     public void load(InputStream is, String pword) throws 
NoSuchAlgorithmException, CertificateException, IOException, KeyStoreException, 
UnrecoverableKeyException {
         char pw[] = pword==null ? null : pword.toCharArray();
@@ -133,6 +165,12 @@ public final class JmeterKeyStore {
 
     /**
      * Get the ordered certificate chain for a specific alias.
+     *
+     * @param alias
+     *            the alias for which the certificate chain should be given
+     * @return the certificate chain for the alias
+     * @throws IllegalArgumentException
+     *             if no chain could be found for the alias
      */
     public X509Certificate[] getCertificateChain(String alias) {
         X509Certificate[] result = this.certsByAlias.get(alias);
@@ -145,7 +183,12 @@ public final class JmeterKeyStore {
 
     /**
      * Get the next or only alias.
+     * 
      * @return the next or only alias.
+     * @throws IllegalArgumentException
+     *             if {@link JmeterKeyStore#clientCertAliasVarName
+     *             clientCertAliasVarName} is not empty and no key for this
+     *             alias could be found
      */
     public String getAlias() {
         if(!StringUtils.isEmpty(clientCertAliasVarName)) {
@@ -181,6 +224,12 @@ public final class JmeterKeyStore {
 
     /**
      * Return the private Key for a specific alias
+     *
+     * @param alias
+     *            the name of the alias for the private key
+     * @return the private key for the given <code>alias</code>
+     * @throws IllegalArgumentException
+     *             when no private key could be found
      */
     public PrivateKey getPrivateKey(String alias) {
         PrivateKey pk = this.privateKeyByAlias.get(alias);
@@ -192,12 +241,22 @@ public final class JmeterKeyStore {
 
     /**
      * Create a keystore which returns a range of aliases (if available)
-     * @param type store type (e.g. JKS)
-     * @param startIndex first index (from 0)
-     * @param endIndex last index (to count -1)
-     * @param clientCertAliasVarName 
+     * 
+     * @param type
+     *            store type (e.g. JKS)
+     * @param startIndex
+     *            first index (from 0)
+     * @param endIndex
+     *            last index (to count -1)
+     * @param clientCertAliasVarName
+     *            name of the default key to, if empty the first key will be
+     *            used as default key
      * @return the keystore
-     * @throws KeyStoreException 
+     * @throws KeyStoreException
+     *             when the type of the store is not supported
+     * @throws IllegalArgumentException
+     *             when <code>startIndex</code> &lt; 0, <code>endIndex</code>
+     *             &lt; 0, or <code>endIndex</code> &lt; 
<code>startIndex</code>
      */
     public static JmeterKeyStore getInstance(String type, int startIndex, int 
endIndex, String clientCertAliasVarName) throws KeyStoreException  {
         return new JmeterKeyStore(type, startIndex, endIndex, 
clientCertAliasVarName);
@@ -205,9 +264,12 @@ public final class JmeterKeyStore {
 
     /**
      * Create a keystore which returns the first alias only.
-     * @param type e.g. JKS
+     * 
+     * @param type
+     *            of the store e.g. JKS
      * @return the keystore
      * @throws KeyStoreException
+     *             when the type of the store is not supported
      */
     public static JmeterKeyStore getInstance(String type) throws 
KeyStoreException {
         return getInstance(type, 0, 0, null);

Modified: jmeter/trunk/src/core/org/apache/jmeter/visualizers/Sample.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/visualizers/Sample.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/visualizers/Sample.java (original)
+++ jmeter/trunk/src/core/org/apache/jmeter/visualizers/Sample.java Sun Dec 28 
12:47:46 2014
@@ -208,6 +208,7 @@ public class Sample implements Serializa
     }
 
     /**
+     * @param format the format of the time to be used
      * @return the start time using the specified format
      * Intended for use from Functors
      */

Modified: jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/visualizers/TableSample.java Sun 
Dec 28 12:47:46 2014
@@ -98,6 +98,7 @@ public class TableSample implements Seri
     }
 
     /**
+     * @param format the format to be used on the time
      * @return the start time using the specified format
      * Intended for use from Functors
      */

Modified: jmeter/trunk/src/core/org/apache/jmeter/visualizers/Visualizer.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/core/org/apache/jmeter/visualizers/Visualizer.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/core/org/apache/jmeter/visualizers/Visualizer.java 
(original)
+++ jmeter/trunk/src/core/org/apache/jmeter/visualizers/Visualizer.java Sun Dec 
28 12:47:46 2014
@@ -33,6 +33,9 @@ public interface Visualizer {
     /**
      * This method is called by sampling thread to inform the visualizer about
      * the arrival of a new sample.
+     *
+     * @param sample
+     *            the newly arrived sample
      */
     void add(SampleResult sample);
 

Modified: 
jmeter/trunk/src/functions/org/apache/jmeter/functions/FileWrapper.java
URL: 
http://svn.apache.org/viewvc/jmeter/trunk/src/functions/org/apache/jmeter/functions/FileWrapper.java?rev=1648193&r1=1648192&r2=1648193&view=diff
==============================================================================
--- jmeter/trunk/src/functions/org/apache/jmeter/functions/FileWrapper.java 
(original)
+++ jmeter/trunk/src/functions/org/apache/jmeter/functions/FileWrapper.java Sun 
Dec 28 12:47:46 2014
@@ -179,7 +179,9 @@ public final class FileWrapper {
      * Gets the current row number (mainly for error reporting)
      *
      * @param file
-     * @return the current row number for this thread
+     *            name of the file for which the row number is asked
+     * @return the current row number for this thread, or <code>-1</code> if
+     *         <code>file</code> was not opened yet
      */
     public static int getCurrentRow(String file) {
 


Reply via email to