Minor com.sun.jndi.dns cleanup

Florian Weimer fweimer at redhat.com
Wed Feb 19 19:16:29 UTC 2014


On 02/19/2014 06:43 PM, Alan Bateman wrote:
> On 19/02/2014 12:56, Florian Weimer wrote:
>>
>> It's encoded as a bit field, like this:
>>
>>   0000 0000  end of name
>>   00xx xxxx  regular label of length xxxxxx
>>   11yy yyyy  upper 6 bits of a compression reference
>>              (followed by another byte containing the lower 8 bits)
>>   01zz zzzz  used to be invalid, then extended type, then invalid again
>>   10zz zzzz  invalid
>>
>> I could rename the variable to typeAndLength.

> Right, just too easy to mis-read compared to the original code. Using
> typeAndLen would be okay, so would "int len = type" and using len when
> constructing the String. Anything to make it easier for future maintainers.

Updated webrev:

<http://fweimer.fedorapeople.org/openjdk/8035105.2/>

> So I think we are almost done and this and I can push it for you.

Thanks.

-- 
Florian Weimer / Red Hat Product Security Team



More information about the core-libs-dev mailing list