[MDSAL-577] Integrate AugmentationHolder into Augmentable Created: 20/Jul/20 Updated: 03/Aug/20 Resolved: 03/Aug/20 |
|
| Status: | Resolved |
| Project: | mdsal |
| Component/s: | Binding codegen |
| Affects Version/s: | None |
| Fix Version/s: | 7.0.0 |
| Type: | Improvement | Priority: | High |
| Reporter: | Robert Varga | Assignee: | Robert Varga |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Epic Link: | Make class loading predictable |
| Description |
|
Since we have predictable class loading, there is no need to play weird class loader tricks: we do not need a class reference grow representations. All implementations already use AugmentationHolder, hence there is no point in pretending these contracts are separate. Merge them and eliminate AugmentationHolder, along with any complexity it implies. This means Augmentable.augmentation() should become a default method calling down to Augmentable.augmentations(). |