git: 9front

Download patch

ref: 4e48ba4cceb3584dfbbb875c62e9bde08f533bf9
parent: 77652a0c5b70decb0b72b63cad097c96294305c4
author: cinap_lenrek <cinap_lenrek@felloff.net>
date: Sun Apr 7 11:22:59 EDT 2024

pc, pc64: simplify error handling setscreensize()

The if(waserror()) nexterror(); is quite useless,
clean it up. do the bootscreenconf() at the very
end.

--- a/sys/src/9/pc/screen.c
+++ b/sys/src/9/pc/screen.c
@@ -130,9 +130,6 @@
 {
 	char *err;
 
-	if(waserror())
-		nexterror();
-
 	if(memimageinit() < 0)
 		error("memimageinit failed");
 
@@ -141,17 +138,13 @@
 		unlock(&vgascreenlock);
 		nexterror();
 	}
-
 	err = setscreensize0(scr, x, y, z, chan, tilt);
 	if(err != nil)
 		error(err);
-
 	vgaimageinit(chan);
 	unlock(&vgascreenlock);
 	poperror();
 
-	bootscreenconf(scr);
-
 	drawcmap();
 
 	if(scr->cur && scr->cur != &vgasoftcur){
@@ -160,7 +153,7 @@
 		cursoron();
 	}
 
-	poperror();
+	bootscreenconf(scr);
 }
 
 int
--