commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Jeff Dever <jsde...@sympatico.ca>
Subject RE: cvs commit:jakarta-commons/httpclient/src/java/org/apache/commons/httpclient/util/HttpURLConnection.java
Date Wed, 04 Sep 2002 00:58:16 GMT
Vincent,

Fixed.  It would be easier on both of us to submit *actual* patches in
the future.  <grin>


Jeff,

There were some problems in applying the "patch". "theKeyPosition" and
"thePosition" are not supposed to be instance variables. They are
parameters passed by the respective methods where they are used (they
are prefixed by "the" as this is the convention in Cactus land). Thus
simply remove the 2 instance variables and remove "the" wherever you
find it and it should work fine.

I'll move Cactus to use the HttpClient wrapper sometime this coming week
end so Cactus will indirectly test it (through Gump runs) ...

Thanks
-Vincent


--
To unsubscribe, e-mail:   <mailto:commons-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:commons-dev-help@jakarta.apache.org>


Mime
View raw message