Details
-
Improvement
-
Status: Confirmed
-
Medium
-
Resolution: Unresolved
-
None
-
None
Description
BindingReflections is a relict from the time when we did not have proper mdsal-binding-runtime support.
The functions of binding-runtime-spi, binding-dom-codec and binding-dom-adapter are making extensive use BindingReflections to match observed Binding classes to their DOM mapping.
This is counter-productive to AOT environments: mdsal-binding-runtime and its support classes should give us the information required to function in a manner which is completely independent of java.lang.reflect – rendering BindingReflections completely unncessary.
Attachments
Issue Links
- is blocked by
-
BGPCEP-1016 Do not use BindingReflections.findQName() for AFI/SAFI
-
- Resolved
-
-
MDSAL-783 Do not use BindingReflections.getQName
-
- Resolved
-
-
-
- Confirmed
-
Gerrit Reviews
| # | Subject | Branch | Project | Status | CR | V |
|---|---|---|---|---|---|---|
| 103340,2 | Move BindingReflections.getChildrenClass* | master | mdsal | Status: MERGED | +2 | +1 |
| 103343,4 | Hide BindingReflections.isRpcMethod() | master | mdsal | Status: ABANDONED | 0 | -1 |
| 103345,4 | Hide BindingReflections.isNotification() | master | mdsal | Status: MERGED | +2 | +1 |
| 106584,2 | Remove BindingReflections.getModelRootPackageName() | master | mdsal | Status: MERGED | +2 | +1 |
| 106585,2 | Hide BindingReflections.getModuleInfo() | master | mdsal | Status: MERGED | +2 | +1 |
| 106587,3 | Move BindingReflections.findAugmentationTarget() | master | mdsal | Status: MERGED | +2 | +1 |
| 106588,5 | Move BindingReflections.cacheModuleInfos() | master | mdsal | Status: MERGED | +2 | +1 |
| 106589,5 | Remove BindingReflections.loadModuleInfos() | master | mdsal | Status: MERGED | +2 | +1 |
| 106590,5 | Remove BindingReflections.getModuleInfoClassName() | master | mdsal | Status: MERGED | +2 | +1 |
| 106591,3 | Remove BindingReflections.isRpcMethod() | master | mdsal | Status: MERGED | +2 | +1 |
| 106593,4 | Move BindingReflections.isNotificationCallback() | master | mdsal | Status: MERGED | +2 | +1 |
| 106618,1 | Hide BindingReflections.isRpcType() | master | mdsal | Status: MERGED | +2 | +1 |
| 106703,2 | Remove BindingReflections.getQName(BaseIdentity) | master | mdsal | Status: MERGED | +2 | +1 |
| 106704,3 | Reduce use of BindingReflections in BindingNormalizedNodeSerializer | master | mdsal | Status: MERGED | +2 | +1 |
| 106706,2 | Do not use BindingReflections in mdsal-binding-dom-codec-api | master | mdsal | Status: MERGED | +2 | +1 |
| 109362,5 | Do not fish for RpcInput/RpcOutput runtime types | master | mdsal | Status: MERGED | +2 | +1 |
| 109363,6 | Use bindingChild() to locate notifications | master | mdsal | Status: MERGED | +2 | +1 |
| 109382,1 | Do not use BindingReflections.findQName() | master | mdsal | Status: MERGED | +2 | +1 |