On Wed, Jun 11, 2003 at 06:36:54PM -0700, Shankar Unni wrote:
> With the latest CVS tip (1.3a3), I can't bring up the desktop. Several bugs
> there, actually.
I've been looking at exactly this stuff this evening...
> PROBLEM 1: In Name.java, when looking up builtin scripts, it does a
> getResourceOnStream of "commands/" + the command name, instead of
> "/bsh/commands/" + the command name.
That works because it's loaded relative to a class in the package "bsh".
On my todo list for 1.3. is to add a real command path (and patches to do just
that have been sumitted).
> PROBLEM 2: Anyway, after I fix this, it dies while loading desktop, with an
> IllegalArgumentException. The precise statement that dies is:
Yes. I just realized that the method resolution stuff had side effects
that were important. I am debating about the best way to fix this... and
I've been doing some cleanup while I'm in there. This stuff was all way too
cryptic and I hope to make it cleaner. We may have to use
NameSpace.getAssignableForm() in invokeMethod(). I just want to check the
performance in the common case and make sure it's not a big penalty.
> I'd like to just open a bunch of bugs about this, but these are all sort of
> related, so I wonder if it should be 1 bug ("desktop does not come up" or 3.
Thanks for the timely report... this bug is my top priority at the moment.
I thought I had the caching stuff nailed down, but it came back to bite me
one more time ;) Oh, and I've added a test to the test suite for this case...
was surprised that nothing in there caught it before.