[vlc-commits] codec: synchro: add logging helpers
Marvin Scholz
git at videolan.org
Thu Mar 26 09:17:28 CET 2020
vlc | branch: master | Marvin Scholz <epirat07 at gmail.com> | Sat Dec 21 03:11:45 2019 +0100| [fb5d2c20589e1627957c6a79cb03e846a467ce57] | committer: Marvin Scholz
codec: synchro: add logging helpers
The synchro module should not print messages when the quiet-synchro
option is set. To simplify this, add two new helper macros for
debug and warning messages (which currently are the only message
types used in this code) and use those instead of always explicitly
checking the b_quiet variable.
> http://git.videolan.org/gitweb.cgi/vlc.git/?a=commit;h=fb5d2c20589e1627957c6a79cb03e846a467ce57
---
modules/codec/synchro.c | 98 +++++++++++++++++++++++++++++--------------------
1 file changed, 58 insertions(+), 40 deletions(-)
diff --git a/modules/codec/synchro.c b/modules/codec/synchro.c
index 3d638cf9ed..0a7f6c1014 100644
--- a/modules/codec/synchro.c
+++ b/modules/codec/synchro.c
@@ -101,6 +101,28 @@
#include <vlc_codec.h>
#include "synchro.h"
+/*
+ * Logging helpers
+ */
+
+/**
+ * Print a debug log message if not silenced
+ * using the `quiet-synchro` option
+ */
+#define synchro_msg_Dbg(p_this, ...) do { \
+ if( !p_this->b_quiet ) \
+ msg_Generic(p_this->p_dec, VLC_MSG_DBG, __VA_ARGS__); \
+ } while (0)
+
+/**
+ * Print a warning log message if not silenced
+ * using the `quiet-synchro` option
+ */
+#define synchro_msg_Warn(p_this, ...) do { \
+ if( !p_this->b_quiet ) \
+ msg_Generic(p_this->p_dec, VLC_MSG_WARN, __VA_ARGS__); \
+ } while (0)
+
/*
* Local prototypes
*/
@@ -259,10 +281,10 @@ bool decoder_SynchroChoose( decoder_synchro_t * p_synchro, int i_coding_type,
if( pts == VLC_TICK_INVALID )
b_decode = 1;
- if( !b_decode && !p_synchro->b_quiet )
+ if( !b_decode )
{
- msg_Warn( p_synchro->p_dec,
- "synchro trashing I (%"PRId64")", pts - now );
+ synchro_msg_Warn( p_synchro,
+ "synchro trashing I (%"PRId64")", pts - now );
}
break;
@@ -415,10 +437,9 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
if( p_synchro->i_eta_p && p_synchro->i_eta_p != p_synchro->i_n_p )
{
#if 0
- if( !p_synchro->b_quiet )
- msg_Dbg( p_synchro->p_dec,
- "stream periodicity changed from P[%d] to P[%d]",
- p_synchro->i_n_p, p_synchro->i_eta_p );
+ synchro_msg_Dbg( p_synchro,
+ "stream periodicity changed from P[%d] to P[%d]",
+ p_synchro->i_n_p, p_synchro->i_eta_p );
#endif
p_synchro->i_n_p = p_synchro->i_eta_p;
}
@@ -430,26 +451,25 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
p_synchro->i_dec_nb_ref = p_synchro->i_nb_ref;
#if 0
- if( !p_synchro->b_quiet )
- msg_Dbg( p_synchro->p_dec, "I(%"PRId64") P(%"PRId64")[%d] B(%"PRId64")"
- "[%d] YUV(%"PRId64") : trashed %d:%d/%d",
- p_synchro->p_tau[I_CODING_TYPE],
- p_synchro->p_tau[P_CODING_TYPE],
- p_synchro->i_n_p,
- p_synchro->p_tau[B_CODING_TYPE],
- p_synchro->i_n_b,
- p_synchro->i_render_time,
- p_synchro->i_not_chosen_pic,
- p_synchro->i_trashed_pic -
- p_synchro->i_not_chosen_pic,
- p_synchro->i_pic );
+ synchro_msg_Dbg( p_synchro, "I(%"PRId64") P(%"PRId64")[%d] B(%"PRId64")"
+ "[%d] YUV(%"PRId64") : trashed %d:%d/%d",
+ p_synchro->p_tau[I_CODING_TYPE],
+ p_synchro->p_tau[P_CODING_TYPE],
+ p_synchro->i_n_p,
+ p_synchro->p_tau[B_CODING_TYPE],
+ p_synchro->i_n_b,
+ p_synchro->i_render_time,
+ p_synchro->i_not_chosen_pic,
+ p_synchro->i_trashed_pic -
+ p_synchro->i_not_chosen_pic,
+ p_synchro->i_pic );
p_synchro->i_trashed_pic = p_synchro->i_not_chosen_pic
= p_synchro->i_pic = 0;
#else
if( p_synchro->i_pic >= 100 )
{
- if( !p_synchro->b_quiet && p_synchro->i_trashed_pic != 0 )
- msg_Dbg( p_synchro->p_dec, "decoded %d/%d pictures",
+ if( p_synchro->i_trashed_pic != 0 )
+ synchro_msg_Dbg( p_synchro, "decoded %d/%d pictures",
p_synchro->i_pic
- p_synchro->i_trashed_pic,
p_synchro->i_pic );
@@ -465,10 +485,9 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
&& p_synchro->i_eta_b != p_synchro->i_n_b )
{
#if 0
- if( !p_synchro->b_quiet )
- msg_Dbg( p_synchro->p_dec,
- "stream periodicity changed from B[%d] to B[%d]",
- p_synchro->i_n_b, p_synchro->i_eta_b );
+ synchro_msg_Dbg( p_synchro,
+ "stream periodicity changed from B[%d] to B[%d]",
+ p_synchro->i_n_b, p_synchro->i_eta_b );
#endif
p_synchro->i_n_b = p_synchro->i_eta_b;
}
@@ -497,12 +516,12 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
if( next_pts != VLC_TICK_INVALID )
{
- if( (next_pts - p_synchro->current_pts
+ if( next_pts - p_synchro->current_pts
> PTS_THRESHOLD
|| p_synchro->current_pts - next_pts
- > PTS_THRESHOLD) && !p_synchro->b_quiet )
+ > PTS_THRESHOLD )
{
- msg_Warn( p_synchro->p_dec, "decoder synchro warning: pts != "
+ synchro_msg_Warn( p_synchro, "decoder synchro warning: pts != "
"current_date (%"PRId64")",
p_synchro->current_pts
- next_pts );
@@ -521,18 +540,18 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
(next_dts - p_synchro->backward_pts
> PTS_THRESHOLD
|| p_synchro->backward_pts - next_dts
- > PTS_THRESHOLD) && !p_synchro->b_quiet )
+ > PTS_THRESHOLD) )
{
- msg_Warn( p_synchro->p_dec, "backward_pts != dts (%"PRId64")",
+ synchro_msg_Warn( p_synchro, "backward_pts != dts (%"PRId64")",
next_dts
- p_synchro->backward_pts );
}
- if( (p_synchro->backward_pts - p_synchro->current_pts
+ if( p_synchro->backward_pts - p_synchro->current_pts
> PTS_THRESHOLD
|| p_synchro->current_pts - p_synchro->backward_pts
- > PTS_THRESHOLD) && !p_synchro->b_quiet )
+ > PTS_THRESHOLD )
{
- msg_Warn( p_synchro->p_dec,
+ synchro_msg_Warn( p_synchro,
"backward_pts != current_pts (%"PRId64")",
p_synchro->current_pts
- p_synchro->backward_pts );
@@ -542,12 +561,12 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
}
else if( next_dts != VLC_TICK_INVALID )
{
- if( (next_dts - p_synchro->current_pts
+ if( next_dts - p_synchro->current_pts
> PTS_THRESHOLD
|| p_synchro->current_pts - next_dts
- > PTS_THRESHOLD) && !p_synchro->b_quiet )
+ > PTS_THRESHOLD )
{
- msg_Warn( p_synchro->p_dec, "dts != current_pts (%"PRId64")",
+ synchro_msg_Warn( p_synchro, "dts != current_pts (%"PRId64")",
p_synchro->current_pts
- next_dts );
}
@@ -569,9 +588,8 @@ void decoder_SynchroNewPicture( decoder_synchro_t * p_synchro, int i_coding_type
{
/* We cannot be _that_ late, something must have happened, reinit
* the dates. */
- if( !p_synchro->b_quiet )
- msg_Warn( p_synchro->p_dec, "PTS << now (%"PRId64"), resetting",
- now - p_synchro->current_pts - DEFAULT_PTS_DELAY );
+ synchro_msg_Warn( p_synchro, "PTS << now (%"PRId64"), resetting",
+ now - p_synchro->current_pts - DEFAULT_PTS_DELAY );
p_synchro->current_pts = now + DEFAULT_PTS_DELAY;
}
if( p_synchro->backward_pts != VLC_TICK_INVALID
More information about the vlc-commits
mailing list