npapi-vlc | branch: master | Daniel Amm <[email protected]> | Thu May 21 
20:24:18 2015 +0200| [ccf162c9c0284187dcf607bf3e56224112ff1082] | committer: 
Jean-Baptiste Kempf

npapi: remove redundant checks

Signed-off-by: Jean-Baptiste Kempf <[email protected]>

> http://git.videolan.org/gitweb.cgi/npapi-vlc.git/?a=commit;h=ccf162c9c0284187dcf607bf3e56224112ff1082
---

 npapi/npruntime/npolibvlc.cpp |    8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/npapi/npruntime/npolibvlc.cpp b/npapi/npruntime/npolibvlc.cpp
index 598995a..5329e8c 100644
--- a/npapi/npruntime/npolibvlc.cpp
+++ b/npapi/npruntime/npolibvlc.cpp
@@ -308,8 +308,8 @@ LibvlcAudioNPObject::setProperty(int index, const NPVariant 
&value)
                     auto tracks = mp.audioTrackDescription();
                     if ( v >= tracks.size() )
                         return INVOKERESULT_INVALID_VALUE;
-                    if ( mp.setAudioTrack( tracks[v].id() ) )
-                        return INVOKERESULT_NO_ERROR;
+                    mp.setAudioTrack( tracks[v].id() );
+                    return INVOKERESULT_NO_ERROR;
                 }
                 return INVOKERESULT_INVALID_VALUE;
             case ID_audio_channel:
@@ -1209,8 +1209,8 @@ LibvlcSubtitleNPObject::setProperty(int index, const 
NPVariant &value)
                     auto tracks = mp.spuDescription();
                     if ( v >= tracks.size() )
                         return INVOKERESULT_INVALID_ARGS;
-                    if ( mp.setSpu( tracks[ v ].id() ) )
-                        return INVOKERESULT_NO_ERROR;
+                    mp.setSpu( tracks[ v ].id() );
+                    return INVOKERESULT_NO_ERROR;
                 }
                 return INVOKERESULT_INVALID_VALUE;
             }

_______________________________________________
vlc-commits mailing list
[email protected]
https://mailman.videolan.org/listinfo/vlc-commits

Reply via email to