[collections] nightly build failures

2005-02-11 Thread Phil Steitz
The nightlies seem to be failing for [collections], but the ant build 
works fine for me. Any ideas why the nigthlies are failing?

Phil
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: [digester2] Additions

2005-02-11 Thread Oliver Zeigermann
Hi Simon,

I have now committed the first proposal for the XMLIO like rule
manager called SupplementaryRuleManager. I also added a very basic
test for it. I had to make a minor visibility change in
DefaultRuleManager that it extends. Path now has match methods.

All this isn't perfect, especially Javadoc is missing and tests need
to be extended, but I wanted to make this public for discussion as
soon as possible.

So, comments?

And, by the, now that I have done some work and I came across some new
questions. Maybe they are just dumb, but what do you think about this?

(1) Why aren't relative paths to actions also cached?

(2) Why are the guts of Context accessible to the Action as well? They
should only be interesting to the Digester core, not to the
application. Why not adding an interface that hides the implentation
details? Like

public interface Context {
Path getCurrentPath();
String getMatchPath();

ClassLoader getClassLoader();
Object getRoot();
boolean isEmpty();
int getStackSize();
Object peek();
Object peek(int n);

putItem(ItemId id, Object data);
Object getItem(ItemId id);
}

and ContextImpl being what Context is now. Similar thing with Path. If
you agree I could do the work.

(3) Bringing this up again: Wouldn't it be more flexible to allow more
than a String as a pattern in RuleManager#addRule(String pattern,
Action action)?

Oliver

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


svn commit: r153487 - in jakarta/commons/proper/digester/branches/digester2/src: java/org/apache/commons/digester2/DefaultRuleManager.java java/org/apache/commons/digester2/Path.java java/org/apache/commons/digester2/SupplementaryRuleManager.java test/org/apache/commons/digester2/SupplementaryRuleManagerTestCase.java

2005-02-11 Thread ozeigermann
Author: ozeigermann
Date: Fri Feb 11 23:27:46 2005
New Revision: 153487

URL: http://svn.apache.org/viewcvs?view=rev&rev=153487
Log:
Added early proposal for xmlio like rule manager and processing style

Added:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SupplementaryRuleManager.java

jakarta/commons/proper/digester/branches/digester2/src/test/org/apache/commons/digester2/SupplementaryRuleManagerTestCase.java
Modified:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/DefaultRuleManager.java

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Path.java

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/DefaultRuleManager.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/DefaultRuleManager.java?view=diff&r1=153486&r2=153487
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/DefaultRuleManager.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/DefaultRuleManager.java
 Fri Feb 11 23:27:46 2005
@@ -58,7 +58,7 @@
  * Map of namespace-prefix to namespace-uri, used only by the
  * addAction() method.
  */
-private HashMap namespaces = new HashMap();
+protected HashMap namespaces = new HashMap();
 
 /**
  * The list of all actions in the cache. This set allows us to
@@ -72,7 +72,7 @@
  * find the pattern that matches the current xml element, then
  * return the list of actions.
  */
-private MultiHashMap rules = new MultiHashMap();
+protected MultiHashMap rules = new MultiHashMap();
 
 // - 
 // Public Methods

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Path.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Path.java?view=diff&r1=153486&r2=153487
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Path.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Path.java
 Fri Feb 11 23:27:46 2005
@@ -141,5 +141,36 @@
 namespaces.clear();
 localNames.clear();
 }
+
+public boolean matches(String pathToMatch) {
+if (pathToMatch.charAt(0) == '/') {
+// absolute
+return getPath().equals(pathToMatch);
+} else {
+// relative
+// XXX looks wrong but protects a match of 
+// "a/b" against a path of "/gotcha/b", but
+// still allows
+// "a/b" to match against "/a/b"
+return getPath().endsWith("/" + pathToMatch);
+}
+}
+
+/** 
+ * Checks if this path matches any of the paths given. This means we 
iterate through 
+ * pathsToMatch and match every entry to this path.
+ */
+public boolean matchsAny(String[] pathsToMatch) {
+for (int i = 0; i < pathsToMatch.length; i++) {
+if (matches(pathsToMatch[i]))
+return true;
+}
+return false;
+}
+
+public String toString() {
+return getPath();
+}
+
 }
 

Added: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SupplementaryRuleManager.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SupplementaryRuleManager.java?view=auto&rev=153487
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SupplementaryRuleManager.java
 (added)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/SupplementaryRuleManager.java
 Fri Feb 11 23:27:46 2005
