git: 9front

Info  •  Files  •  Log  •  Branches

samterm: dont burn 100% cpu when button 2 scrolling

Fix buggy behaviour introduced in commit:

479ecb171f1221861208ad62bc8b3ee412644489 samterm: Make scrollbar act like rio's

Do blocking readmouse() to get next mouse
event instead of the nbrecv() when using
button 2 scrolling.

--- a/sys/src/cmd/samterm/scroll.c

+++ b/sys/src/cmd/samterm/scroll.c

@@ -104,6 +104,8 @@

 			o = (tot / (s.max.y - s.min.y)) * my;

 			n = 0;

 			forcenter(l, o, n);

+			if(readmouse(mousectl) < 0)

+				panic("mouse");

 		}else{

 			o = l->origin;

 			n = my/l->f.font->height;

@@ -120,8 +122,8 @@

 				sleep(175);

 			}

 			sleep(25);

+			if(nbrecv(mousectl->c, mousectl) < 0)

+				panic("mouse");

 		}

-		if(nbrecv(mousectl->c, mousectl) < 0)

-			panic("mouse");

 	}while(mousectl->buttons & (1 << (but-1)));

 }

-- 

⑨