shithub: riscv

Download patch

ref: c33732a5305270321beccbd3e1b44881cd2e0004
parent: 624c9f51125ded8cc3e2aa715558cf9c4a6f7108
author: cinap_lenrek <cinap_lenrek@gmx.de>
date: Sat Oct 20 06:38:11 EDT 2012

rio: wtop() after resize fix

wtop() checked w->topped == topped as a fast exit. if you
had the top most window not being current (== input) which
can happen when you delete the current window, then wtop would
after resize would not make the window current because it
is already top topmost one. wtopme() and wcurrent() both are
non-ops when window is already the topmost or window is already
current, so remove the check as its not needed. also topping
the window feels less sluggish.

--- a/sys/src/cmd/rio/rio.c

+++ b/sys/src/cmd/rio/rio.c

@@ -1220,8 +1220,8 @@

 	for(j=0; j<nwindow; j++)

 		if(window[j] == w){

 			incref(w);

-			wcurrent(w);

 			wtopme(w);

+			wcurrent(w);

 			wclose(w);

 			return;

 		}

--- a/sys/src/cmd/rio/wind.c

+++ b/sys/src/cmd/rio/wind.c

@@ -1311,11 +1311,9 @@

 	w = wpointto(pt);

 	if(w){

-		if(w->topped == topped)

-			return nil;

 		incref(w);

-		wcurrent(w);

 		wtopme(w);

+		wcurrent(w);

 		wclose(w);

 	}

 	return w;

-- 

⑨