@@ -0,0 +1,103 @@
+/* $Id: DefaultRuleManager.java 153050 2005-02-09 12:12:28Z skitching $
+ *
+ * Copyright 2001-2005 The Apache Software Foundation.
+ * 
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ * 
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ * 
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+

[GUMP@brutus]: Project commons-jelly-tags-ant (in module commons-jelly) failed

2005-02-11 Thread commons-jelly-tags-ant development
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at [EMAIL PROTECTED]

Project commons-jelly-tags-ant has an issue affecting its community integration.
This issue affects 2 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- commons-jelly-tags-ant :  Commons Jelly
- commons-jelly-tags-fmt :  Commons Jelly


Full details are available at:

http://brutus.apache.org/gump/public/commons-jelly/commons-jelly-tags-ant/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Sole output [commons-jelly-tags-ant-11022005.jar] identifier set to 
project name
 -DEBUG- Dependency on ant exists, no need to add for property 
maven.jar.ant-optional.
 -DEBUG- Dependency on xml-xerces exists, no need to add for property 
maven.jar.xerces.
 -DEBUG- (Gump generated) Maven Properties in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/ant/build.properties
 -INFO- Failed with reason build failed
 -DEBUG- Maven POM in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/ant/project.xml
 -DEBUG- Maven project properties in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/ant/project.properties
 -INFO- Project Reports in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/ant/target/test-reports
 -INFO- Failed to extract fallback artifacts from Gump Repository



The following work was performed:
http://brutus.apache.org/gump/public/commons-jelly/commons-jelly-tags-ant/gump_work/build_commons-jelly_commons-jelly-tags-ant.html
Work Name: build_commons-jelly_commons-jelly-tags-ant (Type: Build)
Work ended in a state of : Failed
Elapsed: 11 secs
Command Line: maven --offline jar 
[Working Directory: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/ant]
CLASSPATH: 
/opt/jdk1.4/lib/tools.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-swing.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant.jar:/usr/local/gump/public/workspace/jakarta-commons/beanutils/dist/commons-beanutils-core.jar:/usr/local/gump/public/workspace/dist/junit/junit.jar:/usr/local/gump/public/workspace/xml-commons/java/build/resolver.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant.jar:/usr/local/gump/public/workspace/jakarta-commons/cli/target/commons-cli-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons/collections/build/commons-collections-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons-sandbox/grant/target/commons-grant-11022005.jar:/usr/local/gump/public/workspace/commons-jelly/target/commons-jelly-11022005.jar:/usr/local/gump/public/workspace/commons-jelly/jelly-tags/junit/target/commons-jelly-tags-junit-11022005.jar:/usr/local/gump/public/workspace/commons-jelly/jelly-tags/util/target/commons-jelly-tags-util-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons/jexl/dist/commons-jexl-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging-api.jar:/usr/local/gump/public/workspace/dom4j/build/dom4j.jar:/usr/local/gump/public/workspace/jaxen/target/jaxen-11022005.jar
-
java:prepare-filesystem:
[mkdir] Created dir: 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/ant/target/classes

java:compile:
[echo] Compiling to 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/ant/target/classes
[echo] 
==

  NOTE: Targetting JVM 1.4, classes
  will not run on earlier JVMs

==
  
[javac] Compiling 10 source files to 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/ant/target/classes
[javac] 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/ant/src/java/org/apache/commons/jelly/tags/ant/AntTag.java:385:
 warning: 
createElement(org.apache.tools.ant.Project,java.lang.Object,java.lang.String) 
in org.apache.tools.ant.IntrospectionHelper has been deprecated
[javac] dataType = ih.createElement( getAntProject(), 
object, name.toLowerCase() );
[javac]  ^
[javac] 
/home/gump/workspa

[GUMP@brutus]: Project commons-jelly-tags-xml (in module commons-jelly) failed

2005-02-11 Thread commons-jelly-tags-xml development
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at [EMAIL PROTECTED]

Project commons-jelly-tags-xml has an issue affecting its community integration.
This issue affects 12 projects.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- commons-jelly-tags-define :  Commons Jelly
- commons-jelly-tags-html :  Commons Jelly
- commons-jelly-tags-http :  Commons Jelly
- commons-jelly-tags-jaxme :  Commons Jelly
- commons-jelly-tags-jetty :  Commons Jelly
- commons-jelly-tags-jface :  Commons Jelly
- commons-jelly-tags-jsl :  Commons Jelly
- commons-jelly-tags-swing :  Commons Jelly
- commons-jelly-tags-xml :  Commons Jelly
- commons-jelly-tags-xmlunit :  Commons Jelly
- maven :  Project Management Tools
- maven-bootstrap :  Project Management Tools


Full details are available at:

http://brutus.apache.org/gump/public/commons-jelly/commons-jelly-tags-xml/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Sole output [commons-jelly-tags-xml-11022005.jar] identifier set to 
project name
 -DEBUG- Dependency on xml-xerces exists, no need to add for property 
maven.jar.xerces.
 -DEBUG- (Gump generated) Maven Properties in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/xml/build.properties
 -INFO- Failed with reason build failed
 -DEBUG- Maven POM in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/xml/project.xml
 -DEBUG- Maven project properties in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/xml/project.properties
 -INFO- Project Reports in: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/xml/target/test-reports
 -DEBUG- Extracted fallback artifacts from Gump Repository



The following work was performed:
http://brutus.apache.org/gump/public/commons-jelly/commons-jelly-tags-xml/gump_work/build_commons-jelly_commons-jelly-tags-xml.html
Work Name: build_commons-jelly_commons-jelly-tags-xml (Type: Build)
Work ended in a state of : Failed
Elapsed: 14 secs
Command Line: maven --offline jar 
[Working Directory: 
/usr/local/gump/public/workspace/commons-jelly/jelly-tags/xml]
CLASSPATH: 
/opt/jdk1.4/lib/tools.jar:/usr/local/gump/public/workspace/jakarta-commons/beanutils/dist/commons-beanutils-core.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-swing.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant.jar:/usr/local/gump/public/workspace/dist/junit/junit.jar:/usr/local/gump/public/workspace/xml-commons/java/build/resolver.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant.jar:/usr/local/gump/public/workspace/jakarta-commons/collections/build/commons-collections-11022005.jar:/usr/local/gump/public/workspace/commons-jelly/target/commons-jelly-11022005.jar:/usr/local/gump/public/workspace/commons-jelly/jelly-tags/junit/target/commons-jelly-tags-junit-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons/jexl/dist/commons-jexl-11022005.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging-api.jar:/usr/local/gump/public/workspace/dom4j/build/dom4j.jar:/usr/local/gump/public/workspace/jaxen/target/jaxen-11022005.jar
-
[mkdir] Created dir: 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/classes

java:compile:
[echo] Compiling to 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/classes
[echo] 
==

  NOTE: Targetting JVM 1.4, classes
  will not run on earlier JVMs

==
  
[javac] Compiling 16 source files to 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/classes

java:jar-resources:

test:prepare-filesystem:
[mkdir] Created dir: 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/test-classes
[mkdir] Created dir: 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/test-reports

test:test-resources:
Copying 36 files to 
/home/gump/workspaces2/public/workspace/commons-jelly/jelly-tags/xml/target/test-classes

test:c

Re: concurrency and starting a synchronized block

2005-02-11 Thread Simon Kitching
On Fri, 2005-02-11 at 16:53 -0800, David Graham wrote:
> --- Stephen Colebourne <[EMAIL PROTECTED]> wrote:
> 
> > From: "Torsten Curdt" <[EMAIL PROTECTED]>
> > >Regarding the FastHashMap: since the
> > use is more or less discouraged ...why
> > not deprecate it?
> > 
> > Because it works on everything that its been tried on. 
> 
> It's more like no one has ever noticed it breaking.  I'm not certain
> whether we know it actually works.

In fact, it's more like "no-one has ever been sure enough that
FashHashMap has broken their app to file a bug report". If an
application misbehaves once in a while, it's almost impossible to
determine what the cause was, and therefore no bug report would ever get
filed. 

I think FastHashMap should definitely be deprecated if there is a
possibility of intermittent failure.

Regards,

Simon


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: concurrency and starting a synchronized block

2005-02-11 Thread David Graham

--- Stephen Colebourne <[EMAIL PROTECTED]> wrote:

> From: "Torsten Curdt" <[EMAIL PROTECTED]>
> >Regarding the FastHashMap: since the
> use is more or less discouraged ...why
> not deprecate it?
> 
> Because it works on everything that its been tried on. 

It's more like no one has ever noticed it breaking.  I'm not certain
whether we know it actually works.

> No-ones ever
> reported 
> a bug, its just that with all the double-checked locking theoretical
> ideas 
> we have to be cautious.
> 
> I believe that other parts of commons, including some used by struts,
> use 
> FastHashMap for example.

Unfortunately, Commons Validator still uses FastHashMap in some protected
instance variables where changing them to Map references might break
subclasses.  We have deprecated their usage and will be removing them in
future releases.

David


> 
> Stephen 
> 



__ 
Do you Yahoo!? 
Yahoo! Mail - Find what you need with new enhanced search.
http://info.mail.yahoo.com/mail_250

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: concurrency and starting a synchronized block

2005-02-11 Thread Stephen Colebourne
From: "Torsten Curdt" <[EMAIL PROTECTED]>
Regarding the FastHashMap: since the
use is more or less discouraged ...why
not deprecate it?
Because it works on everything that its been tried on. No-ones ever reported 
a bug, its just that with all the double-checked locking theoretical ideas 
we have to be cautious.

I believe that other parts of commons, including some used by struts, use 
FastHashMap for example.

Stephen 

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


svn commit: r153474 - in jakarta/commons: proper/feedparser/ sandbox/feedparser/

2005-02-11 Thread burton
Author: burton
Date: Fri Feb 11 16:17:25 2005
New Revision: 153474

URL: http://svn.apache.org/viewcvs?view=rev&rev=153474
Log:
init

Added:
jakarta/commons/proper/feedparser/
  - copied from r153472, jakarta/commons/sandbox/feedparser/
Removed:
jakarta/commons/sandbox/feedparser/


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


DO NOT REPLY [Bug 33529] - [dbcp] improved Exception nesting in ConnectionPool

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33529


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 23:24 ---
Good suggestion. The underlying exception was already visible in the stacktrace
but the message can be improved.

How about using the following catch block in the connect method.

} catch(SQLException e) {
throw e;
} catch(NoSuchElementException e) {
throw new SQLNestedException("Cannot get a connection, pool error: " +
e.getMessage(), e);
} catch(RuntimeException e) {
throw e;
} catch(Exception e) {
throw new SQLNestedException("Cannot get a connection, general error: " +
e.getMessage(), e);
}


-- Dirk 

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153468 - jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java

2005-02-11 Thread dirkv
Author: dirkv
Date: Fri Feb 11 14:23:56 2005
New Revision: 153468

URL: http://svn.apache.org/viewcvs?view=rev&rev=153468
Log:
Bugzilla 33529 [dbcp] improved Exception nesting in ConnectionPool

Modified:

jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java

jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java

Modified: 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java?view=diff&r1=153467&r2=153468
==
--- 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java
 (original)
+++ 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDataSource.java
 Fri Feb 11 14:23:56 2005
@@ -39,7 +39,7 @@
  * @author Glenn L. Nielsen
  * @author James House
  * @author Dirk Verbeeck
- * @version $Revision: 1.14 $ $Date: 2004/08/21 20:50:39 $
+ * @version $Revision: 1.14 $ $Date$
  */
 public class PoolingDataSource implements DataSource {
 
@@ -100,7 +100,7 @@
 } catch(SQLException e) {
 throw e;
 } catch(NoSuchElementException e) {
-throw new SQLNestedException("Cannot get a connection, pool 
exhausted", e);
+throw new SQLNestedException("Cannot get a connection, pool error 
" + e.getMessage(), e);
 } catch(RuntimeException e) {
 throw e;
 } catch(Exception e) {

Modified: 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java?view=diff&r1=153467&r2=153468
==
--- 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
 (original)
+++ 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
 Fri Feb 11 14:23:56 2005
@@ -180,11 +180,11 @@
 } catch(SQLException e) {
 throw e;
 } catch(NoSuchElementException e) {
-throw new SQLNestedException("Cannot get a connection, 
pool exhausted", e);
+throw new SQLNestedException("Cannot get a connection, 
pool error: " + e.getMessage(), e);
 } catch(RuntimeException e) {
 throw e;
 } catch(Exception e) {
-throw new SQLNestedException("Cannot get a connection, 
general error", e);
+throw new SQLNestedException("Cannot get a connection, 
general error: " + e.getMessage(), e);
 }
 }
 } else {



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33529] New: - [dbcp] improved Exception nesting in ConnectionPool

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33529

   Summary: [dbcp] improved Exception nesting in ConnectionPool
   Product: Commons
   Version: 1.2 Final
  Platform: PC
OS/Version: All
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Dbcp
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


(reported on commons-dev by Meikel Bisping on 26/01/2005)
--
I tried to establish a ConnectionPool with DBCP to an Informix
Database using your examples.
I always got the exeception "pool exhausted" when trying to open the
first connection.
I eventually found out during debugging that the problem was an older
Informix driver (that is generally still in use, though). 
It didn't support read-only mode and threw an SQLException even when
calling setReadOnly(false).
It would be helpful for users if in cases like that the actual
exception would be thrown, "pool exhausted" didn't help much.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Need karma for commons proper...

2005-02-11 Thread Henri Yandell
On Fri, 11 Feb 2005 11:47:53 -0800, Kevin A. Burton
<[EMAIL PROTECTED]> wrote:
> I'm trying to import feedparser from the sandbox and into commons proper
> and I get:

...

> 
> I assume this is because I need karma to commons proper?

Yep (sorry Simon :) ).

I've added karma and asked Infra to do an update.

Hen

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33528] - [dbcp] add DriverManager.invalidateConnection

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33528


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 23:18 ---
Method implemented
http://svn.apache.org/viewcvs.cgi/jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java?rev=153465&r1=132180&r2=153465&diff_format=h

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153465 - in jakarta/commons/proper/dbcp/trunk/src: java/org/apache/commons/dbcp/PoolingDriver.java test/org/apache/commons/dbcp/TestManual.java

2005-02-11 Thread dirkv
Author: dirkv
Date: Fri Feb 11 14:16:47 2005
New Revision: 153465

URL: http://svn.apache.org/viewcvs?view=rev&rev=153465
Log:
Bugzilla 33528 [dbcp] add DriverManager.invalidateConnection

Modified:

jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java

jakarta/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java

Modified: 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java?view=diff&r1=153464&r2=153465
==
--- 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
 (original)
+++ 
jakarta/commons/proper/dbcp/trunk/src/java/org/apache/commons/dbcp/PoolingDriver.java
 Fri Feb 11 14:16:47 2005
@@ -46,7 +46,7 @@
  *
  * @author Rodney Waldhoff
  * @author Dirk Verbeeck
- * @version $Revision: 1.12 $ $Date: 2004/05/17 18:39:44 $
+ * @version $Revision: 1.12 $ $Date$
  */
 public class PoolingDriver implements Driver {
 /** Register an myself with the [EMAIL PROTECTED] DriverManager}. */
@@ -174,7 +174,7 @@
 try {
 Connection conn = (Connection)(pool.borrowObject());
 if (conn != null) {
-conn = new PoolGuardConnectionWrapper(conn);
+conn = new PoolGuardConnectionWrapper(pool, conn);
 } 
 return conn;
 } catch(SQLException e) {
@@ -192,6 +192,23 @@
 }
 }
 
+public void invalidateConnection(Connection conn) throws SQLException {
+if (conn instanceof PoolGuardConnectionWrapper) { // normal case
+PoolGuardConnectionWrapper pgconn = (PoolGuardConnectionWrapper) 
conn;
+ObjectPool pool = pgconn.pool;
+Connection delegate = pgconn.delegate;
+try {
+pool.invalidateObject(delegate);
+} 
+catch (Exception e) { 
+}
+pgconn.delegate = null;
+}
+else {
+throw new SQLException("Invalid connection class");
+}
+}
+
 public int getMajorVersion() {
 return MAJOR_VERSION;
 }
@@ -222,10 +239,12 @@
  */
 private class PoolGuardConnectionWrapper extends DelegatingConnection {
 
+private ObjectPool pool;
 private Connection delegate;
 
-PoolGuardConnectionWrapper(Connection delegate) {
+PoolGuardConnectionWrapper(ObjectPool pool, Connection delegate) {
 super(delegate);
+this.pool = pool;
 this.delegate = delegate;
 }
 

Modified: 
jakarta/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java?view=diff&r1=153464&r2=153465
==
--- 
jakarta/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java
 (original)
+++ 
jakarta/commons/proper/dbcp/trunk/src/test/org/apache/commons/dbcp/TestManual.java
 Fri Feb 11 14:16:47 2005
@@ -35,7 +35,7 @@
  * based [EMAIL PROTECTED] PoolingDriver}.
  * @author Rodney Waldhoff
  * @author Sean C. Sullivan
- * @version $Revision: 1.21 $ $Date: 2004/07/11 19:09:50 $
+ * @version $Revision: 1.21 $ $Date$
  */
 public class TestManual extends TestConnectionPool {
 public TestManual(String testName) {
@@ -155,6 +155,22 @@
 }
 }
 
+public void testInvalidateConnection() throws Exception {
+Connection conn = 
DriverManager.getConnection("jdbc:apache:commons:dbcp:test");
+assertNotNull(conn);
+
+ObjectPool pool = driver.getConnectionPool("test");
+assertEquals(1, pool.getNumActive());
+assertEquals(0, pool.getNumIdle());
+
+PoolingDriver driver = (PoolingDriver) 
DriverManager.getDriver("jdbc:apache:commons:dbcp:");
+driver.invalidateConnection(conn);
+
+assertEquals(0, pool.getNumActive());
+assertEquals(0, pool.getNumIdle());
+assertTrue(conn.isClosed());
+}
+
 public void testLogWriter() throws Exception {
 PrintStream ps = System.out;
 PrintWriter pw = new PrintWriter(System.err);



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33528] New: - [dbcp] add DriverManager.invalidateConnection

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33528

   Summary: [dbcp] add DriverManager.invalidateConnection
   Product: Commons
   Version: 1.2 Final
  Platform: PC
OS/Version: All
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Dbcp
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


(Reported by Meikel Bisping on commons-dev 27/01/2005)

It is not possible to do:
driver.getConnectionPool("mypool").invalidateObject(con)

The connection wrappers don't allow this kind of behaviour.
A new method on DriverManager is needed to provide this feature.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: Need karma for commons proper...

2005-02-11 Thread Simon Kitching
On Fri, 2005-02-11 at 11:47 -0800, Kevin A. Burton wrote:
> I'm trying to import feedparser from the sandbox and into commons proper 
> and I get:
> 
>  > svn move 
> https://svn.apache.org/repos/asf/jakarta/commons/sandbox/feedparser 
> https://svn.apache.org/repos/asf/jakarta/commons/proper/feedparser
> Authentication realm:  ASF Committers
> Password for 'burton':
> Authentication realm:  ASF Committers
> Username: burton
> Password for 'burton':
> Authentication realm:  ASF Committers
> Username: burton
> Password for 'burton':
> svn: CHECKOUT of '/repos/asf/!svn/ver/153448/jakarta/commons/proper': 
> authorization failed (https://svn.apache.org)
> svn: Your commit message was left in a temporary file:
> svn:'svn-commit.2.tmp'
> 
> I assume this is because I need karma to commons proper?

It's probably because you haven't run svnpasswd. See the first paragraph
of the "Committer Subversion Access" section:
  http://www.apache.org/dev/version-control.html

Regards,

Simon


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [logging] discovery error handling

2005-02-11 Thread Ceki Gülcü
At 11:25 PM 2/8/2005, you wrote:
one of the drawbacks about JCL 1.0.x is the approach to handling errors
in the configuration and discovery mechanism. JCL falls down and (in
most commons use cases) takes the application with it. it also fails to
provide useful diagnostic information.
i've been considering for a while adopting a system for error handling
which allows a system property to be used to tune the exactly behaviour:
classic more would throw runtimes (as per now), silent more would
suppress all issues continuing to function as well as it is able and
diagnostic would print diagnostic information to System.out. though not
all environments would allow system properties to be set, i think that
this would improve matters for many common use cases.
Considering that logging often doubles as an error reporting system,
it must be very robust to begin with. One of the toughest problems
addressed in log4j version 1.3 the way log4j logs internally generated
messages using itself recursively. In certain special cases, log4j
needed a fallback logging API for its own use and that is how UGLI
came into being.
Coming back to JCL, printing something on the console in case of
errors is not enough. JCL must to also return a valid Logger back to
the application. Anyway, before camouflaging errors occurring during
Logger retrieval, I'd recommend that you fix the bug exposed by
Example 2 in my analysis [1]. In my opinion it cannot be fixed without
a major redesign and overhaul of JCL, but maybe I am wrong...
[1] http://www.qos.ch/logging/classloader.jsp
- robert
--
Ceki Gülcü
  The complete log4j manual: http://www.qos.ch/log4j/

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: new development: java 5 annotations library

2005-02-11 Thread Kevin A. Burton
James Rosen wrote:
I've been thinking about starting a library of commonly-used java 5
annotations, and I thought it might be good to have it be part of a
commons project.
I first proposed the idea to the sandbox:annotations group, but that
project is really more of a substitute for java 5 annotations.  They
suggested I pitch it to the commons:Lang people.
Some possible annotations:
@TODO - a standardized TODO
@immutable - an instance-level tag that implies that only certain
methods (those marked @non-mutating, perhaps?) can be called on the
instance.  This could be enforced by a security manager at runtime or
by a pre-processor at compile-time.
I'd love to hear any further thoughts on the idea.
 

@threadsafe ?
Kevin
--
Use Rojo (RSS/Atom aggregator).  Visit http://rojo.com. Ask me for an 
invite!  Also see irc.freenode.net #rojo if you want to chat.

Rojo is Hiring! - http://www.rojonetworks.com/JobsAtRojo.html
If you're interested in RSS, Weblogs, Social Networking, etc... then you 
should work for Rojo!  If you recommend someone and we hire them you'll 
get a free iPod!
   
Kevin A. Burton, Location - San Francisco, CA
  AIM/YIM - sfburtonator,  Web - http://peerfear.org/
GPG fingerprint: 5FB2 F3E2 760E 70A8 6174 D393 E84D 8D04 99F1 4412

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


svn commit: r153452 - jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt

2005-02-11 Thread ozeigermann
Author: ozeigermann
Date: Fri Feb 11 11:51:53 2005
New Revision: 153452

URL: http://svn.apache.org/viewcvs?view=rev&rev=153452
Log:
Added recent changes and prepared for rc1

Modified:
jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt

Modified: jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt?view=diff&r1=153451&r2=153452
==
--- jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt (original)
+++ jakarta/commons/proper/transaction/trunk/RELEASE-NOTES.txt Fri Feb 11 
11:51:53 2005
@@ -1,8 +1,8 @@
-Jakarta Commons Transaction Release 1.1 beta2
+Jakarta Commons Transaction Release 1.1 rc1
 -
 
 RELEASE NUMBER: 1.1b2
-RELEASE TAG / BRANCH: TRANSACTION_1_1_B2_RELEASE / none yet
+RELEASE TAG / BRANCH: TRANSACTION_1_1_RC1_RELEASE / none yet
 
 DESCRIPTION
 ---
@@ -19,9 +19,8 @@
 GENERAL RELEASE NOTES
 -
 
-This is the second beta of the Commons Transaction 1.1 feature release.
-It mainly fixes concurrency bugs not discovered before the first beta and
-adds test cases that reveal them.
+This is the first release candidate of the Commons Transaction 1.1 feature 
release.
+It mainly cleanes up some minor issues that come up while beta testing.
 
 Commons Transaction 1.1 aims at polishing (interface) oddities, improving
 locking and making the file store more flexible. Locking now is much more
@@ -72,6 +71,13 @@
 ENHANCEMENTS FROM 1.0 beta1
 ---
 - Many extensions for information about locks
+
+ENHANCEMENTS FROM 1.0 beta2
+---
+- Made GenericLockManager#checkLock, GenericLockManager#hasLock, 
GenericLockManager#release only check on existing locks
+  as creating a new lock was silly in that scenario
+- Split GenericLockManager#lock(Object ownerId, Object resourceId, int 
targetLockLevel, int compatibility, boolean preferred, long timeoutMSecs)
+  into two parts to make subclassing easier
 
 KNOWN ISSUES
 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Need karma for commons proper...

2005-02-11 Thread Kevin A. Burton
I'm trying to import feedparser from the sandbox and into commons proper 
and I get:

> svn move 
https://svn.apache.org/repos/asf/jakarta/commons/sandbox/feedparser 
https://svn.apache.org/repos/asf/jakarta/commons/proper/feedparser
Authentication realm:  ASF Committers
Password for 'burton':
Authentication realm:  ASF Committers
Username: burton
Password for 'burton':
Authentication realm:  ASF Committers
Username: burton
Password for 'burton':
svn: CHECKOUT of '/repos/asf/!svn/ver/153448/jakarta/commons/proper': 
authorization failed (https://svn.apache.org)
svn: Your commit message was left in a temporary file:
svn:'svn-commit.2.tmp'

I assume this is because I need karma to commons proper?
Kevin
--
Use Rojo (RSS/Atom aggregator).  Visit http://rojo.com. Ask me for an 
invite!  Also see irc.freenode.net #rojo if you want to chat.

Rojo is Hiring! - http://www.rojonetworks.com/JobsAtRojo.html
If you're interested in RSS, Weblogs, Social Networking, etc... then you 
should work for Rojo!  If you recommend someone and we hire them you'll 
get a free iPod!
   
Kevin A. Burton, Location - San Francisco, CA
  AIM/YIM - sfburtonator,  Web - http://peerfear.org/
GPG fingerprint: 5FB2 F3E2 760E 70A8 6174 D393 E84D 8D04 99F1 4412

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


svn commit: r153450 - jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java

2005-02-11 Thread ozeigermann
Author: ozeigermann
Date: Fri Feb 11 11:46:40 2005
New Revision: 153450

URL: http://svn.apache.org/viewcvs?view=rev&rev=153450
Log:
Split 

lock(Object ownerId, Object resourceId, int targetLockLevel, int compatibility, 
boolean preferred, long timeoutMSecs)

into two parts to make subclassing easier

Modified:

jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java

Modified: 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java?view=diff&r1=153449&r2=153450
==
--- 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
 (original)
+++ 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
 Fri Feb 11 11:46:40 2005
@@ -190,13 +190,19 @@
  */
 public void lock(Object ownerId, Object resourceId, int targetLockLevel, 
int compatibility,
 boolean preferred, long timeoutMSecs) throws LockException {
+timeoutCheck(ownerId);
+GenericLock lock = (GenericLock) atomicGetOrCreateLock(resourceId);
+doLock(lock, ownerId, resourceId, targetLockLevel, compatibility, 
preferred, timeoutMSecs);
+}
+
+protected void doLock(GenericLock lock, Object ownerId, Object resourceId, 
int targetLockLevel,
+  int compatibility, boolean preferred, long 
timeoutMSecs)
+{
 long now = System.currentTimeMillis();
 long waitEnd = now + timeoutMSecs;
 
 timeoutCheck(ownerId);
 
-GenericLock lock = (GenericLock) atomicGetOrCreateLock(resourceId);
-
 GenericLock.LockOwner lockWaiter = new GenericLock.LockOwner(ownerId, 
targetLockLevel,
 compatibility, preferred);
 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153449 - jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java

2005-02-11 Thread burton
Author: burton
Date: Fri Feb 11 11:43:03 2005
New Revision: 153449

URL: http://svn.apache.org/viewcvs?view=rev&rev=153449
Log:
...

Modified:

jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java

Modified: 
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java?view=diff&r1=153448&r2=153449
==
--- 
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
 (original)
+++ 
jakarta/commons/sandbox/feedparser/trunk/src/java/org/apache/commons/feedparser/FeedFilter.java
 Fri Feb 11 11:43:03 2005
@@ -182,7 +182,10 @@
 
 boolean hasFilterDecodedEntities = false;
 boolean hasFilterFoundUnknownEntity = false;
-
+
+//FIXME: note that when I was benchmarking this code that this showed 
up
+//as a MAJOR bottleneck so we might want to optimize it a little more.
+
 while ( m.find() ) {
 
 buff.append( content.substring( begin, m.start() ) );



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153448 - jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java

2005-02-11 Thread ozeigermann
Author: ozeigermann
Date: Fri Feb 11 11:42:32 2005
New Revision: 153448

URL: http://svn.apache.org/viewcvs?view=rev&rev=153448
Log:
Made #checkLock, #hasLock, #release only check on existing locks
as creating a new lock was silly in that scenario

Modified:

jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java

Modified: 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java?view=diff&r1=153447&r2=153448
==
--- 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
 (original)
+++ 
jakarta/commons/proper/transaction/trunk/src/java/org/apache/commons/transaction/locking/GenericLockManager.java
 Fri Feb 11 11:42:32 2005
@@ -139,11 +139,14 @@
  */
 public boolean checkLock(Object ownerId, Object resourceId, int 
targetLockLevel, boolean reentrant) {
 timeoutCheck(ownerId);
+boolean possible = true;
 
-GenericLock lock = (GenericLock) atomicGetOrCreateLock(resourceId);
-boolean possible = lock.test(ownerId, targetLockLevel,
-reentrant ? GenericLock.COMPATIBILITY_REENTRANT : 
GenericLock.COMPATIBILITY_NONE);
-
+GenericLock lock = (GenericLock) getLock(resourceId);
+if (lock != null) {
+possible = lock.test(ownerId, targetLockLevel,
+reentrant ? GenericLock.COMPATIBILITY_REENTRANT
+: GenericLock.COMPATIBILITY_NONE);
+}
 return possible;
 }
 
@@ -153,10 +156,12 @@
  */
 public boolean hasLock(Object ownerId, Object resourceId, int lockLevel) {
 timeoutCheck(ownerId);
+boolean owned = false;
 
-GenericLock lock = (GenericLock) atomicGetOrCreateLock(resourceId);
-boolean owned = lock.has(ownerId, lockLevel);
-
+GenericLock lock = (GenericLock) getLock(resourceId);
+if (lock != null) {
+owned = lock.has(ownerId, lockLevel);
+}
 return owned;
 }
 
@@ -290,9 +295,13 @@
  */
 public boolean release(Object ownerId, Object resourceId) {
 timeoutCheck(ownerId);
-GenericLock lock = (GenericLock) atomicGetOrCreateLock(resourceId);
-boolean released = lock.release(ownerId);
-removeOwner(ownerId, lock);
+boolean released = false;
+
+GenericLock lock = (GenericLock) getLock(resourceId);
+if (lock != null) {
+released = lock.release(ownerId);
+removeOwner(ownerId, lock);
+}
 return released;
 }
 



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33525] New: - [functor][Enhancement]Parent interface for all functors

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33525

   Summary: [functor][Enhancement]Parent interface for all functors
   Product: Commons
   Version: unspecified
  Platform: All
OS/Version: All
Status: NEW
  Severity: enhancement
  Priority: P2
 Component: Sandbox
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


Functor Enhancement: Parent interface for all functors

I would like to see an empty Functor interface as a marker for all functors.
This would be useful for storing functors in typed Maps, providing functor
metadata lookup (e.g. parameters and return types), and passing functors
generically as method parameters. It's an easy enhancement to make, and
potentially very powerful.

public interface Functor {}
public interface Predicate extends Functor 
public interface UnaryPredicate extends Functor 
public interface BinaryPredicate extends Functor 
public interface Function extends Functor 
public interface UnaryFunction extends Functor 
public interface BinaryFunction extends Functor 
public interface Procedure extends Functor
public interface UnaryProcedure extends Functor
public interface BinaryProcedure extends Functor

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [vfs] ISO 9660 file system?

2005-02-11 Thread filipdef
Could be a great place to start.. Looks like
this one is read-only though.

- Filip

> After some gooogling, I found a LGPL implementation...
>
> http://www.jnode.org
>
> http://cvs.sourceforge.net/viewcvs.py/jnode/jnode/fs/src/fs/org/jnode/fs/iso9660/
>
> -Mark
>
> [EMAIL PROTECTED] wrote:
>> Yep, this would be really great! Esp. if it could
>> be used to create a ISO 9660 file system.
>>
>> - Filip
>>
>>
>>>Yes, I would like to have that functionality; I'm sure this would also
>>>be a valuable addition for many other developers.
>>>
>>>-Brian
>>>
>>>
>>>Mark Diggory wrote:
>>>
>>>
Hi all,

Are there any interests in having a VFS that works with ISO 9660 files
used to burn data CDROMs/DVDs? I could really use a means to work with
building iso files in Java for a project I'm working on.

-Mark

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



>>>
>>>
>>>
>>>-
>>>To unsubscribe, e-mail: [EMAIL PROTECTED]
>>>For additional commands, e-mail: [EMAIL PROTECTED]
>>>
>>>
>>
>>
>>
>> -
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [vfs] ISO 9660 file system?

2005-02-11 Thread Mark R. Diggory
After some gooogling, I found a LGPL implementation...
http://www.jnode.org
http://cvs.sourceforge.net/viewcvs.py/jnode/jnode/fs/src/fs/org/jnode/fs/iso9660/
-Mark
[EMAIL PROTECTED] wrote:
Yep, this would be really great! Esp. if it could
be used to create a ISO 9660 file system.
- Filip

Yes, I would like to have that functionality; I'm sure this would also
be a valuable addition for many other developers.
-Brian
Mark Diggory wrote:

Hi all,
Are there any interests in having a VFS that works with ISO 9660 files
used to burn data CDROMs/DVDs? I could really use a means to work with
building iso files in Java for a project I'm working on.
-Mark
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


Re: [vfs] ISO 9660 file system?

2005-02-11 Thread filipdef
Yep, this would be really great! Esp. if it could
be used to create a ISO 9660 file system.

- Filip

>
> Yes, I would like to have that functionality; I'm sure this would also
> be a valuable addition for many other developers.
>
> -Brian
>
>
> Mark Diggory wrote:
>
>> Hi all,
>>
>> Are there any interests in having a VFS that works with ISO 9660 files
>> used to burn data CDROMs/DVDs? I could really use a means to work with
>> building iso files in Java for a project I'm working on.
>>
>> -Mark
>>
>> -
>> To unsubscribe, e-mail: [EMAIL PROTECTED]
>> For additional commands, e-mail: [EMAIL PROTECTED]
>>
>>
>>
>
>
>
> -
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
>
>


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: [vfs] ISO 9660 file system?

2005-02-11 Thread Brian Eubanks
Yes, I would like to have that functionality; I'm sure this would also 
be a valuable addition for many other developers.

-Brian
Mark Diggory wrote:
Hi all,
Are there any interests in having a VFS that works with ISO 9660 files 
used to burn data CDROMs/DVDs? I could really use a means to work with 
building iso files in Java for a project I'm working on.

-Mark
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


DO NOT REPLY [Bug 33524] New: - [configuration] Wrong primitive conversion in DataConfiguration

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33524

   Summary: [configuration] Wrong primitive conversion in
DataConfiguration
   Product: Commons
   Version: unspecified
  Platform: All
OS/Version: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: Configuration
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


The values in getLongArray(), getFloatArray() and getDoubleArray() are not
converted to the right type, they are all converted into integers.


Index: DataConfiguration.java
===
--- DataConfiguration.java  (revision 153417)
+++ DataConfiguration.java  (working copy)
@@ -825,7 +826,7 @@
 Iterator it = values.iterator();
 while (it.hasNext())
 {
-array[i++] = PropertyConverter.toLong(it.next()).intValue();
+array[i++] = PropertyConverter.toLong(it.next()).longValue();
 }
 }
 else
@@ -834,7 +835,7 @@
 {
 // attempt to convert a single value
 array = new long[1];
-array[0] = PropertyConverter.toLong(value).intValue();
+array[0] = PropertyConverter.toLong(value).longValue();
 }
 catch (ConversionException e)
 {
@@ -974,7 +975,7 @@
 Iterator it = values.iterator();
 while (it.hasNext())
 {
-array[i++] = PropertyConverter.toFloat(it.next()).intValue();
+array[i++] = PropertyConverter.toFloat(it.next()).floatValue();
 }
 }
 else
@@ -983,7 +984,7 @@
 {
 // attempt to convert a single value
 array = new float[1];
-array[0] = PropertyConverter.toFloat(value).intValue();
+array[0] = PropertyConverter.toFloat(value).floatValue();
 }
 catch (ConversionException e)
 {
@@ -1124,7 +1125,7 @@
 Iterator it = values.iterator();
 while (it.hasNext())
 {
-array[i++] = PropertyConverter.toDouble(it.next()).intValue();
+array[i++] = 
PropertyConverter.toDouble(it.next()).doubleValue();
 }
 }
 else
@@ -1133,7 +1134,7 @@
 {
 // attempt to convert a single value
 array = new double[1];
-array[0] = PropertyConverter.toDouble(value).intValue();
+array[0] = PropertyConverter.toDouble(value).doubleValue();
 }
 catch (ConversionException e)
 {

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33520] - [configuration] DataConfiguration.getXXXArray() fails on empty values

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33520





--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 17:28 ---
Created an attachment (id=14256)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=14256&action=view)
Patch to return the default value when a list/array is requested from an empty
String value


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[vfs] ISO 9660 file system?

2005-02-11 Thread Mark Diggory
Hi all,
Are there any interests in having a VFS that works with ISO 9660 files 
used to burn data CDROMs/DVDs? I could really use a means to work with 
building iso files in Java for a project I'm working on.

-Mark
-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]


[Jakarta Commons Wiki] Updated: Digester/TODO

2005-02-11 Thread commons-dev
   Date: 2005-02-11T07:45:02
   Editor: WendySmoak
   Wiki: Jakarta Commons Wiki
   Page: Digester/TODO
   URL: http://wiki.apache.org/jakarta-commons/Digester/TODO

   no comment

Change Log:

--
@@ -75,7 +75,7 @@
 
 Most Digester rules handle !DynaBeans just like ordinary bean classes, because 
they use the !BeanUtils reflection methods.
 
-However it has been reported that the !SetNextRule doesn't handle !DynaBeans 
well (see mail archives for 2004-11-25, subject '!DynaBean Hierarchy'). 
+However it has been reported that the !SetNextRule doesn't handle !DynaBeans 
well (see mail archives for 2004-11-25, subject '!DynaBean Hierarchy' 
http://www.mail-archive.com/commons-user@jakarta.apache.org/msg09115.html). 
 It would be nice to fix this if an elegant solution can be found (and fix any 
other rules that might not work well with !DynaBeans).
 
 === Add Missing XML Rules ===

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33475] - [configuration] ClassNotFoundException on Sun App Server

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33475


[EMAIL PROTECTED] changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution||FIXED




--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 16:21 ---
Patch applied.
Thank you very much!

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153409 - in jakarta/commons/proper/configuration/trunk: src/java/org/apache/commons/configuration/ConfigurationFactory.java xdocs/changes.xml

2005-02-11 Thread oheger
Author: oheger
Date: Fri Feb 11 07:19:52 2005
New Revision: 153409

URL: http://svn.apache.org/viewcvs?view=rev&rev=153409
Log:
Fix for issue 33475: Configuring digester to use context class loader

Modified:

jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/ConfigurationFactory.java
jakarta/commons/proper/configuration/trunk/xdocs/changes.xml

Modified: 
jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/ConfigurationFactory.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/ConfigurationFactory.java?view=diff&r1=153408&r2=153409
==
--- 
jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/ConfigurationFactory.java
 (original)
+++ 
jakarta/commons/proper/configuration/trunk/src/java/org/apache/commons/configuration/ConfigurationFactory.java
 Fri Feb 11 07:19:52 2005
@@ -45,7 +45,7 @@
  * @author mailto:[EMAIL PROTECTED]">Eric Pugh
  * @author mailto:[EMAIL PROTECTED]">Henning P. Schmiedehausen
  * @author mailto:[EMAIL PROTECTED]">Oliver Heger
- * @version $Id: ConfigurationFactory.java,v 1.20 2004/12/23 18:42:25 oheger 
Exp $
+ * @version $Id$
  */
 public class ConfigurationFactory
 {
@@ -152,6 +152,9 @@
 // awareness must be configured before the digester rules are 
loaded.
 configureNamespace(digester);
 }
+
+// Configure digester to always enable the context class loader
+digester.setUseContextClassLoader(true);
 // Put the composite builder object below all of the other objects.
 digester.push(builder);
 // Parse the input stream to configure our mappings

Modified: jakarta/commons/proper/configuration/trunk/xdocs/changes.xml
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/configuration/trunk/xdocs/changes.xml?view=diff&r1=153408&r2=153409
==
--- jakarta/commons/proper/configuration/trunk/xdocs/changes.xml (original)
+++ jakarta/commons/proper/configuration/trunk/xdocs/changes.xml Fri Feb 11 
07:19:52 2005
@@ -8,6 +8,12 @@
   
 
 
+  
+ConfigurationFactory now always configures digester to use the context
+classloader. This avoids problems in application server environments,
+which use their own version of digester. Thanks to Mike Colbert for the
+patch!
+  
   
 Added a new configuration, XMLPropertiesConfiguration, supporting the
 new XML format for java.util.Properties introduced in Java 1.5.



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33520] New: - [configuration] DataConfiguration.getXXXArray() fails on empty values

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33520

   Summary: [configuration] DataConfiguration.getXXXArray() fails on
empty values
   Product: Commons
   Version: unspecified
  Platform: All
OS/Version: All
Status: NEW
  Severity: normal
  Priority: P2
 Component: Configuration
AssignedTo: commons-dev@jakarta.apache.org
ReportedBy: [EMAIL PROTECTED]


The methods returning a primitive array in DataConfiguration do not work with
empty properties. They throw a ConversionException instead of returning an empty
array.

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



new development: java 5 annotations library

2005-02-11 Thread James Rosen
I've been thinking about starting a library of commonly-used java 5
annotations, and I thought it might be good to have it be part of a
commons project.
I first proposed the idea to the sandbox:annotations group, but that
project is really more of a substitute for java 5 annotations.  They
suggested I pitch it to the commons:Lang people.

Some possible annotations:
@TODO - a standardized TODO
@immutable - an instance-level tag that implies that only certain
methods (those marked @non-mutating, perhaps?) can be called on the
instance.  This could be enforced by a security manager at runtime or
by a pre-processor at compile-time.

I'd love to hear any further thoughts on the idea.

Thank you,
James Rosen

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



DO NOT REPLY [Bug 33475] - [configuration] ClassNotFoundException on Sun App Server

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33475





--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 14:13 ---
Thanks to everyone for weighing in and taking a look at the problem so quickly.




-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[digester2] general development strategy

2005-02-11 Thread Simon Kitching
Hi to all Digester types,

You may have seen a fair few commits to digester2 recently that change
fundamental stuff. I just want to point out that it is not my intention
to charge ahead with this without anyone's input; it's just that I don't
see anyone else who really cares to debate these changes with me at the
moment, so it's easier to commit them and see if people shout.

If someone wants to query or debate a change, I'm always happy to have
that discussion and potentially back out or change stuff that's been
committed. And of course when things stabilise more, I'll transition to
a "propose/commit" strategy rather than "commit/commit-more/..." :-)

