[John Keiser]
> libnative can be dealt with easily with a switch, or even
> auto-discovery of whether it is Classpath or JDK classes.zip you're
> working with.
I would like to have auto-discovery if that is nessesary. I want to
avoid having to do that.
> Throwable's only dependency is "message", and you really should just
> call getMessage() for that--an extra method call here doesn't matter
> since it is not a critical path item. Exceptions themselves are by
> definition exceptional, and it is even rarer for an exception to
> reach the top-level handler. In fact, that's what I'd like you to
> do, please, so that I can change the fieldname back to message and
> retain the integrity of the private fields.
Sounds reasonable. I'll try to make time to do that change.
> This leaves only String. If we change fieldnames right now, it'll
> work fine. But if we find some cool new (better) way to represent
> strings that speeds them up even more, str, len and offset might not
> cut it.
Then we will deal with that probem when it arises. Until then, I
would like Japhar with Classpath to be runtime compatible with JDK.
> I think String is one place we'll have to find some other way of
> working. I guess the only thing to do there if you want runtime
> compatibility is to use if(classpath_enabled) instead of #ifdef
> FOR_CLASSPATH.
One of the reasons I want to avoid #ifdefs is that they produce
untested, uncompiled and over time buggy code. There must be a very
strong need to use ifdefs. I don't think it is necessary in this
case.
Another thing. I am trying to figure out why --enable-classpath don't
work at my place. There may be something wrong with the current patch
in CVS. JFYI.
BTW: I have lost track of which changes are required to get Classpath
running, and as I am unable to compile Classpath myself, I can not
test to find out. Could you make a new patch for me to look at?
BTW2: I'm on the Japhar list. You are sending me two mails. :-)
--
##> Petter Reinholdtsen <## | [EMAIL PROTECTED]
O- <SCRIPT Language="Javascript">window.close()</SCRIPT>
http://www.hungry.com/~pere/ | Go Mozilla, go! Go!
