Amebis/wxWidgets
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Use temporary variable in gtk_dataview_button_press_callback()

Browse Source 
No real changes, just simplify the code a bit by using a local variable
instead of repeating the same expression a couple of times.
This commit is contained in:
Vadim Zeitlin
2021-05-09 00:15:12 +02:00
parent 0585c96a3b
commit 1ede92afa1
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/gtk/dataview.cpp
View File

@@ -4662,13 +4662,15 @@ gtk_dataview_button_press_callback( GtkWidget *WXUNUSED(widget),
{ {
if ((gdk_event->button == 3) && (gdk_event->type == GDK_BUTTON_PRESS)) if ((gdk_event->button == 3) && (gdk_event->type == GDK_BUTTON_PRESS))
{ {
GtkTreeView* const treeview = GTK_TREE_VIEW(dv->GtkGetTreeView());
wxGtkTreePath path; wxGtkTreePath path;
GtkTreeViewColumn *column = NULL; GtkTreeViewColumn *column = NULL;
gint cell_x = 0; gint cell_x = 0;
gint cell_y = 0; gint cell_y = 0;
gtk_tree_view_get_path_at_pos gtk_tree_view_get_path_at_pos
( (
GTK_TREE_VIEW(dv->GtkGetTreeView()), treeview,
(int) gdk_event->x, (int) gdk_event->y, (int) gdk_event->x, (int) gdk_event->y,
path.ByRef(), path.ByRef(),
&column, &column,
@@ -4681,7 +4683,7 @@ gtk_dataview_button_press_callback( GtkWidget *WXUNUSED(widget),
// because it could be a part of multi-item selection. // because it could be a part of multi-item selection.
if ( path ) if ( path )
{ {
GtkTreeSelection *selection = gtk_tree_view_get_selection(GTK_TREE_VIEW(dv->GtkGetTreeView())); GtkTreeSelection *selection = gtk_tree_view_get_selection(treeview);
if ( !gtk_tree_selection_path_is_selected(selection, path) ) if ( !gtk_tree_selection_path_is_selected(selection, path) )
{ {
gtk_tree_selection_unselect_all(selection); gtk_tree_selection_unselect_all(selection);