[vlc-devel] [PATCH 2/2] qt: mlnetworkmediamodel: fix warnings

Pierre Lamot pierre at videolabs.io
Mon Nov 18 09:51:05 CET 2019


LGTM

On 2019-11-17 19:16, Alexandre Janniaux wrote:
> ---
>  .../gui/qt/components/mediacenter/mlnetworkmediamodel.cpp | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git
> a/modules/gui/qt/components/mediacenter/mlnetworkmediamodel.cpp
> b/modules/gui/qt/components/mediacenter/mlnetworkmediamodel.cpp
> index e8f8609187..1454b06540 100644
> --- a/modules/gui/qt/components/mediacenter/mlnetworkmediamodel.cpp
> +++ b/modules/gui/qt/components/mediacenter/mlnetworkmediamodel.cpp
> @@ -170,7 +170,7 @@ bool MLNetworkMediaModel::addToPlaylist(int index)
>  {
>      if (!(m_ctx && m_hasTree))
>          return false;
> -    if (index < 0 || index >= m_items.size() )
> +    if (index < 0 || (size_t)index >= m_items.size() )
>          return false;
>      auto item =  m_items[index];
>      vlc::playlist::Media media{ item.tree.media.get() };
> @@ -196,7 +196,7 @@ bool MLNetworkMediaModel::addAndPlay(int index)
>  {
>      if (!(m_ctx && m_hasTree))
>          return false;
> -    if (index < 0 || index >= m_items.size() )
> +    if (index < 0 || (size_t)index >= m_items.size() )
>          return false;
>      auto item =  m_items[index];
>      vlc::playlist::Media media{ item.tree.media.get() };
> @@ -267,7 +267,7 @@ bool MLNetworkMediaModel::initializeMediaSources()
>      return true;
>  }
> 
> -void MLNetworkMediaModel::onItemCleared( MediaSourcePtr mediaSource,
> input_item_node_t* node )
> +void MLNetworkMediaModel::onItemCleared( MediaSourcePtr mediaSource,
> input_item_node_t*)
>  {
>      input_item_node_t *res;
>      input_item_node_t *parent;
> @@ -322,7 +322,7 @@ void MLNetworkMediaModel::onItemRemoved( 
> MediaSourcePtr,
>      }
>  }
> 
> -void MLNetworkMediaModel::onItemPreparseEnded(MediaSourcePtr
> mediaSource, input_item_node_t* node, enum input_item_preparse_status
> status)
> +void MLNetworkMediaModel::onItemPreparseEnded(MediaSourcePtr,
> input_item_node_t*, enum input_item_preparse_status)
>  {
>      QMetaObject::invokeMethod(this, [this]() {
>          m_parsingPending = false;


More information about the vlc-devel mailing list