commons-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 33926] - [configuration] The Iterator returned by getKeys() should support remove()
Date Sat, 25 Jun 2005 15:00:41 GMT
DO NOT REPLY TO THIS EMAIL, BUT PLEASE POST YOUR BUG·
RELATED COMMENTS THROUGH THE WEB INTERFACE AVAILABLE AT
<http://issues.apache.org/bugzilla/show_bug.cgi?id=33926>.
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=33926





------- Additional Comments From oliver.heger@t-online.de  2005-06-25 17:00 -------
With this issue floating around for a while I changed my mind. The patch I
provided would solve the problem, but it would also break code that inherits
from AbstractConfiguration (by introducing a new abstract method) - by following
some recent discussions on the dev list I am now much more concerned about
backwards compatibility.

Because I don't consider this remove() feature of an iterator as an important
one I tend to resolve this issue by only adding a comment to the getKeys()
method stating in an implementation note that calling remove() on a returned
iterator will probably not work on most concrete implementations and if it
works, it is no real replacement for calling a configuration's clearProperty()
method.

I doubt whether this undocumented remove feature is often used. By adding this
warning to the documentation we won't break existing code, but we state clearly
that usage might cause problems in the future. With this approach I would also
resolve the related issue 34333.

Other opinions?

-- 
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: commons-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: commons-dev-help@jakarta.apache.org


Mime
View raw message