[ https://issues.apache.org/jira/browse/GEODE-2684?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15961347#comment-15961347 ]
ASF subversion and git services commented on GEODE-2684: -------------------------------------------------------- Commit 742c8f27ea45ec55445f1047fda3c54b86f89f94 in geode's branch refs/heads/feature/GEODE-2632 from [~bschuchardt] [ https://git-wip-us.apache.org/repos/asf?p=geode.git;h=742c8f2 ] GEODE-2684 Connection & ConnectionTable cleanup removed another old comment > org.apache.geode.internal.tcp.Connection & ConnectionTable cleanup > ------------------------------------------------------------------ > > Key: GEODE-2684 > URL: https://issues.apache.org/jira/browse/GEODE-2684 > Project: Geode > Issue Type: Task > Components: membership > Reporter: Bruce Schuchardt > Fix For: 1.2.0 > > > These classes contain a lot of commented-out code, poorly named methods and > incorrect calculations that need to be cleaned up. For instance, Connection > contains this: > short aShort = -1; > bytes[x] = (byte)(aShort & 0xff / 0x100); > bytes[x+1] = (byte)(aShort & 0xff); > which is incorrect and results in bytes[x] being zero when the intent is for > it to be 255. -- This message was sent by Atlassian JIRA (v6.3.15#6346)