[vlc-devel] [PATCH 2/5] qt: variables: don't create variables we already read from

Thomas Guillem thomas at gllm.fr
Fri Mar 13 15:28:35 CET 2020


LGTM for the set

On Fri, Mar 13, 2020, at 15:00, Steve Lhomme wrote:
> A few lines above we already use the variable:
> var_Get(m_object->get(), qtu(m_property), &currentvalue)
> ---
>  modules/gui/qt/util/variables.hpp | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/modules/gui/qt/util/variables.hpp 
> b/modules/gui/qt/util/variables.hpp
> index dff7e84bcb2..08f1de8ef07 100644
> --- a/modules/gui/qt/util/variables.hpp
> +++ b/modules/gui/qt/util/variables.hpp
> @@ -154,7 +154,6 @@ public:
>                  emit derived->valueChanged( m_value );
>              }
>  
> -            var_Create(m_object->get(), qtu(m_property), 
> VLCVarTypeTraits<BaseType>::var_type);
>              var_AddCallback(m_object->get(), qtu(m_property), 
> value_modified, &cref);
>          }
>      }
> @@ -164,7 +163,6 @@ public:
>          if (m_object->get())
>          {
>              var_DelCallback( m_object->get(), qtu(m_property), 
> value_modified, &cref );
> -            var_Destroy(m_object->get(), qtu(m_property));
>              m_object->clear();
>          }
>      }
> -- 
> 2.17.1
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list