Re: Review request for JDK-8191878: Reduce code duplication in BeanLinker

[Sundararajan Athijegannathan]

Looks good

-Sundar

On 27/11/17, 12:21 AM, Attila Szegedi wrote:

Hey folks,

I’m able to spend some free time again on OpenJDK development, and one thing I 
have on my plate is adding a REMOVE operation to Dynalink.

This CR however, is not yet it: that will require a CCC sponsorship etc. as 
I’ll be adding a new enum value to StandardOperations, and I will follow up 
separately once I’m done with it.

What this is a refactoring of BeanLinker code to eliminate most of the code 
duplication between getElementGetter and getElementSetter methods. This would 
be justified on its own, but once I need to add the getElementRemover method, 
it will really pay off as then we won’t have triplication of similar code.

With that in mind, please review JDK-8191878 "Reduce code duplication in BeanLinker" 
at  
for

Thanks,
   Attila.

Re: Review request for JDK-8191878: Reduce code duplication in BeanLinker

[Hannes Wallnöfer]

Hi Attila,

> Am 26.11.2017 um 19:51 schrieb Attila Szegedi :
> 
> I’m able to spend some free time again on OpenJDK development, and one thing 
> I have on my plate is adding a REMOVE operation to Dynalink.
> 

great!

> This CR however, is not yet it: that will require a CCC sponsorship etc. as 
> I’ll be adding a new enum value to StandardOperations, and I will follow up 
> separately once I’m done with it. 

I was going to point you to the new CSR issue type, but I see you found your 
way already.

> What this is a refactoring of BeanLinker code to eliminate most of the code 
> duplication between getElementGetter and getElementSetter methods. This would 
> be justified on its own, but once I need to add the getElementRemover method, 
> it will really pay off as then we won’t have triplication of similar code.
> 
> With that in mind, please review JDK-8191878 "Reduce code duplication in 
> BeanLinker" at  for 
> 
> 

+1

Hannes

> Thanks,
>  Attila.
> 

Review request for JDK-8191878: Reduce code duplication in BeanLinker

[Attila Szegedi]

Hey folks,

I’m able to spend some free time again on OpenJDK development, and one thing I 
have on my plate is adding a REMOVE operation to Dynalink.

This CR however, is not yet it: that will require a CCC sponsorship etc. as 
I’ll be adding a new enum value to StandardOperations, and I will follow up 
separately once I’m done with it. 

What this is a refactoring of BeanLinker code to eliminate most of the code 
duplication between getElementGetter and getElementSetter methods. This would 
be justified on its own, but once I need to add the getElementRemover method, 
it will really pay off as then we won’t have triplication of similar code.

With that in mind, please review JDK-8191878 "Reduce code duplication in 
BeanLinker" at  for 


Thanks,
  Attila.