The changes still generally follow the TO-DO list that's been here for
many months:
  http://wiki.apache.org/jakarta-commons/Digester/TODO

Regarding the current plans: I still have some work to do cleaning up
CallMethodRule and its CallParam associates. But that should then be the
bulk of the radical changes; the rest is just filling in the bits that
haven't yet been carried over from digester1. In other words, in a week
or so would be a good time for people to have a look at what's there if
they care, as things will be reasonably stable from then on.


Cheers,

Simon


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153393 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:25:31 2005
New Revision: 153393

URL: http://svn.apache.org/viewcvs?view=rev&rev=153393
Log:
Updates due to changes in CallMethodAction and the CallParam actions.

Modified:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java?view=diff&r1=153392&r2=153393
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/factory/ActionFactory.java
 Fri Feb 11 04:25:31 2005
@@ -261,7 +261,7 @@
  */
 public Action addCallMethod(String pattern, String methodName)
 throws InvalidRuleException {
-Action action = new CallMethodAction(methodName); 
+Action action = new CallMethodAction(methodName, 0); 
 return addRule(pattern, action);
 }
 
@@ -280,18 +280,12 @@
 return addRule(pattern, action);
 }
 
-
 /**
  * Add an "call method" rule for the specified parameters.
- * If paramCount is set to zero the rule will use
- * the body of the matched element as the single argument of the
- * method, unless paramTypes is null or empty, in this
- * case the rule will call the specified method with no arguments.
  *
  * @param pattern Element matching pattern
  * @param methodName Method name to be called
- * @param paramCount Number of expected parameters (or zero
- *  for a single parameter from the body of this element)
+ * @param paramCount Number of expected parameters.
  * @param paramTypes Set of Java class names for the types
  *  of the expected parameters
  *  (if you wish to use a primitive type, specify the corresonding
@@ -310,10 +304,6 @@
 
 /**
  * Add an "call method" rule for the specified parameters.
- * If paramCount is set to zero the rule will use
- * the body of the matched element as the single argument of the
- * method, unless paramTypes is null or empty, in this
- * case the rule will call the specified method with no arguments.
  *
  * @param pattern Element matching pattern
  * @param methodName Method name to be called
@@ -338,69 +328,47 @@
 }
 
 /**
- * Add a "call parameter" rule for the specified parameters.
+ * Set a call parameter from the body of the matched element.
  *
  * @param pattern Element matching pattern
  * @param paramIndex Zero-relative parameter index to set
- *  (from the body of this element)
  * @see CallParamAction
  */
