review for 7086585: make Java field injection more flexible
tom.rodriguez at oracle.com
Fri Sep 9 10:26:38 PDT 2011
On Sep 9, 2011, at 10:21 AM, Vladimir Kozlov wrote:
> Can we just have one class FieldDescriptor. Methods and fields are almost the same.
They serve different functions. RawFieldDescriptor hides and describes the layout of the fields array itself. fieldDescriptor serves as a self contained object for talking about a field. Think of it as a field handle. So they have to be separate types but they could be more closely related. Much of fieldDescriptor could be replaced by subclassing RawFieldDescriptor.
> Tom Rodriguez wrote:
>> I had a side conversion with Coleen about this and she suggested moving RawFieldDescriptor into it's own file, which I think is a good idea. She also commented on the confusion between RawFieldDescriptor and fieldDescriptor because of the name similarity. I had looked at making fieldDescriptor a subclass of RawField at one point and I think I might be able to make that work. I also think the RawFieldDescriptor name is bad. How about FieldLayout? Any opinions? I had also considered fixing the capitalization of fieldDescriptor.
>> On Sep 8, 2011, at 3:47 PM, John Rose wrote:
>>> I've reviewed it. It's excellent. Extra thanks for getting rid of instanceKlass::next_offset. -- John
>>> On Sep 8, 2011, at 1:38 PM, Tom Rodriguez wrote:
>>>> 1577 lines changed: 621 ins; 612 del; 344 mod; 65541 unchg
>>>> 7086585: make Java field injection more flexible
More information about the hotspot-compiler-dev