From 23bc8818b3282745a640853ce93a66df4a2da4ba Mon Sep 17 00:00:00 2001 From: dec05eba Date: Wed, 17 May 2023 12:56:55 +0200 Subject: Fix all warnings --- project.conf | 3 +++ src/library_loader.h | 7 ------ src/main.cpp | 62 +++++++++++++++++++++++++--------------------------- 3 files changed, 33 insertions(+), 39 deletions(-) diff --git a/project.conf b/project.conf index dbf928d..bc9b3b6 100644 --- a/project.conf +++ b/project.conf @@ -4,6 +4,9 @@ type = "executable" version = "1.3.2" platforms = ["posix"] +[config] +error_on_warning = "true" + [dependencies] gtk+-3.0 = "3" x11 = "1" diff --git a/src/library_loader.h b/src/library_loader.h index 1622521..fbd9cdf 100644 --- a/src/library_loader.h +++ b/src/library_loader.h @@ -32,11 +32,4 @@ static bool dlsym_load_list(void *handle, const dlsym_assign *dlsyms) { return success; } -/* |dlsyms| should be null terminated */ -static void dlsym_load_list_optional(void *handle, const dlsym_assign *dlsyms) { - for(int i = 0; dlsyms[i].func; ++i) { - *dlsyms[i].func = dlsym_print_fail(handle, dlsyms[i].name, false); - } -} - #endif /* GSR_LIBRARY_LOADER_H */ diff --git a/src/main.cpp b/src/main.cpp index 1b863ed..119c6cf 100644 --- a/src/main.cpp +++ b/src/main.cpp @@ -191,11 +191,7 @@ static void for_each_used_audio_input(GtkListBox *list_box, std::functionid = id; g_object_set_data(G_OBJECT(row), "audio-row", audio_row); - g_signal_connect(remove_button, "clicked", G_CALLBACK(+[](GtkButton *button, gpointer userdata){ + g_signal_connect(remove_button, "clicked", G_CALLBACK(+[](GtkButton*, gpointer userdata){ AudioRow *audio_row = (AudioRow*)userdata; gtk_combo_box_text_append(audio_input_menu_todo, audio_row->id.c_str(), gtk_label_get_text(GTK_LABEL(audio_row->label))); gtk_container_remove (GTK_CONTAINER(gtk_widget_get_parent(audio_row->row)), audio_row->row); @@ -535,7 +534,7 @@ static Window window_get_target_window_child(Display *display, Window window) { } /* TODO: Look at xwininfo source to figure out how to make this work for different types of window managers */ -static GdkFilterReturn filter_callback(GdkXEvent *xevent, GdkEvent *event, gpointer userdata) { +static GdkFilterReturn filter_callback(GdkXEvent *xevent, GdkEvent*, gpointer userdata) { SelectWindowUserdata *select_window_userdata = (SelectWindowUserdata*)userdata; XEvent *ev = (XEvent*)xevent; //assert(ev->type == ButtonPress); @@ -593,8 +592,7 @@ static GdkFilterReturn filter_callback(GdkXEvent *xevent, GdkEvent *event, gpoin return GDK_FILTER_REMOVE; } -static gboolean on_select_window_button_click(GtkButton *button, gpointer userdata) { - GtkApplication *app = (GtkApplication*)userdata; +static gboolean on_select_window_button_click(GtkButton *button, gpointer) { Display *display = gdk_x11_get_default_xdisplay(); select_window_userdata.display = display; select_window_userdata.select_window_button = button; @@ -721,12 +719,12 @@ static int key_get_name(KeySym key_sym, char *buffer, int buffer_size) { return 0; } -static int xerror_dummy(Display *dpy, XErrorEvent *ee) { +static int xerror_dummy(Display*, XErrorEvent*) { return 0; } static bool x_failed = false; -static int xerror_grab_error(Display *dpy, XErrorEvent *ee) { +static int xerror_grab_error(Display*, XErrorEvent*) { x_failed = true; return 0; } @@ -894,7 +892,7 @@ static bool show_pkexec_flatpak_error_if_needed() { return false; } -static gboolean on_start_replay_click(GtkButton *button, gpointer userdata) { +static gboolean on_start_replay_click(GtkButton*, gpointer userdata) { if(show_pkexec_flatpak_error_if_needed()) return true; @@ -914,7 +912,7 @@ static gboolean on_start_replay_click(GtkButton *button, gpointer userdata) { return true; } -static gboolean on_start_recording_click(GtkButton *button, gpointer userdata) { +static gboolean on_start_recording_click(GtkButton*, gpointer userdata) { if(show_pkexec_flatpak_error_if_needed()) return true; @@ -929,14 +927,14 @@ static gboolean on_start_recording_click(GtkButton *button, gpointer userdata) { return true; } -void on_stream_key_icon_click(GtkWidget *widget, gpointer data) { +void on_stream_key_icon_click(GtkWidget *widget, gpointer) { gboolean visible = gtk_entry_get_visibility(GTK_ENTRY(widget)); gtk_entry_set_visibility(GTK_ENTRY(widget), !visible); gtk_entry_set_icon_from_icon_name(GTK_ENTRY(widget), GTK_ENTRY_ICON_SECONDARY, visible ? "view-reveal-symbolic.symbolic" : "view-conceal-symbolic.symbolic"); } -static gboolean on_start_streaming_click(GtkButton *button, gpointer userdata) { +static gboolean on_start_streaming_click(GtkButton*, gpointer userdata) { if(show_pkexec_flatpak_error_if_needed()) return true; @@ -964,7 +962,7 @@ static gboolean on_start_streaming_click(GtkButton *button, gpointer userdata) { return true; } -static gboolean on_streaming_recording_replay_page_back_click(GtkButton *button, gpointer userdata) { +static gboolean on_streaming_recording_replay_page_back_click(GtkButton*, gpointer userdata) { PageNavigationUserdata *page_navigation_userdata = (PageNavigationUserdata*)userdata; gtk_stack_set_visible_child(page_navigation_userdata->stack, page_navigation_userdata->common_settings_page); ungrab_keys(gdk_x11_get_default_xdisplay()); @@ -993,13 +991,13 @@ static gboolean file_choose_button_click_handler(GtkButton *button, const char * return true; } -static gboolean on_record_file_choose_button_click(GtkButton *button, gpointer userdata) { +static gboolean on_record_file_choose_button_click(GtkButton *button, gpointer) { gboolean res = file_choose_button_click_handler(button, "Where do you want to save the video?", GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER, gtk_combo_box_text_get_active_text(record_container)); config.record_config.save_directory = gtk_button_get_label(button); return res; } -static gboolean on_replay_file_chooser_button_click(GtkButton *button, gpointer userdata) { +static gboolean on_replay_file_chooser_button_click(GtkButton *button, gpointer) { gboolean res = file_choose_button_click_handler(button, "Where do you want to save the replays?", GTK_FILE_CHOOSER_ACTION_SELECT_FOLDER, gtk_combo_box_text_get_active_text(replay_container)); config.replay_config.save_directory = gtk_button_get_label(button); return res; @@ -1142,7 +1140,7 @@ static gboolean on_start_replay_button_click(GtkButton *button, gpointer userdat return true; } -static gboolean on_replay_save_button_click(GtkButton *button, gpointer userdata) { +static gboolean on_replay_save_button_click(GtkButton*, gpointer userdata) { GtkApplication *app = (GtkApplication*)userdata; kill(gpu_screen_recorder_process, SIGUSR1); show_notification(app, "GPU Screen Recorder", "Saved replay", G_NOTIFICATION_PRIORITY_NORMAL); @@ -1419,7 +1417,7 @@ struct AudioInput { std::string description; }; -static void pa_sourcelist_cb(pa_context *ctx, const pa_source_info *source_info, int eol, void *userdata) { +static void pa_sourcelist_cb(pa_context*, const pa_source_info *source_info, int eol, void *userdata) { if(eol > 0) return; @@ -1591,7 +1589,7 @@ static void keypress_toggle_recording(bool recording_state, GtkButton *record_bu } } -static GdkFilterReturn hotkey_filter_callback(GdkXEvent *xevent, GdkEvent *event, gpointer userdata) { +static GdkFilterReturn hotkey_filter_callback(GdkXEvent *xevent, GdkEvent*, gpointer userdata) { if(hotkey_mode == HotkeyMode::NoAction) return GDK_FILTER_CONTINUE; @@ -1898,7 +1896,7 @@ static GtkWidget* create_common_settings_page(GtkStack *stack, GtkApplication *a add_audio_input_button = gtk_button_new_with_label("Add"); gtk_widget_set_halign(add_audio_input_button, GTK_ALIGN_END); gtk_grid_attach(add_audio_grid, add_audio_input_button, 1, 0, 1, 1); - g_signal_connect(add_audio_input_button, "clicked", G_CALLBACK(+[](GtkButton *button, gpointer userdata){ + g_signal_connect(add_audio_input_button, "clicked", G_CALLBACK(+[](GtkButton*, gpointer){ const gint selected_audio_input = gtk_combo_box_get_active(GTK_COMBO_BOX(audio_input_menu_todo)); const char *active_id = gtk_combo_box_get_active_id(GTK_COMBO_BOX(audio_input_menu_todo)); const char *active_text = gtk_combo_box_text_get_active_text(audio_input_menu_todo); @@ -2270,7 +2268,7 @@ static GtkWidget* create_streaming_page(GtkApplication *app, GtkStack *stack) { return GTK_WIDGET(grid); } -static gboolean on_destroy_window(GtkWidget *widget, GdkEvent *event, gpointer data) { +static gboolean on_destroy_window(GtkWidget*, GdkEvent*, gpointer) { if(gpu_screen_recorder_process != -1) { kill(gpu_screen_recorder_process, SIGINT); int status; @@ -2504,7 +2502,7 @@ static const char* gpu_vendor_to_name(gpu_vendor vendor) { return ""; } -static void activate(GtkApplication *app, gpointer userdata) { +static void activate(GtkApplication *app, gpointer) { nvfbc_installed = is_nv_fbc_installed(); if(is_wayland() || is_xwayland()) { @@ -2598,7 +2596,7 @@ static void activate(GtkApplication *app, gpointer userdata) { int main(int argc, char **argv) { setlocale(LC_ALL, "C"); - GtkApplication *app = gtk_application_new("com.dec05eba.gpu_screen_recorder", G_APPLICATION_FLAGS_NONE); + GtkApplication *app = gtk_application_new("com.dec05eba.gpu_screen_recorder", G_APPLICATION_NON_UNIQUE); g_signal_connect(app, "activate", G_CALLBACK(activate), nullptr); int status = g_application_run(G_APPLICATION(app), argc, argv); g_object_unref(app); -- cgit v1.2.3