Hi,

The patch attached to my previous email contained an error. Somehow I'd
inadvertently removed the 'x' from an xwimp call which I didn't intend
to change.

I've used git to revert the changes and then (once again) removed the
xwimp_create_menu(WIMP_CLOSE_MENU,0,0) call from
ro_gui_send_datasave--the change I originally intended to make. So the
patch attached here should be sufficient on its own.

Thanks to Jeremy Nicoll for pointing this out.


-- 
David Gee
Gateshead

>From 43889fdbc04b88b208ac676186d0773d3ffa4715 Mon Sep 17 00:00:00 2001
From: David Gee <dr_d_...@blueyonder.co.uk>
Date: Sat, 14 Mar 2015 20:28:49 +0000
Subject: [PATCH] Change to save.c so menu tree is not needlessly collapsed on
 send_datasave.

---
 riscos/save.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/riscos/save.c b/riscos/save.c
index b5d96c4..c4fd839 100644
--- a/riscos/save.c
+++ b/riscos/save.c
@@ -723,7 +723,7 @@ void ro_gui_send_datasave(gui_save_type save_type,
 {
 	/* Close the save window because otherwise we need two contexts
 	*/
-	xwimp_create_menu(wimp_CLOSE_MENU, 0, 0);
+
 	ro_gui_dialog_close(dialog_saveas);
 
 	if (ro_message_send_message(wimp_USER_MESSAGE_RECORDED, (wimp_message*)message,
-- 
1.9.1

Reply via email to