> On Feb 24, 2018, at 12:38 AM, Daniel Fuchs <daniel.fu...@oracle.com> wrote: > > Hi Max, > > 85 // has larger timestamp. > 86 entries.addFirst(t); > > shouldn't you set oldestTime here as well?
The order might be a little misleading, you addFirst() the latest entry and the oldest one is at getLast(). oldestTime is set on line 78 because it's the only entry. But for line 86, there are already existing entries. --Max > > best regards, > > -- daniel > > On 23/02/2018 15:21, Xuelei Fan wrote: >> Looks fine to me. >> Xuelei >> On 2/23/2018 6:13 AM, Weijun Wang wrote: >>> Updated at http://cr.openjdk.java.net/~weijun/8197518/webrev.02/.