Saturday, 14 March 2015

Bug 2182 (revised patch--please ignore previous email)

>From 43889fdbc04b88b208ac676186d0773d3ffa4715 Mon Sep 17 00:00:00 2001
From: David Gee <dr_d_gee@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

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

No comments:

Post a Comment