Code Review / openflowplugin.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 1650b3e) | patch
Fix bug in PortConvertor#maskPortConfigFields 05/5705/2
authorMoiz Raja <moraja@cisco.com>
Fri, 21 Mar 2014 00:22:53 +0000 (17:22 -0700)
committerMichal Rehak <mirehak@cisco.com>
Fri, 21 Mar 2014 12:39:50 +0000 (13:39 +0100)
commit84d8f8c03850c998ba1ae37b72050bf2c59f186f
tree57fa1a1930ab61ce9b4fba088a47cab65998d460tree | snapshot
parent1650b3e09320d3dad33fbe27e405fc59fe59b288commit | diff
Fix bug in PortConvertor#maskPortConfigFields

This method was trying to set a passed reference to a different value and expecting to see it set in the caller
This won't work in java because java passes object references also as a value

See http://www.javaworld.com/article/2077424/learn-java/does-java-pass-by-reference-or-pass-by-value.html

Change-Id: I809f0887fb435b5ae2dd2b91304b8c20fa2adb17
Signed-off-by: Moiz Raja <moraja@cisco.com>
Signed-off-by: Michal Rehak <mirehak@cisco.com>
openflowplugin/src/main/java/org/opendaylight/openflowplugin/openflow/md/core/sal/convertor/PortConvertor.java diff | blob | history
OpenFlow Plugin