~ghugesss/xpad/refactor_code

« back to all changes in this revision

Viewing changes to src/xpad-toolbar.c

  • Committer: Arthur Borsboom
  • Date: 2014-06-20 00:28:20 UTC
  • mfrom: (665.1.67 xpad-4.3)
  • Revision ID: arthurborsboom@gmail.com-20140620002820-c4nuneh78qv4pu31
    Arthur Borsboom 2014-06-20 * Fix: reduced code by using existing functions, merging functions, removing obsolete code and duplicate texts
    Arthur Borsboom 2014-06-19 * Fix: Another try to prevent an error loop on application exit

Show diffs side-by-side

added added

removed removed

Lines of Context:
97
97
static GtkToolItem *xpad_toolbar_button_to_item (XpadToolbar *toolbar, const XpadToolbarButton *button);
98
98
static void xpad_toolbar_button_activated (GtkToolButton *button);
99
99
static void xpad_toolbar_change_buttons (XpadToolbar *toolbar);
100
 
static void xpad_toolbar_remove_all_buttons (XpadSettings *settings);
101
 
static void xpad_toolbar_remove_last_button (XpadSettings *settings);
102
100
static void xpad_toolbar_add_button (GtkMenuItem *menu_item, XpadSettings *settings);
103
101
static gboolean xpad_toolbar_popup_context_menu (GtkToolbar *toolbar, gint x, gint y, gint button);
104
102
 
398
396
{
399
397
        GList *list, *temp;
400
398
        const GSList *slist, *stemp;
401
 
        guint i = 0;
402
 
        guint j = 0;
 
399
        guint i = 0, j = 0;
403
400
        GtkToolItem *item;
404
401
        XpadSettings *settings;
405
402
        
462
459
}
463
460
 
464
461
static void
465
 
xpad_toolbar_remove_all_buttons (XpadSettings *settings)
466
 
{
467
 
        xpad_settings_remove_all_toolbar_buttons (settings);
468
 
}
469
 
 
470
 
static void
471
 
xpad_toolbar_remove_last_button (XpadSettings *settings)
472
 
{
473
 
        xpad_settings_remove_last_toolbar_button (settings);
474
 
}
475
 
 
476
 
static void
477
462
xpad_toolbar_add_button (GtkMenuItem *menu_item, XpadSettings *settings)
478
463
{
479
464
        xpad_settings_add_toolbar_button (settings, gtk_menu_item_get_accel_path (menu_item) + 2);
528
513
                GtkWidget *item;
529
514
 
530
515
                item = gtk_menu_item_new_with_mnemonic (N_("Remove All _Buttons"));
531
 
                g_signal_connect_swapped (item, "activate", G_CALLBACK (xpad_toolbar_remove_all_buttons), settings);
 
516
                g_signal_connect_swapped (item, "activate", G_CALLBACK (xpad_settings_remove_all_toolbar_buttons), settings);
532
517
                gtk_menu_attach (menu, item, 0, 1, i, i + 1);
533
518
                gtk_widget_show (item);
534
519
 
535
520
                i++;
536
521
                
537
522
                item = gtk_menu_item_new_with_mnemonic (N_("Remo_ve Last Button"));
538
 
                g_signal_connect_swapped (item, "activate", G_CALLBACK (xpad_toolbar_remove_last_button), settings);
 
523
                g_signal_connect_swapped (item, "activate", G_CALLBACK (xpad_settings_remove_last_toolbar_button), settings);
539
524
                gtk_menu_attach (menu, item, 0, 1, i, i + 1);
540
525
                gtk_widget_show (item);
541
526
        }