It's silly to pass private core data around the front end, then fish out the frond then data from there, so don't.

This commit is contained in:
Michael Drake 2014-10-18 17:56:45 +01:00
parent b727c31de7
commit 2f67de6889
2 changed files with 5 additions and 9 deletions

View File

@ -143,7 +143,7 @@ void ro_gui_window_quit(void);
/* void ro_gui_window_close_all(void); */ /* void ro_gui_window_close_all(void); */
#define ro_gui_window_close_all ro_gui_window_quit /* no need for a separate fn */ #define ro_gui_window_close_all ro_gui_window_quit /* no need for a separate fn */
void ro_gui_throb(void); void ro_gui_throb(void);
void ro_gui_window_default_options(struct browser_window *bw); void ro_gui_window_default_options(struct gui_window *gui);
struct gui_window *ro_gui_window_lookup(wimp_w window); struct gui_window *ro_gui_window_lookup(wimp_w window);
struct gui_window *ro_gui_toolbar_lookup(wimp_w window); struct gui_window *ro_gui_toolbar_lookup(wimp_w window);
bool ro_gui_window_to_window_pos(struct gui_window *g, int x, int y, bool ro_gui_window_to_window_pos(struct gui_window *g, int x, int y,

View File

@ -2957,7 +2957,7 @@ bool ro_gui_window_menu_select(wimp_w w, wimp_i i, wimp_menu *menu,
break; break;
case BROWSER_SAVE_VIEW: case BROWSER_SAVE_VIEW:
if (bw != NULL) { if (bw != NULL) {
ro_gui_window_default_options(bw); ro_gui_window_default_options(g);
ro_gui_save_options(); ro_gui_save_options();
} }
break; break;
@ -4534,16 +4534,12 @@ void ro_gui_window_clone_options(
* \param bw the browser window to read options from * \param bw the browser window to read options from
*/ */
void ro_gui_window_default_options(struct browser_window *bw) void ro_gui_window_default_options(struct gui_window *gui)
{ {
struct gui_window *gui; struct gui_window *gui;
assert(bw); if (gui == NULL)
return;
/* Get our GUI
*/
gui = bw->window;
if (!gui) return;
/* Save the basic options /* Save the basic options
*/ */