#20 Possible typo in org.simpleframework.util.lease.Entry

closed
nobody
Bug (26)
5
2009-03-08
2009-02-05
Anonymous
No

may be compareTo's last return should return 0;

private int compareTo(Entry entry) {
long diff = time - entry.time;

if(diff < 0) {
return -1;
} else if(diff > 0) {
return 1;
}
return 1; // should return 0??
}

Discussion

  • Nobody/Anonymous

    Thanks

     
  • Nobody/Anonymous

    Thanks

     
  • Niall Gallagher

    Niall Gallagher - 2009-03-08
    • status: open --> closed
     

Log in to post a comment.