<!DOCTYPE html><html><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body><div>Indeed, it should be a noop.<br></div><div><br></div><div>On Wed, Sep 4, 2019, at 11:34, Rémi Denis-Courmont wrote:<br></div><blockquote type="cite" id="qt"><div>Hi,<br></div><div><br></div><div>Conventionally, assertions are not evaluated in release builds. It's confusing enough already. I don't think we should add exceptions to that principle.<br></div><div><br></div><div class="qt-gmail_quote"><div>Le 4 septembre 2019 10:14:13 GMT+03:00, Thomas Guillem <thomas@gllm.fr> a écrit :<br></div><blockquote style="margin-top:0pt;margin-right:0pt;margin-bottom:0pt;margin-left:0.8ex;border-left-color:rgb(204, 204, 204);border-left-style:solid;border-left-width:1px;padding-left:1ex;" class="qt-gmail_quote"><pre class="qt-k9mail"><div><hr> src/player/player.h | 4 ++++<br></div><div> 1 file changed, 4 insertions(+)<br></div><div><br></div><div>diff --git a/src/player/player.h b/src/player/player.h<br></div><div>index 36caa46299..60321cb8d0 100644<br></div><div>--- a/src/player/player.h<br></div><div>+++ b/src/player/player.h<br></div><div>@@ -169,6 +169,7 @@ struct vlc_player_t<br></div><div> } destructor;<br></div><div> };<br></div><div> <br></div><div>+#ifndef NDEBUG<br></div><div> /*<br></div><div> * Assert that the player mutex is locked.<br></div><div> *<br></div><div>@@ -181,6 +182,9 @@ vlc_player_assert_locked(vlc_player_t *player)<br></div><div> assert(player);<br></div><div> vlc_mutex_assert(&player->lock);<br></div><div> }<br></div><div>+#else<br></div><div>+#define vlc_player_assert_locked(x) VLC_UNUSED(x)<br></div><div>+#endif<br></div><div> <br></div><div> static inline struct vlc_player_input *<br></div><div> vlc_player_get_input_locked(vlc_player_t *player)<br></div></pre></blockquote></div><div><br></div><div>-- <br></div><div>Envoyé de mon appareil Android avec Courriel K-9 Mail. Veuillez excuser ma brièveté. <br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div></blockquote><div><br></div></body></html>