search

lishunli / projectlombok

Automatically exported from code.google.com/p/projectlombok
0 stars 0 forks source link

@Delegate should have include fields option similar to excludes #595

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1.@Delegate has an excludes property, similarly there should be an includes 
property option, because if the delegate class is big, and we want to include 
only a few fields, excludes list can become very big.
2.
3.

What is the expected output? What do you see instead?

What version of the product are you using? On what operating system?

Please provide any additional information below.

Original issue reported on code.google.com by TVP.Sath...@gmail.com on 22 Oct 2013 at 11:34

GoogleCodeExporter commented 9 years ago 
This is an enhancement, didn't know how to change the type

Original comment by TVP.Sath...@gmail.com on 22 Oct 2013 at 11:35

GoogleCodeExporter commented 9 years ago 
Isn't this exactly what Delegate.types does?

Original comment by Maaarti...@gmail.com on 23 Oct 2013 at 7:54

GoogleCodeExporter commented 9 years ago 
[deleted comment]
GoogleCodeExporter commented 9 years ago 
Isn't there a better way to include/exclude getters/setters rather than 
declaring them in an interface and declaring them as exclude/type in delegate? 
For business-logic methods, it looks reasonable to extract them into interface 
and control delegation, but not so for getters/setters. I am closing this issue 
and raising another one as :add better way to exclude/include getters/setters 
of composed class when using @Delegate". Hope that is OK to be picked up.

Original comment by TVP.Sath...@gmail.com on 24 Oct 2013 at 7:46

GoogleCodeExporter commented 9 years ago 
How to close the issue or edit the title of the issue?

Original comment by TVP.Sath...@gmail.com on 24 Oct 2013 at 7:55

GoogleCodeExporter commented 9 years ago 
Closing per request of reporter.

Original comment by askon...@gmail.com on 4 Apr 2014 at 11:12