List:Commits« Previous MessageNext Message »
From:Øystein Grøvlen Date:November 25 2010 8:27am
Subject:Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393
View as plain text  
On 11/25/10 09:20 AM, Øystein Grøvlen wrote:
> I also think this code would be more evident if it was explained in a
> bit more detail how the extra element in bounded_queue is used for
> replacing. It is not quite clear from comments that at any point in time
> when the underlying queue is full, it will contain the top-N elements so
> far plus the last element that was checked.

Correction: If the last element checked, is among the top-N elements so 
far, the additional element will be the top-N+1 element.

-- 
Øystein
Thread
bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Tor Didriksen22 Nov
  • Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Jorgen Loland24 Nov
    • Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Tor Didriksen24 Nov
      • Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Øystein Grøvlen25 Nov
        • Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Øystein Grøvlen25 Nov
Re: bzr commit into mysql-trunk branch (tor.didriksen:3247) WL#1393Øystein Grøvlen28 Nov