Re: Small patch for dnsjnio compatibility

[Stefano B. <sou...@ba...>]

Brian Wellington ha scritto:
> On Mon, 8 Jun 2009, Al...@no... wrote:
> 
>> I'd like to sound you out on a small potential patch for dnsjava. The
>> patch would allow dnsjnio to exist in its own namespace, and not to have
>> to declare classes in the org.xbill.DNS package (an ugly hack which is
>> starting to get in the way).
>>
>> Basically, the patch would involve changing the access of some classes in
>> the org.xbill.DNS package.
>>
>> If you thought you might be able to include such a patch, I'd happily
>> knock one up against the latest dnsjava code (or the 2.0.6 release -
>> whichever was easiest for you) and submit it before the next dnsjava
>> release. Does this sound reasonable?
> 
> Without knowing what the changes are, it's hard to say.
> 
> Brian

Hi Brian,

maybe a small list is better than a diff in this case. It is all about
package visibility to be changed to public in order to allow usage in
different packages.

dnsjnio needs all of this to be made public:

DClass.check method
Type.check method
Mnemonic class (for the Mnemonic.toInteger(dclass) usage)
Message.TSIG_VERIFIED field
Message.tsigState (we need to set this to TSIG_VERIFIED in a custom
resolver)

Do you think this is feasible? I can provide a diff if you think that
they can be made public.

Stefano