You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jmeter.apache.org by fs...@apache.org on 2016/03/18 11:42:05 UTC

svn commit: r1735572 - /jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java

Author: fschumacher
Date: Fri Mar 18 10:42:05 2016
New Revision: 1735572

URL: http://svn.apache.org/viewvc?rev=1735572&view=rev
Log:
javadoc. Explain the parameters and return values a bit more.

Modified:
    jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java

Modified: jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java
URL: http://svn.apache.org/viewvc/jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java?rev=1735572&r1=1735571&r2=1735572&view=diff
==============================================================================
--- jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java (original)
+++ jmeter/trunk/src/protocol/http/org/apache/jmeter/protocol/http/parser/BaseParser.java Fri Mar 18 10:42:05 2016
@@ -35,16 +35,22 @@ public abstract class BaseParser impleme
     private static final Map<String, LinkExtractorParser> PARSERS = new ConcurrentHashMap<>(5);
 
     /**
-     * 
+     * Constructor for BaseParser
      */
     public BaseParser() {
     }
 
     /**
-     * Factory method of parsers
+     * Factory method of parsers. Instances might get cached, when
+     * {@link LinkExtractorParser#isReusable()} on the newly created instance
+     * equals {@code true}.
+     * 
      * @param parserClassName
-     * @return {@link LinkExtractorParser}
+     *            name of the class that should be used to create new parsers
+     * @return a possibly cached instance of the wanted
+     *         {@link LinkExtractorParser}
      * @throws LinkExtractorParseException
+     *             when a new instance could not be instantiated
      */
     public static LinkExtractorParser getParser(String parserClassName) 
             throws LinkExtractorParseException {
@@ -79,7 +85,7 @@ public abstract class BaseParser impleme
      * Parsers should over-ride this method if the parser class is re-usable, in
      * which case the class will be cached for the next getParser() call.
      *
-     * @return true if the Parser is reusable
+     * @return {@code true} if the Parser is reusable
      */
     @Override
     public boolean isReusable() {