Re: [Xen-devel] [PATCH] Fix pygrub with "many" kernels

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 4/10/06 22:57, "Jeremy Katz" <katzj@xxxxxxxxxx> wrote:

> Attached fixes running pygrub with many kernels to do proper scrolling
> 
> Signed-off-by: Jeremy Katz <katzj@xxxxxxxxxx>

+        if self.selected_image > self.start_image + maxy:
+            self.start_image = self.selected_image
+        if self.selected_image < self.start_image:
+            self.start_image = self.selected_image

Shouldn't the second line above be:
     self.start_image = self.selected_image - maxy

So that the selected image is the one at the bottom of the displayed
subsection of the list?

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel