classpath-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [cp-patches] FYI: ParserDelegator compatibility fix.


From: Roman Kennke
Subject: Re: [cp-patches] FYI: ParserDelegator compatibility fix.
Date: Tue, 12 Jul 2005 13:00:33 +0200
User-agent: Mozilla Thunderbird 1.0.2 (X11/20050331)

Hi,


Roman says this is required to compile the class with JDK1.3's javac
(and only for that compiler).

> could you please add a small comment that this is a workaround for 1.3's
> javac to make sure no one removes it when the next spring clean happens. :)

I found this one with the JDK1.3 compiler. But honestly, besides the compiler issue, this helps understand the code. Intuitivly one (at least I) would probably think that the static dtd field of the surrounding class is accessed, but instead the protected member field dtd of the super class is meant. So yes, this fixes the compiler thingy, but more importantly it improves readability and understandability of the code.

/Roman




reply via email to

[Prev in Thread] Current Thread [Next in Thread]