rik/jlibredis
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

getIntegerReply should return Integer and not int to support null replies

Browse Source
This commit is contained in:
Jonathan Leibiusky
2010-11-02 12:33:28 -03:00
parent d70c42edb9
commit 07f5820a32
2 changed files with 209 additions and 206 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/main/java/redis/clients/jedis/Connection.java
View File

@@ -126,9 +126,9 @@ public class Connection {
return (String) protocol.read(inputStream); return (String) protocol.read(inputStream);
} }
public int getIntegerReply() { public Integer getIntegerReply() {
pipelinedCommands--; pipelinedCommands--;
return ((Integer) protocol.read(inputStream)).intValue(); return (Integer) protocol.read(inputStream);
} }
@SuppressWarnings("unchecked") @SuppressWarnings("unchecked")

9
src/test/java/redis/clients/jedis/tests/commands/SortedSetCommandsTest.java
View File

@@ -100,11 +100,14 @@ public class SortedSetCommandsTest extends JedisCommandTestBase {
jedis.zadd("foo", 1d, "a"); jedis.zadd("foo", 1d, "a");
jedis.zadd("foo", 2d, "b"); jedis.zadd("foo", 2d, "b");
int rank = jedis.zrank("foo", "a"); Integer rank = jedis.zrank("foo", "a");
assertEquals(0, rank); assertEquals(0, rank.intValue());
rank = jedis.zrank("foo", "b"); rank = jedis.zrank("foo", "b");
assertEquals(1, rank); assertEquals(1, rank.intValue());
rank = jedis.zrank("car", "b");
assertNull(rank);
} }
@Test @Test