You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by se...@apache.org on 2008/05/05 12:59:58 UTC

svn commit: r653420 - /jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java

Author: sebb
Date: Mon May  5 03:59:57 2008
New Revision: 653420

URL: http://svn.apache.org/viewvc?rev=653420&view=rev
Log:
Update comments

Modified:
    jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java

Modified: jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java
URL: http://svn.apache.org/viewvc/jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java?rev=653420&r1=653419&r2=653420&view=diff
==============================================================================
--- jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java (original)
+++ jakarta/jmeter/trunk/src/protocol/ftp/org/apache/jmeter/protocol/ftp/config/gui/FtpConfigGui.java Mon May  5 03:59:57 2008
@@ -70,8 +70,8 @@
 	}
 
 	public void configure(TestElement element) {
-		super.configure(element);
-		// Note: the element is a ConfigTestElement, so cannot use FTPSampler access methods
+		super.configure(element); // TODO - should this be done for embedded usage?
+		// Note: the element is a ConfigTestElement when used standalone, so we cannot use FTPSampler access methods
 		server.setText(element.getPropertyAsString(FTPSampler.SERVER));
 		remoteFile.setText(element.getPropertyAsString(FTPSampler.REMOTE_FILENAME));
 		localFile.setText(element.getPropertyAsString(FTPSampler.LOCAL_FILENAME));



---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org