-public Action addCallParam(String pattern, int paramIndex)
+public Action addCallParamBody(String pattern, int paramIndex)
 throws InvalidRuleException {
-Action action = new CallParamAction(paramIndex);
+Action action = new CallParamBodyAction(paramIndex);
 return addRule(pattern, action);
 }
 
 /**
- * Add a "call parameter" rule for the specified parameters.
+ * Set a call parameter from an xml attribute of the matched element.
  *
  * @param pattern Element matching pattern
  * @param paramIndex Zero-relative parameter index to set
- *  (from the specified attribute)
- * @param attributeName Attribute whose value is used as the
- *  parameter value
+ * @param attributeName Attribute whose value is used.
  * @see CallParamAction
  */
-public void addCallParam(
+public void addCallParamAttribute(
 String pattern, 
 int paramIndex, 
 String attributeName)
 throws InvalidRuleException {
 addRule(pattern,
-new CallParamAction(paramIndex, attributeName));
-}
-
-
-/**
- * Add a "call parameter" rule.
- * This will either take a parameter from the stack 
- * or from the current element body text. 
- *
- * @param paramIndex The zero-relative parameter number
- * @param fromStack Should the call parameter be taken from the top of the 
stack?
- * @see CallParamAction
- */
-public void addCallParam(
-String pattern, 
-int paramIndex, 
-boolean fromStack)
-throws InvalidRuleException {
-addRule(pattern,
-new CallParamAction(paramIndex, fromStack));
+new CallParamAttributeAction(paramIndex, attributeName));
 }
 
 /**
- * Add a "call parameter" rule that sets a parameter from the stack.
- * This takes a parameter from the given position on the stack.
+ * Set a call parameter from an object on the digester object

svn commit: r153392 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAction.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:24:12 2005
New Revision: 153392

URL: http://svn.apache.org/viewcvs?view=rev&rev=153392
Log:
No longer needed; the functionality has been split out into separate 
CallParam...Action classes.

Removed:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAction.java


-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153391 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:23:23 2005
New Revision: 153391

URL: http://svn.apache.org/viewcvs?view=rev&rev=153391
Log:
Functionality split out from old CallParamRule

Added:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAttributeAction.java
   (with props)

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamBodyAction.java
   (with props)

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamFromStackAction.java
   (with props)

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamLiteralAction.java
   (with props)

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamPathAction.java
   (with props)

Added: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAttributeAction.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAttributeAction.java?view=auto&rev=153391
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAttributeAction.java
 (added)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallParamAttributeAction.java
 Fri Feb 11 04:23:23 2005
@@ -0,0 +1,106 @@
+/* $Id$
+ *
+ * Copyright 2005 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+
+package org.apache.commons.digester2.actions;
+
+import org.xml.sax.Attributes;
+
+import org.apache.commons.digester2.ArrayStack;
+import org.apache.commons.logging.Log;
+
+import org.apache.commons.digester2.Context;
+import org.apache.commons.digester2.AbstractAction;
+import org.apache.commons.digester2.ParseException;
+
+/**
+ * Action which saves an xml attribute as a parameter value for a method
+ * invoked by a CallMethodRule.
+ */
+
+public class CallParamAttributeAction extends AbstractAction {
+
+/**
+ * The zero-relative index of the parameter we are saving.
+ */
+protected int paramIndex = 0;
+
+/**
+ * The attribute from which to save the parameter value
+ */
+protected String attributeName;
+
+// -
+// Constructor
+// -
+
+/**
+ * Construct a "call parameter" rule that will save the value of the
+ * specified attribute as the parameter value.
+ *
+ * @param paramIndex The zero-relative parameter number
+ * @param attributeName The name of the attribute to save
+ */
+public CallParamAttributeAction(int paramIndex, String attributeName) {
+this.paramIndex = paramIndex;
+this.attributeName = attributeName;
+}
+
+// -
+// Public Methods
+// -
+
+/**
+ * Process the start of this element.
+ *
+ * @param attributes The attribute list for this element
+ */
+public void begin(
+Context context,
+String namespace, String name, Attributes attributes)
+throws ParseException {
+Object paramValue = attributes.getValue(attributeName);
+
+Log log = context.getLogger();
+if (log.isDebugEnabled()) {
+StringBuffer sb = new StringBuffer("[CallParamAttributeAction]{");
+sb.append(context.getMatchPath());
+sb.append("} saving xml attribute [");
+sb.append(attributeName);
+sb.append("] value [");
+sb.append(paramValue);
+sb.append("]");
+log.debug(sb.toString());
+}
+
+Parameters params = (Parameters) 
context.peek(CallMethodAction.PARAM_STACK);
+params.put(paramIndex, paramValue);
+}
+
+/**
+ * Render a printable version of this Rule.
+ */
+public String toString() {
+StringBuffer sb = new StringBuffer("CallParamAttributeAction[");
+sb.append("paramIndex=");
+sb.append(paramIndex);
+sb.append(", attributeName=");
+sb.app

svn commit: r153389 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:22:11 2005
New Revision: 153389

URL: http://svn.apache.org/viewcvs?view=rev&rev=153389
Log:
New class used by CallMethodAction and CallParam actions to store parameter 
values.
This is much nicer than the old object[] approach.

Added:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java
   (with props)

Added: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java?view=auto&rev=153389
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java
 (added)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java
 Fri Feb 11 04:22:11 2005
@@ -0,0 +1,44 @@
+/* $Id$
+ *
+ * Copyright 2005 The Apache Software Foundation.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *  http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+
+package org.apache.commons.digester2.actions;
+
+import org.apache.commons.logging.Log;
+
+/**
+ * Represents a set of parameters to a method. Instances of this class
+ * are created when a CallMethodAction fires, and populated when
+ * CallParamAction instances fire. The values are then extracted in
+ * order to be passed to the target method.
+ */
+
+public class Parameters {
+private Object[] values;
+
+public Parameters(int size) {
+values = new Object[size];
+}
+
+public void put(int index, Object obj) {
+values[index] = obj;
+}
+
+public Object[] getValues() {
+return values;
+}
+}

Propchange: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/Parameters.java
--
svn:keywords = Id



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153388 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:21:15 2005
New Revision: 153388

URL: http://svn.apache.org/viewcvs?view=rev&rev=153388
Log:
Major rework.

Modified:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java?view=diff&r1=153387&r2=153388
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CallMethodAction.java
 Fri Feb 11 04:21:15 2005
@@ -1,19 +1,19 @@
-/* $Id: $
+/* $Id$
+ *
+ * Copyright 2001-2005 The Apache Software Foundation.
  *
- * Copyright 2001-2004 The Apache Software Foundation.
- * 
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *  http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
  * See the License for the specific language governing permissions and
  * limitations under the License.
- */ 
+ */
 
 
 package org.apache.commons.digester2.actions;
@@ -30,180 +30,165 @@
 import org.apache.commons.digester2.ParseException;
 
 /**
- * Action that calls a method on an object on the stack
- * (normally the top/parent object), passing arguments collected from 
- * subsequent ActionCallParam actions or from the body of this
- * element. 
- *
- * By using [EMAIL PROTECTED] #ActionCallMethod(String methodName)} 
- * a method call can be made to a method which accepts no
- * arguments.
- *
- * Incompatible method parameter types are converted 
- * using org.apache.commons.beanutils.ConvertUtils.
- * 
- *
- * Note that the target method is invoked when the end of
+ * An Action that calls a method on an object on the stack
+ * (normally the top/parent object), passing arguments collected from
+ * subsequent CallParam...Action actions.
+ * 
+ * Incompatible method parameter types are converted using the
+ * org.apache.commons.beanutils.ConvertUtils. library.
+ * 
+ * Note that the target method is invoked when the end of
  * the tag the CallMethodAction fired on is encountered, not when the
  * last parameter becomes available. This implies that rules which fire on
- * tags nested within the one associated with the CallMethodAction will 
+ * tags nested within the one associated with the CallMethodAction will
  * fire before the CallMethodAction invokes the target method. This behaviour 
is
- * not configurable. 
- *
- * Note also that if a CallMethodAction is expecting exactly one parameter
- * and that parameter is not available (eg CallParamAction is used with an
- * attribute name but the attribute does not exist) then the method will
- * not be invoked. If a CallMethodAction is expecting more than one parameter,
- * then it is always invoked, regardless of whether the parameters were
- * available or not (missing parameters are passed as null values).
+ * not configurable.
  */
 
 public class CallMethodAction extends AbstractAction {
 
-// --- Constructors
+// -
+// Instance Variables
+// -
+
+public static Context.StackId PARAM_STACK 
+= new Context.StackId(CallMethodAction.class, "ParamStack");
+
+private Context.ItemId PARAM_TYPES
+= new Context.ItemId(CallMethodAction.class, "ParamTypes", this);
 
 /**
- * Construct a "call method" instance with the specified method name.  The
- * parameter types (if any) default to java.lang.String.
- *
- * @param methodName Method name of the parent method to call
- * @param paramCount The number of parameters to collect, or
- *  zero for a single argument from the body of this element.
+ * location of the target object for the call, relative to the
+ * top of the digester object stack. The default value of zero
+ * means the target object is the one on top of the stack.
  */
-public CallMethodAction(String methodName,
-  int paramCount) {
-this(0, methodName, paramCount);
-}
+private int targetOffset;
 
 /**
- * Construct a "call method" instance with the specified method name.  The
- * parame

svn commit: r153387 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:20:27 2005
New Revision: 153387

URL: http://svn.apache.org/viewcvs?view=rev&rev=153387
Log:
Fixes due to change to Context.putInstanceData method.

Modified:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java?view=diff&r1=153386&r2=153387
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/actions/CreateObjectWithFactoryAction.java
 Fri Feb 11 04:20:27 2005
@@ -49,6 +49,10 @@
 // Instance Variables
 // -
 
+private Context.ItemId OBJECT_FACTORY_ITEM
+= new Context.ItemId(
+CreateObjectWithFactoryAction.class, "ObjectFactory", this);
+
 /**
  * The object creation factory (if any) explicitly provided to a
  * constructor. Note that (as per Action requirements) this value
@@ -252,7 +256,7 @@
 
 // now retrieve the cached ObjectFactory for this classname
 ObjectFactory factory = 
-(ObjectFactory) context.getInstanceData(this, "objectFactory");
+(ObjectFactory) context.getItem(OBJECT_FACTORY_ITEM);
 
 if (factory == null) {
 // this rule instance has never created a factory of this class
@@ -267,7 +271,7 @@
 factory = (ObjectFactory) clazz.newInstance();
 
 // and cache the object for later retrieval by this instance
-context.putInstanceData(this, "objectFactory", factory);
+context.putItem(OBJECT_FACTORY_ITEM, factory);
 } catch(ClassNotFoundException ex) {
 throw new ParseException(
 "Unable to load class '" + realClassName + "'", ex);



-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



svn commit: r153386 - jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java

2005-02-11 Thread skitching
Author: skitching
Date: Fri Feb 11 04:18:59 2005
New Revision: 153386

URL: http://svn.apache.org/viewcvs?view=rev&rev=153386
Log:
Reimplemented arbitrary item storage facility:
 * renamed putInstanceData --> putItem
 * renamed getInstanceData --> getItem
 * added ItemId class, removed InstanceItem class

Removed param stacks, as CallMethodRule now uses the generic
"scratch stacks" feature to store param info.

Cleaned up whitespace.

Modified:

jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java

Modified: 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java
URL: 
http://svn.apache.org/viewcvs/jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java?view=diff&r1=153385&r2=153386
==
--- 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java
 (original)
+++ 
jakarta/commons/proper/digester/branches/digester2/src/java/org/apache/commons/digester2/Context.java
 Fri Feb 11 04:18:59 2005
@@ -59,10 +59,10 @@
  * using those push/pop/peek/isEmpty methods that take a StackId parameter.
  * 
  * An object of class Foo which wishes to store data on a private scratch
- * stack for its own use should declare a StackId member variable then 
+ * stack for its own use should declare a StackId member variable then
  * later reference it:
  * 
- *private final Context.StackId WIDGET_STACK 
+ *private final Context.StackId WIDGET_STACK
  *  = new Context.StackId(Foo.class, "WidgetStack", this);
  *
  *context.push(WIDGET_STACK, someObject);
@@ -70,10 +70,10 @@
  *Object savedObject = context.pop(WIDGET_STACK);
  * 
  * 
- * If an class Bar wishes to share a scratch stack across all instances of 
+ * If an class Bar wishes to share a scratch stack across all instances of
  * itself, then it should declare a static StackId:
  * 
- *private static final Context.StackId GADGET_STACK 
+ *private static final Context.StackId GADGET_STACK
  *  = new Context.StackId(Bar.class, "GadgetStack");
  * 
  * 
@@ -94,14 +94,14 @@
  * Create an instance which has no specific owner object.
  */
 public StackId(Class sourceClass, String desc) {
-this.desc = sourceClass.getName() + ":" + desc; 
+this.desc = sourceClass.getName() + ":" + desc;
 }
 
 /**
  * Create an instance which has an owner object.
  */
 public StackId(Class sourceClass, String desc, Object owner) {
-this.desc = sourceClass.getName() + ":" + desc 
+this.desc = sourceClass.getName() + ":" + desc
+ ":" + System.identityHashCode(owner);
 }
 
@@ -115,19 +115,41 @@
 }
 }
 
-   /**
-* See method [EMAIL PROTECTED] #putInstanceData}.
-*/
-private static class InstanceItem {
-public Object key;
-public Map map;
-
-public InstanceItem(Object key, Map map) {
-this.key = key;
-this.map = map;
+/**
+ * The context provides "scratch storage" that any other object with
+ * access to the context can use for storing data; instances of this
+ * class are used to identify which "scratch item" is to be used when
+ * using the getItem/setItem methods.
+ * 
+ * An object of class Foo which wishes to store a private data item
+ * should declare an ItemId member variable then later reference it:
+ * 
+ *private final Context.ItemId MY_ITEM
+ *  = new Context.ItemId(Foo.class, "MyItem", this);
+ *
+ *context.putItem(MY_ITEM, someObject);
+ *
+ *Object savedObject = context.getItem(MY_ITEM);
+ * 
+ * 
+ * See method [EMAIL PROTECTED] #putItem} for more information.
+ */
+public static class ItemId {
+private String desc;
+
+public ItemId(Class sourceClass, String desc) {
+this.desc = sourceClass.getName() + ":" + desc;
 }
-}
 
+public ItemId(Class sourceClass, String desc, Object owner) {
+this.desc = sourceClass.getName() + ":" + desc
+   + ":" + System.identityHashCode(owner);
+}
+
+public String toString() {
+return desc;
+}
+}
 
 // ---
 // Instance Variables
@@ -239,15 +261,9 @@
 
 /**
  * Place where other objects can store any data they like during a parse.
- * See method [EMAIL PROTECTED] #putInstanceData} for more information.
- */
-private List instanceData = new ArrayList();
-
-/**
- * The parameters stack being utilized by CallMethodAction and
- * CallParamAction.
+ * See 

DO NOT REPLY [Bug 33336] - [io] CountingInputStream.getCount() often returns invalid values

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=6





--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 12:36 ---
Created an attachment (id=14253)
 --> (http://issues.apache.org/bugzilla/attachment.cgi?id=14253&action=view)
Patch containing source file and corresponding testcase modifications


-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



Re: concurrency and starting a synchronized block

2005-02-11 Thread Torsten Curdt
Guys,
thanks everyone for the useful input!
Since this part of the code is really(!)
a hotspot I am really careful with any
additional overhead.
...but looks like I need to have a
closer look into the concurrent package
then.
Regarding the FastHashMap: since the
use is more or less discouraged ...why
not deprecate it?
...and I know for commons stuff we
want to have as less dependencies as
possible ...but since the concurrent
package comes with 1.5.
What about using the stuff in commons?
...or the backport? ...just some RT.
cheers
--
Torsten


signature.asc
Description: OpenPGP digital signature


DO NOT REPLY [Bug 32643] - [fileupload] FileUploadException, Stream ended unexpectedly

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=32643





--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 11:59 ---
Hi,
I've exactly the same problem on Sybase EAServer 5.1 on Win2k Server.
It arrives 1/10 times on different file sizes and types.
Any idee ?
Thanks a lot,
David

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]



[GUMP@brutus]: Project commons-id (in module jakarta-commons-sandbox) failed

2005-02-11 Thread Adam Jack
To whom it may engage...

This is an automated request, but not an unsolicited one. For 
more information please visit http://gump.apache.org/nagged.html, 
and/or contact the folk at [EMAIL PROTECTED]

Project commons-id has an issue affecting its community integration.
This issue affects 1 projects,
 and has been outstanding for 6 runs.
The current state of this project is 'Failed', with reason 'Build Failed'.
For reference only, the following projects are affected by this:
- commons-id :  Commons Identifier Package


Full details are available at:

http://brutus.apache.org/gump/public/jakarta-commons-sandbox/commons-id/index.html

That said, some information snippets are provided here.

The following annotations (debug/informational/warning/error messages) were 
provided:
 -DEBUG- Sole output [commons-id-11022005.jar] identifier set to project name
 -DEBUG- (Gump generated) Maven Properties in: 
/usr/local/gump/public/workspace/jakarta-commons-sandbox/id/build.properties
 -INFO- Failed with reason build failed
 -DEBUG- Maven POM in: 
/usr/local/gump/public/workspace/jakarta-commons-sandbox/id/project.xml
 -DEBUG- Maven project properties in: 
/usr/local/gump/public/workspace/jakarta-commons-sandbox/id/project.properties
 -INFO- Failed to extract fallback artifacts from Gump Repository



The following work was performed:
http://brutus.apache.org/gump/public/jakarta-commons-sandbox/commons-id/gump_work/build_jakarta-commons-sandbox_commons-id.html
Work Name: build_jakarta-commons-sandbox_commons-id (Type: Build)
Work ended in a state of : Failed
Elapsed: 1 sec
Command Line: maven --offline jar 
[Working Directory: /usr/local/gump/public/workspace/jakarta-commons-sandbox/id]
CLASSPATH: 
/opt/jdk1.4/lib/tools.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-jmf.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-swing.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-apache-resolver.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-trax.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-junit.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant-nodeps.jar:/usr/local/gump/public/workspace/ant/dist/lib/ant.jar:/usr/local/gump/public/workspace/jakarta-commons/discovery/dist/commons-discovery.jar:/usr/local/gump/public/workspace/dist/junit/junit.jar:/usr/local/gump/public/workspace/xml-commons/java/build/resolver.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant-launcher.jar:/usr/local/gump/public/workspace/ant/bootstrap/lib/ant.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging.jar:/usr/local/gump/public/workspace/jakarta-commons/logging/dist/commons-logging-api.jar
-
 __  __
|  \/  |__ _Apache__ ___
| |\/| / _` \ V / -_) ' \  ~ intelligent projects ~
|_|  |_\__,_|\_/\___|_||_|  v. 1.0.2

org.apache.maven.MavenException: Error reading XML or initializing
at org.apache.maven.MavenUtils.getProject(MavenUtils.java:156)
at org.apache.maven.MavenUtils.getProject(MavenUtils.java:122)
at 
org.apache.maven.MavenSession.initializeRootProject(MavenSession.java:232)
at org.apache.maven.MavenSession.initialize(MavenSession.java:172)
at org.apache.maven.cli.App.doMain(App.java:475)
at org.apache.maven.cli.App.main(App.java:1239)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:324)
at com.werken.forehead.Forehead.run(Forehead.java:551)
at com.werken.forehead.Forehead.main(Forehead.java:581)
--- Nested Exception ---
java.io.FileNotFoundException: Parent POM not found: 
/home/gump/workspaces2/public/workspace/jakarta-commons-sandbox/sandbox-build/project.xml
at org.apache.maven.MavenUtils.getNonJellyProject(MavenUtils.java:230)
at org.apache.maven.MavenUtils.getProject(MavenUtils.java:143)
at org.apache.maven.MavenUtils.getProject(MavenUtils.java:122)
at 
org.apache.maven.MavenSession.initializeRootProject(MavenSession.java:232)
at org.apache.maven.MavenSession.initialize(MavenSession.java:172)
at org.apache.maven.cli.App.doMain(App.java:475)
at org.apache.maven.cli.App.main(App.java:1239)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at 
sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at 
sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:324)
at com.werken.forehead.Forehead.run(Forehead.java:551)
at com.werken.forehead.Forehead.main(Forehead.java:581)

You have encountered an unknown error run

DO NOT REPLY [Bug 33475] - [configuration] ClassNotFoundException on Sun App Server

2005-02-11 Thread bugzilla
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
.
ANY REPLY MADE TO THIS MESSAGE WILL NOT BE COLLECTED AND·
INSERTED IN THE BUG DATABASE.

http://issues.apache.org/bugzilla/show_bug.cgi?id=33475





--- Additional Comments From [EMAIL PROTECTED]  2005-02-11 10:16 ---
Go ahead and apply, I'm not fully operational with SVN yet ;)

-- 
Configure bugmail: http://issues.apache.org/bugzilla/userprefs.cgi?tab=email
--- You are receiving this mail because: ---
You are the assignee for the bug, or are watching the assignee.

-
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]