From: Rémi Denis-Courmont <rdenis@simphalempin.com>
Date: Tue, 16 Dec 2008 21:00:08 +0000 (+0200)
Subject: b_menu_change: remove write-only variable
X-Git-Tag: 1.0.0-pre1~1747
X-Git-Url: https://git.sesse.net/?a=commitdiff_plain;h=78e7bd3059dc2647a9786b8c531e79c5821df756;p=vlc

b_menu_change: remove write-only variable
---

diff --git a/include/vlc_interface.h b/include/vlc_interface.h
index f390972edc..572148f8de 100644
--- a/include/vlc_interface.h
+++ b/include/vlc_interface.h
@@ -72,7 +72,6 @@ struct intf_thread_t
 
     /* XXX: new message passing stuff will go here */
     vlc_mutex_t  change_lock;
-    bool   b_menu_change;
     bool   b_menu;
 };
 
diff --git a/modules/video_output/aa.c b/modules/video_output/aa.c
index 9123a5153f..d1735141b6 100644
--- a/modules/video_output/aa.c
+++ b/modules/video_output/aa.c
@@ -200,16 +200,8 @@ static int Manage( vout_thread_t *p_vout )
     {
     case AA_MOUSE:
         aa_getmouse( p_vout->p_sys->aa_context, &x, &y, &b );
-        if ( b & AA_BUTTON3 )
-        {
-            intf_thread_t *p_intf;
-            p_intf = vlc_object_find( p_vout, VLC_OBJECT_INTF, FIND_ANYWHERE );
-            if( p_intf )
-            {
-                p_intf->b_menu_change = 1;
-                vlc_object_release( p_intf );
-            }
-        }
+        /*if ( b & AA_BUTTON3 )
+             trigger contextual menu here */
         break;
     case AA_RESIZE:
         p_vout->i_changes |= VOUT_SIZE_CHANGE;
diff --git a/modules/video_output/ggi.c b/modules/video_output/ggi.c
index 7ee119f54e..da70775b9e 100644
--- a/modules/video_output/ggi.c
+++ b/modules/video_output/ggi.c
@@ -311,16 +311,7 @@ static int Manage( vout_thread_t *p_vout )
                         break;
 
                     case GII_PBUTTON_RIGHT:
-                        {
-                            intf_thread_t *p_intf;
-                            p_intf = vlc_object_find( p_vout, VLC_OBJECT_INTF,
-                                                              FIND_ANYWHERE );
-                            if( p_intf )
-                            {
-                                p_intf->b_menu_change = 1;
-                                vlc_object_release( p_intf );
-                            }
-                        }
+                        /* trigger contextual menu here */
                         break;
                 }
                 break;
diff --git a/modules/video_output/sdl.c b/modules/video_output/sdl.c
index 17efea735d..08fb157c86 100644
--- a/modules/video_output/sdl.c
+++ b/modules/video_output/sdl.c
@@ -459,18 +459,9 @@ static int Manage( vout_thread_t *p_vout )
 
             case SDL_BUTTON_RIGHT:
                 {
-                    intf_thread_t *p_intf;
-
                     var_Get( p_vout, "mouse-button-down", &val );
                     val.i_int &= ~4;
                     var_Set( p_vout, "mouse-button-down", val );
-                    p_intf = vlc_object_find( p_vout, VLC_OBJECT_INTF,
-                                                      FIND_ANYWHERE );
-                    if( p_intf )
-                    {
-                        p_intf->b_menu_change = 1;
-                        vlc_object_release( p_intf );
-                    }
 
                     val.b_bool = true;
                     var_Set( p_vout->p_libvlc, "intf-popupmenu", val );
diff --git a/modules/video_output/x11/xcommon.c b/modules/video_output/x11/xcommon.c
index 83a3b8858c..c64ad7470d 100644
--- a/modules/video_output/x11/xcommon.c
+++ b/modules/video_output/x11/xcommon.c
@@ -1327,13 +1327,6 @@ static int ManageVideo( vout_thread_t *p_vout )
                         var_Get( p_vout, "mouse-button-down", &val );
                         val.i_int &= ~4;
                         var_Set( p_vout, "mouse-button-down", val );
-                        p_intf = vlc_object_find( p_vout, VLC_OBJECT_INTF,
-                                                          FIND_ANYWHERE );
-                        if( p_intf )
-                        {
-                            p_intf->b_menu_change = 1;
-                            vlc_object_release( p_intf );
-                        }
 
                         vlc_value_t val; val.b_bool = true;
                         var_Set( p_vout->p_libvlc, "intf-popupmenu", val );
diff --git a/src/interface/interface.c b/src/interface/interface.c
index 6c8c9f9a3d..2bd8115f38 100644
--- a/src/interface/interface.c
+++ b/src/interface/interface.c
@@ -108,7 +108,6 @@ intf_thread_t* __intf_Create( vlc_object_t *p_this, const char *psz_module )
 
     /* Initialize structure */
     p_intf->b_menu        = false;
-    p_intf->b_menu_change = false;
 
     /* Initialize mutexes */
     vlc_mutex_init( &p_intf->change_lock );