[CONTROLLER-517] Moved Nested Classes into dedicated class files Created: 28/May/14 Updated: 19/Oct/17 Resolved: 08/Dec/15 |
|
| Status: | Resolved |
| Project: | controller |
| Component/s: | mdsal |
| Affects Version/s: | Helium |
| Fix Version/s: | None |
| Type: | Improvement | ||
| Reporter: | Devin Avery | Assignee: | Unassigned |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: Mac OS |
||
| Issue Links: |
|
||||||||
| Description |
|
In multiple locations in MD-SAL there are large nested classes, which leads to difficult reading of the source code. This bugs should track the effort to moved nested classes out to classes dedicated for their simple purpose. Nested classes should only be used to perform, simple, insignificant tasks (generally they are anonymous). Each class should have a simple purpose. For example, one EXAMPLE of a class that should have its nested classes removed is: org.opendaylight.controller.sal.binding.impl.connect.dom.BindingIndependentConnector |
| Comments |
| Comment by Tony Tkacik [ 15/Jul/14 ] |
|
BindingIndependentConnector fixed in |
| Comment by Tony Tkacik [ 13/Nov/14 ] |
|
Added as blocking issue for Topic: Decrease Technical Debt. |
| Comment by Tony Tkacik [ 08/Dec/15 ] |
|
Was fixed in Lithium when offending classes were obsoloted and removed. |