[FFmpeg-devel] [PATCH 1/2] lavfi/setpts: use macro to access variables array

2023-12-27 Thread Stefano Sabatini
---
 libavfilter/setpts.c | 94 +---
 1 file changed, 45 insertions(+), 49 deletions(-)

diff --git a/libavfilter/setpts.c b/libavfilter/setpts.c
index d057fc91c4..88a8d6af86 100644
--- a/libavfilter/setpts.c
+++ b/libavfilter/setpts.c
@@ -103,6 +103,9 @@ typedef struct SetPTSContext {
 enum AVMediaType type;
 } SetPTSContext;
 
+#define V(name_) \
+setpts->var_values[VAR_##name_]
+
 static av_cold int init(AVFilterContext *ctx)
 {
 SetPTSContext *setpts = ctx->priv;
@@ -114,15 +117,15 @@ static av_cold int init(AVFilterContext *ctx)
 return ret;
 }
 
-setpts->var_values[VAR_N]   = 0.0;
-setpts->var_values[VAR_S]   = 0.0;
-setpts->var_values[VAR_PREV_INPTS]  = NAN;
-setpts->var_values[VAR_PREV_INT]= NAN;
-setpts->var_values[VAR_PREV_OUTPTS] = NAN;
-setpts->var_values[VAR_PREV_OUTT]   = NAN;
-setpts->var_values[VAR_STARTPTS]= NAN;
-setpts->var_values[VAR_STARTT]  = NAN;
-setpts->var_values[VAR_T_CHANGE]= NAN;
+V(N)   = 0.0;
+V(S)   = 0.0;
+V(PREV_INPTS)  = NAN;
+V(PREV_INT)= NAN;
+V(PREV_OUTPTS) = NAN;
+V(PREV_OUTT)   = NAN;
+V(STARTPTS)= NAN;
+V(STARTT)  = NAN;
+V(T_CHANGE)= NAN;
 return 0;
 }
 
@@ -132,22 +135,18 @@ static int config_input(AVFilterLink *inlink)
 SetPTSContext *setpts = ctx->priv;
 
 setpts->type = inlink->type;
-setpts->var_values[VAR_TB] = av_q2d(inlink->time_base);
-setpts->var_values[VAR_RTCSTART] = av_gettime();
+V(TB) = av_q2d(inlink->time_base);
+V(RTCSTART) = av_gettime();
 
-setpts->var_values[VAR_SR] =
-setpts->var_values[VAR_SAMPLE_RATE] =
+V(SR) = V(SAMPLE_RATE) =
 setpts->type == AVMEDIA_TYPE_AUDIO ? inlink->sample_rate : NAN;
 
-setpts->var_values[VAR_FRAME_RATE] =
-setpts->var_values[VAR_FR] = inlink->frame_rate.num &&
- inlink->frame_rate.den ?
-av_q2d(inlink->frame_rate) : NAN;
+V(FRAME_RATE) = V(FR) =
+inlink->frame_rate.num && inlink->frame_rate.den ?
+av_q2d(inlink->frame_rate) : NAN;
 
 av_log(inlink->src, AV_LOG_VERBOSE, "TB:%f FRAME_RATE:%f SAMPLE_RATE:%f\n",
-   setpts->var_values[VAR_TB],
-   setpts->var_values[VAR_FRAME_RATE],
-   setpts->var_values[VAR_SAMPLE_RATE]);
+   V(TB), V(FRAME_RATE), V(SAMPLE_RATE));
 return 0;
 }
 
@@ -162,28 +161,28 @@ static inline char *double2int64str(char *buf, double v)
 
 static double eval_pts(SetPTSContext *setpts, AVFilterLink *inlink, AVFrame 
*frame, int64_t pts)
 {
-if (isnan(setpts->var_values[VAR_STARTPTS])) {
-setpts->var_values[VAR_STARTPTS] = TS2D(pts);
-setpts->var_values[VAR_STARTT  ] = TS2T(pts, inlink->time_base);
+if (isnan(V(STARTPTS))) {
+V(STARTPTS) = TS2D(pts);
+V(STARTT  ) = TS2T(pts, inlink->time_base);
 }
-if (isnan(setpts->var_values[VAR_T_CHANGE])) {
-setpts->var_values[VAR_T_CHANGE] = TS2T(pts, inlink->time_base);
+if (isnan(V(T_CHANGE))) {
+V(T_CHANGE) = TS2T(pts, inlink->time_base);
 }
-setpts->var_values[VAR_PTS   ] = TS2D(pts);
-setpts->var_values[VAR_T ] = TS2T(pts, inlink->time_base);
+V(PTS   ) = TS2D(pts);
+V(T ) = TS2T(pts, inlink->time_base);
 #if FF_API_FRAME_PKT
 FF_DISABLE_DEPRECATION_WARNINGS
-setpts->var_values[VAR_POS   ] = !frame || frame->pkt_pos == -1 ? NAN 
: frame->pkt_pos;
+V(POS   ) = !frame || frame->pkt_pos == -1 ? NAN : frame->pkt_pos;
 FF_ENABLE_DEPRECATION_WARNINGS
 #endif
-setpts->var_values[VAR_RTCTIME   ] = av_gettime();
+V(RTCTIME   ) = av_gettime();
 
 if (frame) {
 if (inlink->type == AVMEDIA_TYPE_VIDEO) {
-setpts->var_values[VAR_INTERLACED] = !!(frame->flags & 
AV_FRAME_FLAG_INTERLACED);
+V(INTERLACED) = !!(frame->flags & AV_FRAME_FLAG_INTERLACED);
 } else if (inlink->type == AVMEDIA_TYPE_AUDIO) {
-setpts->var_values[VAR_S] = frame->nb_samples;
-setpts->var_values[VAR_NB_SAMPLES] = frame->nb_samples;
+V(S) = frame->nb_samples;
+V(NB_SAMPLES) = frame->nb_samples;
 }
 }
 
@@ -202,34 +201,32 @@ static int filter_frame(AVFilterLink *inlink, AVFrame 
*frame)
 
 av_log(inlink->dst, AV_LOG_TRACE,
 "N:%"PRId64" PTS:%s T:%f",
-(int64_t)setpts->var_values[VAR_N],
-d2istr(setpts->var_values[VAR_PTS]),
-setpts->var_values[VAR_T]);
+   (int64_t)V(N), d2istr(V(PTS)), V(T));
 switch (inlink->type) {
 case AVMEDIA_TYPE_VIDEO:
 av_log(inlink->dst, AV_LOG_TRACE, " INTERLACED:%"PRId64,
-(int64_t)setpts->var_values[VAR_INTERLACED]);
+(int64_t)V(INTERLACED));
 break;
 case AVMEDIA_TYPE_AUDIO:
 av_log(inlink->dst, AV_LOG_TRACE, " N

Re: [FFmpeg-devel] [PATCH 1/2] lavfi/setpts: use macro to access variables array

2024-01-02 Thread Stefano Sabatini
On date Thursday 2023-12-28 01:38:25 +0100, Stefano Sabatini wrote:
> ---
>  libavfilter/setpts.c | 94 +---
>  1 file changed, 45 insertions(+), 49 deletions(-)

Applied.
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".