These are in, thanks. Kernel's on the chopping block though.
On Thursday, July 18, 2002, at 09:27 AM, Fernando Padilla wrote:
Hello.
So I'm going through the code to really learn what is really going on, so I can add features. As I am going I'll be doing janitorial work.
Cleaning out unused/useless code. Looking for conceptual bugs. Refactoring and Streamlining.
So just looking at Kernel and Configuration, I've found a lot of little modifications to be made:
Configuration - had a unused 'Modified' Map, and thus defunct 'setDirty' method. - all of the 'setValue' methods had bad signatures - and as an aside, these are all unused.
Kernel - system.xml parsing code had minor cruft, unused variables, etc - it exposed a Runnable interface, which is not good programming - moved to internal class - then saw that the runnable interface was just for task scheduling - replaced with java.util.Timer and java.util.TimerTask implementation of Task scheduling.
So except for the java.util.Timer, it was all interchangeable modifications. Could someone get me feedback and/or check these in. I would really appreciate it because I don't like being too much out of sync with CVS.
Fernando
Kimbro Staken Java and XML Software, Consulting and Writing http://www.xmldatabases.org/ Apache Xindice native XML database http://xml.apache.org/xindice XML:DB Initiative http://www.xmldb.org
