andreacavalli/netty5
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #379 from jroper/patch-1

Browse Source 
Fixing #378, bug in DefaultCookie equals and compareTo
This commit is contained in:
Trustin Lee 2012-06-02 19:49:18 -07:00
commit 26307cbc3e
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
codec-http/src/main/java/io/netty/handler/codec/http/DefaultCookie.java
View File

@ -273,7 +273,7 @@ public class DefaultCookie implements Cookie {
if (getPath() == null && that.getPath() != null) {
return false;
} else if (that.getPath() == null) {
} else if (that.getPath() == null && getPath() != null) {
return false;
}
if (!getPath().equals(that.getPath())) {
@ -282,7 +282,7 @@ public class DefaultCookie implements Cookie {
if (getDomain() == null && that.getDomain() != null) {
return false;
} else if (that.getDomain() == null) {
} else if (that.getDomain() == null && getDomain() != null) {
return false;
}
return getDomain().equalsIgnoreCase(that.getDomain());
@ -298,7 +298,7 @@ public class DefaultCookie implements Cookie {
if (getPath() == null && c.getPath() != null) {
return -1;
} else if (c.getPath() == null) {
} else if (c.getPath() == null && getPath() != null) {
return 1;
}
v = getPath().compareTo(c.getPath());
@ -308,7 +308,7 @@ public class DefaultCookie implements Cookie {
if (getDomain() == null && c.getDomain() != null) {
return -1;
} else if (c.getDomain() == null) {
} else if (c.getDomain() == null && getDomain() != null) {
return 1;
}
v = getDomain().compareToIgnoreCase(c.getDomain());