[NETCONF-44] missing good practice Created: 29/Jun/15 Updated: 15/Mar/19 Resolved: 01/Oct/18 |
|
| Status: | Resolved |
| Project: | netconf |
| Component/s: | netconf |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Bug | ||
| Reporter: | Shashidhar Shekar | Assignee: | Jakub Morvay |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Operating System: All |
||
| External issue ID: | 3890 |
| Description |
|
missing default case in class -> public final class EXIParameters { function --> public static EXIParameters fromXmlElement |
| Comments |
| Comment by Robert Varga [ 13/Nov/15 ] |
|
Move to NETCONFI project. |
| Comment by Shivani Pathak [ 19/Nov/15 ] |
|
Switch is written for possible values of 'alignmentTextContent'. A defualt case can be added with a break statement to improve the code. |
| Comment by Shivani Pathak [ 25/Nov/15 ] |
|
Changes submitted: https://git.opendaylight.org/gerrit/30174 |
| Comment by Jakub Morvay [ 01/Oct/18 ] |
|
Actually done in https://git.opendaylight.org/gerrit/#/c/30187/. The above mentioned commit is not related to this issue. |