You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ant.apache.org by st...@apache.org on 2005/08/18 16:20:31 UTC
cvs commit: ant/src/main/org/apache/tools/ant/util ProxySetup.java
stevel 2005/08/18 07:20:31
Modified: src/main/org/apache/tools/ant/util ProxySetup.java
Log:
finish off comments
Revision Changes Path
1.2 +8 -2 ant/src/main/org/apache/tools/ant/util/ProxySetup.java
Index: ProxySetup.java
===================================================================
RCS file: /home/cvs/ant/src/main/org/apache/tools/ant/util/ProxySetup.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -r1.1 -r1.2
--- ProxySetup.java 17 Aug 2005 13:59:28 -0000 1.1
+++ ProxySetup.java 18 Aug 2005 14:20:30 -0000 1.2
@@ -20,12 +20,16 @@
import org.apache.tools.ant.Project;
/**
- * Code to do proxy setup. This is just factored out of the main system just for
+ * Code to do proxy setup. This is just factored out of the main system just to
+ * keep everything else less convoluted.
* @since Ant1.7
*/
public class ProxySetup {
+ /**
+ * owner project; used for logging and extracting properties
+ */
private Project owner;
/**
@@ -51,7 +55,7 @@
try {
return System.getProperty(USE_SYSTEM_PROXIES);
} catch (SecurityException e) {
- //if you cannot read it, you wont be able to write it either
+ //if you cannot read it, you won't be able to write it either
return null;
}
}
@@ -73,6 +77,8 @@
owner.log(message,Project.MSG_DEBUG);
System.setProperty(USE_SYSTEM_PROXIES,proxies);
} catch (SecurityException e) {
+ //log security exceptions and continue; it aint that
+ //important and may be quite common running Ant embedded.
owner.log("Security Exception when "+message);
}
}
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@ant.apache.org
For additional commands, e-mail: dev-help@ant.apache.org