Hi all, Apologies for the copy/pasted patch, but can't send via send-email. Copy pasted for review below, attached also for a non-clobbered version.
---------------------------------------------------------------------------------- >From 99cd052b614553757efa1954bf8bf4dd3caf66d0 Mon Sep 17 00:00:00 2001 From: Ganesh Ajjanagadde <gajjanaga...@gmail.com> Date: Fri, 23 Oct 2015 11:59:40 -0400 Subject: [PATCH 2/5] all: fix incorrect usage of %p format specifier ISO C %p format specifier applies to void * data pointers. This casts relevant pointers to void * in order to be strictly correct C. Found by -Wpedantic on clang 3.7 (in particular -Wformat-pedantic). Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> --- libavcodec/alsdec.c | 8 ++++---- libavcodec/h264_refs.c | 6 +++--- libavcodec/libopencore-amr.c | 2 +- libavcodec/mjpegdec.c | 2 +- libavcodec/mpegpicture.c | 2 +- libavcodec/mpegvideo.c | 8 ++++---- libavcodec/options.c | 2 +- libavcodec/pthread_frame.c | 6 +++--- libavcodec/utils.c | 4 ++-- libavcodec/vaapi_h264.c | 2 +- libavcodec/vaapi_hevc.c | 2 +- libavcodec/vaapi_mpeg2.c | 2 +- libavcodec/vaapi_mpeg4.c | 2 +- libavcodec/vaapi_vc1.c | 2 +- libavcodec/vorbisdec.c | 2 +- libavfilter/avfilter.c | 6 +++--- libavformat/asfdec_f.c | 2 +- libavformat/rtpdec_h264.c | 2 +- libavformat/utils.c | 2 +- libswscale/swscale.c | 8 ++++---- 20 files changed, 36 insertions(+), 36 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index ebd364e..19bdbbd 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -1308,8 +1308,8 @@ static int revert_channel_correlation(ALSDecContext *ctx, ALSBlockData *bd, FFMAX(end + 1, end + 1 + t) > ctx->raw_buffer + channels * channel_size - master) { av_log(ctx->avctx, AV_LOG_ERROR, "sample pointer range [%p, %p] not contained in raw_buffer [%p, %p].\n", - master + FFMIN(begin - 1, begin - 1 + t), master + FFMAX(end + 1, end + 1 + t), - ctx->raw_buffer, ctx->raw_buffer + channels * channel_size); + (void *)(master + FFMIN(begin - 1, begin - 1 + t)), (void *)(master + FFMAX(end + 1, end + 1 + t)), + (void *)ctx->raw_buffer, (void *)(ctx->raw_buffer + channels * channel_size)); return AVERROR_INVALIDDATA; } @@ -1330,8 +1330,8 @@ static int revert_channel_correlation(ALSDecContext *ctx, ALSBlockData *bd, end + 1 > ctx->raw_buffer + channels * channel_size - master) { av_log(ctx->avctx, AV_LOG_ERROR, "sample pointer range [%p, %p] not contained in raw_buffer [%p, %p].\n", - master + begin - 1, master + end + 1, - ctx->raw_buffer, ctx->raw_buffer + channels * channel_size); + (void *)(master + begin - 1), (void *)(master + end + 1), + (void *)(ctx->raw_buffer), (void *)(ctx->raw_buffer + channels * channel_size)); return AVERROR_INVALIDDATA; } diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 619f2ed..da6e7fd 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -437,7 +437,7 @@ static H264Picture *find_short(H264Context *h, int frame_num, int *idx) for (i = 0; i < h->short_ref_count; i++) { H264Picture *pic = h->short_ref[i]; if (h->avctx->debug & FF_DEBUG_MMCO) - av_log(h->avctx, AV_LOG_DEBUG, "%d %d %p\n", i, pic->frame_num, pic); + av_log(h->avctx, AV_LOG_DEBUG, "%d %d %p\n", i, pic->frame_num, (void *)pic); if (pic->frame_num == frame_num) { *idx = i; return pic; @@ -543,7 +543,7 @@ static void print_short_term(H264Context *h) for (i = 0; i < h->short_ref_count; i++) { H264Picture *pic = h->short_ref[i]; av_log(h->avctx, AV_LOG_DEBUG, "%"PRIu32" fn:%d poc:%d %p\n", - i, pic->frame_num, pic->poc, pic->f->data[0]); + i, pic->frame_num, pic->poc, (void *)pic->f->data[0]); } } } @@ -560,7 +560,7 @@ static void print_long_term(H264Context *h) H264Picture *pic = h->long_ref[i]; if (pic) { av_log(h->avctx, AV_LOG_DEBUG, "%"PRIu32" fn:%d poc:%d %p\n", - i, pic->frame_num, pic->poc, pic->f->data[0]); + i, pic->frame_num, pic->poc, (void *)pic->f->data[0]); } } } diff --git a/libavcodec/libopencore-amr.c b/libavcodec/libopencore-amr.c index f0e3426..850ad94 100644 --- a/libavcodec/libopencore-amr.c +++ b/libavcodec/libopencore-amr.c @@ -100,7 +100,7 @@ static int amr_nb_decode_frame(AVCodecContext *avctx, void *data, int packet_size, ret; ff_dlog(avctx, "amr_decode_frame buf=%p buf_size=%d frame_count=%d!!\n", - buf, buf_size, avctx->frame_number); + (void *)buf, buf_size, avctx->frame_number); /* get output buffer */ frame->nb_samples = 160; diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index 1a86b7b..36812aa 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -652,7 +652,7 @@ static inline int mjpeg_decode_dc(MJpegDecodeContext *s, int dc_index) if (code < 0 || code > 16) { av_log(s->avctx, AV_LOG_WARNING, "mjpeg_decode_dc: bad vlc: %d:%d (%p)\n", - 0, dc_index, &s->vlcs[0][dc_index]); + 0, dc_index, (void *)&s->vlcs[0][dc_index]); return 0xfffff; } diff --git a/libavcodec/mpegpicture.c b/libavcodec/mpegpicture.c index 16b8f52..76b8dd8 100644 --- a/libavcodec/mpegpicture.c +++ b/libavcodec/mpegpicture.c @@ -123,7 +123,7 @@ static int alloc_frame_buffer(AVCodecContext *avctx, Picture *pic, if (r < 0 || !pic->f->buf[0]) { av_log(avctx, AV_LOG_ERROR, "get_buffer() failed (%d %p)\n", - r, pic->f->data[0]); + r, (void *)pic->f->data[0]); return -1; } diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 96634ec..7f5e617 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -1274,10 +1274,10 @@ int ff_mpv_frame_start(MpegEncContext *s, AVCodecContext *avctx) s->next_picture_ptr = s->current_picture_ptr; } ff_dlog(s->avctx, "L%p N%p C%p L%p N%p C%p type:%d drop:%d\n", - s->last_picture_ptr, s->next_picture_ptr,s->current_picture_ptr, - s->last_picture_ptr ? s->last_picture_ptr->f->data[0] : NULL, - s->next_picture_ptr ? s->next_picture_ptr->f->data[0] : NULL, - s->current_picture_ptr ? s->current_picture_ptr->f->data[0] : NULL, + (void *)s->last_picture_ptr, (void *)s->next_picture_ptr, (void *)s->current_picture_ptr, + s->last_picture_ptr ? (void *)s->last_picture_ptr->f->data[0] : NULL, + s->next_picture_ptr ? (void *)s->next_picture_ptr->f->data[0] : NULL, + s->current_picture_ptr ? (void *)s->current_picture_ptr->f->data[0] : NULL, s->pict_type, s->droppable); if ((!s->last_picture_ptr || !s->last_picture_ptr->f->buf[0]) && diff --git a/libavcodec/options.c b/libavcodec/options.c index ea2563b..9b4f3af 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -187,7 +187,7 @@ int avcodec_copy_context(AVCodecContext *dest, const AVCodecContext *src) if (avcodec_is_open(dest)) { // check that the dest context is uninitialized av_log(dest, AV_LOG_ERROR, "Tried to copy AVCodecContext %p into already-initialized %p\n", - src, dest); + (void *)src, (void *)dest); return AVERROR(EINVAL); } diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 7651211..52161aa 100644 --- a/libavcodec/pthread_frame.c +++ b/libavcodec/pthread_frame.c @@ -486,7 +486,7 @@ void ff_thread_report_progress(ThreadFrame *f, int n, int field) p = f->owner->internal->thread_ctx; if (f->owner->debug&FF_DEBUG_THREADS) - av_log(f->owner, AV_LOG_DEBUG, "%p finished %d field %d\n", progress, n, field); + av_log(f->owner, AV_LOG_DEBUG, "%p finished %d field %d\n", (void *)progress, n, field); pthread_mutex_lock(&p->progress_mutex); progress[field] = n; @@ -504,7 +504,7 @@ void ff_thread_await_progress(ThreadFrame *f, int n, int field) p = f->owner->internal->thread_ctx; if (f->owner->debug&FF_DEBUG_THREADS) - av_log(f->owner, AV_LOG_DEBUG, "thread awaiting %d field %d from %p\n", n, field, progress); + av_log(f->owner, AV_LOG_DEBUG, "thread awaiting %d field %d from %p\n", n, field, (void *)progress); pthread_mutex_lock(&p->progress_mutex); while (progress[field] < n) @@ -868,7 +868,7 @@ void ff_thread_release_buffer(AVCodecContext *avctx, ThreadFrame *f) return; if (avctx->debug & FF_DEBUG_BUFFERS) - av_log(avctx, AV_LOG_DEBUG, "thread_release_buffer called on pic %p\n", f); + av_log(avctx, AV_LOG_DEBUG, "thread_release_buffer called on pic %p\n", (void *)f); av_buffer_unref(&f->progress); f->owner = NULL; diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 83a2078..b78422e 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -642,7 +642,7 @@ static int audio_get_buffer(AVCodecContext *avctx, AVFrame *frame) } if (avctx->debug & FF_DEBUG_BUFFERS) - av_log(avctx, AV_LOG_DEBUG, "default_get_buffer called on frame %p", frame); + av_log(avctx, AV_LOG_DEBUG, "default_get_buffer called on frame %p", (void *)frame); return 0; fail: @@ -680,7 +680,7 @@ static int video_get_buffer(AVCodecContext *s, AVFrame *pic) avpriv_set_systematic_pal2((uint32_t *)pic->data[1], s->pix_fmt); if (s->debug & FF_DEBUG_BUFFERS) - av_log(s, AV_LOG_DEBUG, "default_get_buffer called on pic %p\n", pic); + av_log(s, AV_LOG_DEBUG, "default_get_buffer called on pic %p\n", (void *)pic); return 0; fail: diff --git a/libavcodec/vaapi_h264.c b/libavcodec/vaapi_h264.c index ded2cb3..fc427a6 100644 --- a/libavcodec/vaapi_h264.c +++ b/libavcodec/vaapi_h264.c @@ -324,7 +324,7 @@ static int vaapi_h264_decode_slice(AVCodecContext *avctx, VASliceParameterBufferH264 *slice_param; ff_dlog(avctx, "vaapi_h264_decode_slice(): buffer %p, size %d\n", - buffer, size); + (void *)buffer, size); /* Fill in VASliceParameterBufferH264. */ slice_param = (VASliceParameterBufferH264 *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_hevc.c b/libavcodec/vaapi_hevc.c index 62f783e..d63ac76 100644 --- a/libavcodec/vaapi_hevc.c +++ b/libavcodec/vaapi_hevc.c @@ -407,7 +407,7 @@ static int vaapi_hevc_decode_slice(AVCodecContext *avctx, if (sh->slice_type == I_SLICE) nb_list = 0; - ff_dlog(avctx, "vaapi_hevc_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_hevc_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Fill in VASliceParameterBufferH264. */ slice_param = (VASliceParameterBufferHEVC *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_mpeg2.c b/libavcodec/vaapi_mpeg2.c index 518fec0..c6cda3b 100644 --- a/libavcodec/vaapi_mpeg2.c +++ b/libavcodec/vaapi_mpeg2.c @@ -109,7 +109,7 @@ static int vaapi_mpeg2_decode_slice(AVCodecContext *avctx, const uint8_t *buffer GetBitContext gb; uint32_t quantiser_scale_code, intra_slice_flag, macroblock_offset; - ff_dlog(avctx, "vaapi_mpeg2_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_mpeg2_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Determine macroblock_offset */ init_get_bits(&gb, buffer, 8 * size); diff --git a/libavcodec/vaapi_mpeg4.c b/libavcodec/vaapi_mpeg4.c index b5b946d..a678a3d 100644 --- a/libavcodec/vaapi_mpeg4.c +++ b/libavcodec/vaapi_mpeg4.c @@ -125,7 +125,7 @@ static int vaapi_mpeg4_decode_slice(AVCodecContext *avctx, const uint8_t *buffer FFVAContext * const vactx = ff_vaapi_get_context(avctx); VASliceParameterBufferMPEG4 *slice_param; - ff_dlog(avctx, "vaapi_mpeg4_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_mpeg4_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Fill in VASliceParameterBufferMPEG4 */ slice_param = (VASliceParameterBufferMPEG4 *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_vc1.c b/libavcodec/vaapi_vc1.c index 5ded5db..38a387e 100644 --- a/libavcodec/vaapi_vc1.c +++ b/libavcodec/vaapi_vc1.c @@ -318,7 +318,7 @@ static int vaapi_vc1_decode_slice(AVCodecContext *avctx, const uint8_t *buffer, FFVAContext * const vactx = ff_vaapi_get_context(avctx); VASliceParameterBufferVC1 *slice_param; - ff_dlog(avctx, "vaapi_vc1_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_vc1_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Current bit buffer is beyond any marker for VC-1, so skip it */ if (avctx->codec_id == AV_CODEC_ID_VC1 && IS_MARKER(AV_RB32(buffer))) { diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c index f773afa..3075223 100644 --- a/libavcodec/vorbisdec.c +++ b/libavcodec/vorbisdec.c @@ -663,7 +663,7 @@ static int vorbis_parse_setup_hdr_floors(vorbis_context *vc) ff_dlog(NULL, "floor0 number of books: %u\n", floor_setup->data.t0.num_books); ff_dlog(NULL, "floor0 book list pointer: %p\n", - floor_setup->data.t0.book_list); + (void *)floor_setup->data.t0.book_list); { int idx; for (idx = 0; idx < floor_setup->data.t0.num_books; ++idx) { diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c index 8b1b7d2..841373a 100644 --- a/libavfilter/avfilter.c +++ b/libavfilter/avfilter.c @@ -47,7 +47,7 @@ void ff_tlog_ref(void *ctx, AVFrame *ref, int end) av_unused char buf[16]; ff_tlog(ctx, "ref[%p buf:%p data:%p linesize[%d, %d, %d, %d] pts:%"PRId64" pos:%"PRId64, - ref, ref->buf, ref->data[0], + (void *)ref, (void *)ref->buf, (void *)ref->data[0], ref->linesize[0], ref->linesize[1], ref->linesize[2], ref->linesize[3], ref->pts, av_frame_get_pkt_pos(ref)); @@ -321,7 +321,7 @@ void ff_tlog_link(void *ctx, AVFilterLink *link, int end) if (link->type == AVMEDIA_TYPE_VIDEO) { ff_tlog(ctx, "link[%p s:%dx%d fmt:%s %s->%s]%s", - link, link->w, link->h, + (void *)link, link->w, link->h, av_get_pix_fmt_name(link->format), link->src ? link->src->filter->name : "", link->dst ? link->dst->filter->name : "", @@ -332,7 +332,7 @@ void ff_tlog_link(void *ctx, AVFilterLink *link, int end) ff_tlog(ctx, "link[%p r:%d cl:%s fmt:%s %s->%s]%s", - link, (int)link->sample_rate, buf, + (void *)link, (int)link->sample_rate, buf, av_get_sample_fmt_name(link->format), link->src ? link->src->filter->name : "", link->dst ? link->dst->filter->name : "", diff --git a/libavformat/asfdec_f.c b/libavformat/asfdec_f.c index 7c31cfa..538c965 100644 --- a/libavformat/asfdec_f.c +++ b/libavformat/asfdec_f.c @@ -1327,7 +1327,7 @@ static int asf_parse_packet(AVFormatContext *s, AVIOContext *pb, AVPacket *pkt) /* read data */ av_log(asf, AV_LOG_TRACE, "READ PACKET s:%d os:%d o:%d,%d l:%d DATA:%p\n", s->packet_size, asf_st->pkt.size, asf->packet_frag_offset, - asf_st->frag_offset, asf->packet_frag_size, asf_st->pkt.data); + asf_st->frag_offset, asf->packet_frag_size, (void *)asf_st->pkt.data); asf->packet_size_left -= asf->packet_frag_size; if (asf->packet_size_left < 0) continue; diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c index b399be4..a556fda 100644 --- a/libavformat/rtpdec_h264.c +++ b/libavformat/rtpdec_h264.c @@ -171,7 +171,7 @@ static int sdp_parse_fmtp_config_h264(AVFormatContext *s, ret = ff_h264_parse_sprop_parameter_sets(s, &codec->extradata, &codec->extradata_size, value); av_log(s, AV_LOG_DEBUG, "Extradata set to %p (size: %d)\n", - codec->extradata, codec->extradata_size); + (void *)codec->extradata, codec->extradata_size); return ret; } return 0; diff --git a/libavformat/utils.c b/libavformat/utils.c index 689473e..9a25b77 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -1096,7 +1096,7 @@ static void compute_pkt_fields(AVFormatContext *s, AVStream *st, av_log(s, AV_LOG_TRACE, "IN delayed:%d pts:%s, dts:%s cur_dts:%s st:%d pc:%p duration:%"PRId64" delay:%d onein_oneout:%d\n", presentation_delayed, av_ts2str(pkt->pts), av_ts2str(pkt->dts), av_ts2str(st->cur_dts), - pkt->stream_index, pc, pkt->duration, delay, onein_oneout); + pkt->stream_index, (void *)pc, pkt->duration, delay, onein_oneout); /* Interpolate PTS and DTS if they are not present. We skip H264 * currently because delay and has_b_frames are not reliably set. */ diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 1769348..2567390 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -421,10 +421,10 @@ static int swscale(SwsContext *c, const uint8_t *src[], srcStride[2] <<= c->vChrDrop; DEBUG_BUFFERS("swscale() %p[%d] %p[%d] %p[%d] %p[%d] -> %p[%d] %p[%d] %p[%d] %p[%d]\n", - src[0], srcStride[0], src[1], srcStride[1], - src[2], srcStride[2], src[3], srcStride[3], - dst[0], dstStride[0], dst[1], dstStride[1], - dst[2], dstStride[2], dst[3], dstStride[3]); + (void *)src[0], srcStride[0], (void *)src[1], srcStride[1], + (void *)src[2], srcStride[2], (void *)src[3], srcStride[3], + (void *)dst[0], dstStride[0], (void *)dst[1], dstStride[1], + (void *)dst[2], dstStride[2], (void *)dst[3], dstStride[3]); DEBUG_BUFFERS("srcSliceY: %d srcSliceH: %d dstY: %d dstH: %d\n", srcSliceY, srcSliceH, dstY, dstH); DEBUG_BUFFERS("vLumFilterSize: %d vLumBufSize: %d vChrFilterSize: %d vChrBufSize: %d\n", -- 2.6.2
From 99cd052b614553757efa1954bf8bf4dd3caf66d0 Mon Sep 17 00:00:00 2001 From: Ganesh Ajjanagadde <gajjanaga...@gmail.com> Date: Fri, 23 Oct 2015 11:59:40 -0400 Subject: [PATCH 2/5] all: fix incorrect usage of %p format specifier ISO C %p format specifier applies to void * data pointers. This casts relevant pointers to void * in order to be strictly correct C. Found by -Wpedantic on clang 3.7 (in particular -Wformat-pedantic). Signed-off-by: Ganesh Ajjanagadde <gajjanaga...@gmail.com> --- libavcodec/alsdec.c | 8 ++++---- libavcodec/h264_refs.c | 6 +++--- libavcodec/libopencore-amr.c | 2 +- libavcodec/mjpegdec.c | 2 +- libavcodec/mpegpicture.c | 2 +- libavcodec/mpegvideo.c | 8 ++++---- libavcodec/options.c | 2 +- libavcodec/pthread_frame.c | 6 +++--- libavcodec/utils.c | 4 ++-- libavcodec/vaapi_h264.c | 2 +- libavcodec/vaapi_hevc.c | 2 +- libavcodec/vaapi_mpeg2.c | 2 +- libavcodec/vaapi_mpeg4.c | 2 +- libavcodec/vaapi_vc1.c | 2 +- libavcodec/vorbisdec.c | 2 +- libavfilter/avfilter.c | 6 +++--- libavformat/asfdec_f.c | 2 +- libavformat/rtpdec_h264.c | 2 +- libavformat/utils.c | 2 +- libswscale/swscale.c | 8 ++++---- 20 files changed, 36 insertions(+), 36 deletions(-) diff --git a/libavcodec/alsdec.c b/libavcodec/alsdec.c index ebd364e..19bdbbd 100644 --- a/libavcodec/alsdec.c +++ b/libavcodec/alsdec.c @@ -1308,8 +1308,8 @@ static int revert_channel_correlation(ALSDecContext *ctx, ALSBlockData *bd, FFMAX(end + 1, end + 1 + t) > ctx->raw_buffer + channels * channel_size - master) { av_log(ctx->avctx, AV_LOG_ERROR, "sample pointer range [%p, %p] not contained in raw_buffer [%p, %p].\n", - master + FFMIN(begin - 1, begin - 1 + t), master + FFMAX(end + 1, end + 1 + t), - ctx->raw_buffer, ctx->raw_buffer + channels * channel_size); + (void *)(master + FFMIN(begin - 1, begin - 1 + t)), (void *)(master + FFMAX(end + 1, end + 1 + t)), + (void *)ctx->raw_buffer, (void *)(ctx->raw_buffer + channels * channel_size)); return AVERROR_INVALIDDATA; } @@ -1330,8 +1330,8 @@ static int revert_channel_correlation(ALSDecContext *ctx, ALSBlockData *bd, end + 1 > ctx->raw_buffer + channels * channel_size - master) { av_log(ctx->avctx, AV_LOG_ERROR, "sample pointer range [%p, %p] not contained in raw_buffer [%p, %p].\n", - master + begin - 1, master + end + 1, - ctx->raw_buffer, ctx->raw_buffer + channels * channel_size); + (void *)(master + begin - 1), (void *)(master + end + 1), + (void *)(ctx->raw_buffer), (void *)(ctx->raw_buffer + channels * channel_size)); return AVERROR_INVALIDDATA; } diff --git a/libavcodec/h264_refs.c b/libavcodec/h264_refs.c index 619f2ed..da6e7fd 100644 --- a/libavcodec/h264_refs.c +++ b/libavcodec/h264_refs.c @@ -437,7 +437,7 @@ static H264Picture *find_short(H264Context *h, int frame_num, int *idx) for (i = 0; i < h->short_ref_count; i++) { H264Picture *pic = h->short_ref[i]; if (h->avctx->debug & FF_DEBUG_MMCO) - av_log(h->avctx, AV_LOG_DEBUG, "%d %d %p\n", i, pic->frame_num, pic); + av_log(h->avctx, AV_LOG_DEBUG, "%d %d %p\n", i, pic->frame_num, (void *)pic); if (pic->frame_num == frame_num) { *idx = i; return pic; @@ -543,7 +543,7 @@ static void print_short_term(H264Context *h) for (i = 0; i < h->short_ref_count; i++) { H264Picture *pic = h->short_ref[i]; av_log(h->avctx, AV_LOG_DEBUG, "%"PRIu32" fn:%d poc:%d %p\n", - i, pic->frame_num, pic->poc, pic->f->data[0]); + i, pic->frame_num, pic->poc, (void *)pic->f->data[0]); } } } @@ -560,7 +560,7 @@ static void print_long_term(H264Context *h) H264Picture *pic = h->long_ref[i]; if (pic) { av_log(h->avctx, AV_LOG_DEBUG, "%"PRIu32" fn:%d poc:%d %p\n", - i, pic->frame_num, pic->poc, pic->f->data[0]); + i, pic->frame_num, pic->poc, (void *)pic->f->data[0]); } } } diff --git a/libavcodec/libopencore-amr.c b/libavcodec/libopencore-amr.c index f0e3426..850ad94 100644 --- a/libavcodec/libopencore-amr.c +++ b/libavcodec/libopencore-amr.c @@ -100,7 +100,7 @@ static int amr_nb_decode_frame(AVCodecContext *avctx, void *data, int packet_size, ret; ff_dlog(avctx, "amr_decode_frame buf=%p buf_size=%d frame_count=%d!!\n", - buf, buf_size, avctx->frame_number); + (void *)buf, buf_size, avctx->frame_number); /* get output buffer */ frame->nb_samples = 160; diff --git a/libavcodec/mjpegdec.c b/libavcodec/mjpegdec.c index 1a86b7b..36812aa 100644 --- a/libavcodec/mjpegdec.c +++ b/libavcodec/mjpegdec.c @@ -652,7 +652,7 @@ static inline int mjpeg_decode_dc(MJpegDecodeContext *s, int dc_index) if (code < 0 || code > 16) { av_log(s->avctx, AV_LOG_WARNING, "mjpeg_decode_dc: bad vlc: %d:%d (%p)\n", - 0, dc_index, &s->vlcs[0][dc_index]); + 0, dc_index, (void *)&s->vlcs[0][dc_index]); return 0xfffff; } diff --git a/libavcodec/mpegpicture.c b/libavcodec/mpegpicture.c index 16b8f52..76b8dd8 100644 --- a/libavcodec/mpegpicture.c +++ b/libavcodec/mpegpicture.c @@ -123,7 +123,7 @@ static int alloc_frame_buffer(AVCodecContext *avctx, Picture *pic, if (r < 0 || !pic->f->buf[0]) { av_log(avctx, AV_LOG_ERROR, "get_buffer() failed (%d %p)\n", - r, pic->f->data[0]); + r, (void *)pic->f->data[0]); return -1; } diff --git a/libavcodec/mpegvideo.c b/libavcodec/mpegvideo.c index 96634ec..7f5e617 100644 --- a/libavcodec/mpegvideo.c +++ b/libavcodec/mpegvideo.c @@ -1274,10 +1274,10 @@ int ff_mpv_frame_start(MpegEncContext *s, AVCodecContext *avctx) s->next_picture_ptr = s->current_picture_ptr; } ff_dlog(s->avctx, "L%p N%p C%p L%p N%p C%p type:%d drop:%d\n", - s->last_picture_ptr, s->next_picture_ptr,s->current_picture_ptr, - s->last_picture_ptr ? s->last_picture_ptr->f->data[0] : NULL, - s->next_picture_ptr ? s->next_picture_ptr->f->data[0] : NULL, - s->current_picture_ptr ? s->current_picture_ptr->f->data[0] : NULL, + (void *)s->last_picture_ptr, (void *)s->next_picture_ptr, (void *)s->current_picture_ptr, + s->last_picture_ptr ? (void *)s->last_picture_ptr->f->data[0] : NULL, + s->next_picture_ptr ? (void *)s->next_picture_ptr->f->data[0] : NULL, + s->current_picture_ptr ? (void *)s->current_picture_ptr->f->data[0] : NULL, s->pict_type, s->droppable); if ((!s->last_picture_ptr || !s->last_picture_ptr->f->buf[0]) && diff --git a/libavcodec/options.c b/libavcodec/options.c index ea2563b..9b4f3af 100644 --- a/libavcodec/options.c +++ b/libavcodec/options.c @@ -187,7 +187,7 @@ int avcodec_copy_context(AVCodecContext *dest, const AVCodecContext *src) if (avcodec_is_open(dest)) { // check that the dest context is uninitialized av_log(dest, AV_LOG_ERROR, "Tried to copy AVCodecContext %p into already-initialized %p\n", - src, dest); + (void *)src, (void *)dest); return AVERROR(EINVAL); } diff --git a/libavcodec/pthread_frame.c b/libavcodec/pthread_frame.c index 7651211..52161aa 100644 --- a/libavcodec/pthread_frame.c +++ b/libavcodec/pthread_frame.c @@ -486,7 +486,7 @@ void ff_thread_report_progress(ThreadFrame *f, int n, int field) p = f->owner->internal->thread_ctx; if (f->owner->debug&FF_DEBUG_THREADS) - av_log(f->owner, AV_LOG_DEBUG, "%p finished %d field %d\n", progress, n, field); + av_log(f->owner, AV_LOG_DEBUG, "%p finished %d field %d\n", (void *)progress, n, field); pthread_mutex_lock(&p->progress_mutex); progress[field] = n; @@ -504,7 +504,7 @@ void ff_thread_await_progress(ThreadFrame *f, int n, int field) p = f->owner->internal->thread_ctx; if (f->owner->debug&FF_DEBUG_THREADS) - av_log(f->owner, AV_LOG_DEBUG, "thread awaiting %d field %d from %p\n", n, field, progress); + av_log(f->owner, AV_LOG_DEBUG, "thread awaiting %d field %d from %p\n", n, field, (void *)progress); pthread_mutex_lock(&p->progress_mutex); while (progress[field] < n) @@ -868,7 +868,7 @@ void ff_thread_release_buffer(AVCodecContext *avctx, ThreadFrame *f) return; if (avctx->debug & FF_DEBUG_BUFFERS) - av_log(avctx, AV_LOG_DEBUG, "thread_release_buffer called on pic %p\n", f); + av_log(avctx, AV_LOG_DEBUG, "thread_release_buffer called on pic %p\n", (void *)f); av_buffer_unref(&f->progress); f->owner = NULL; diff --git a/libavcodec/utils.c b/libavcodec/utils.c index 83a2078..b78422e 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -642,7 +642,7 @@ static int audio_get_buffer(AVCodecContext *avctx, AVFrame *frame) } if (avctx->debug & FF_DEBUG_BUFFERS) - av_log(avctx, AV_LOG_DEBUG, "default_get_buffer called on frame %p", frame); + av_log(avctx, AV_LOG_DEBUG, "default_get_buffer called on frame %p", (void *)frame); return 0; fail: @@ -680,7 +680,7 @@ static int video_get_buffer(AVCodecContext *s, AVFrame *pic) avpriv_set_systematic_pal2((uint32_t *)pic->data[1], s->pix_fmt); if (s->debug & FF_DEBUG_BUFFERS) - av_log(s, AV_LOG_DEBUG, "default_get_buffer called on pic %p\n", pic); + av_log(s, AV_LOG_DEBUG, "default_get_buffer called on pic %p\n", (void *)pic); return 0; fail: diff --git a/libavcodec/vaapi_h264.c b/libavcodec/vaapi_h264.c index ded2cb3..fc427a6 100644 --- a/libavcodec/vaapi_h264.c +++ b/libavcodec/vaapi_h264.c @@ -324,7 +324,7 @@ static int vaapi_h264_decode_slice(AVCodecContext *avctx, VASliceParameterBufferH264 *slice_param; ff_dlog(avctx, "vaapi_h264_decode_slice(): buffer %p, size %d\n", - buffer, size); + (void *)buffer, size); /* Fill in VASliceParameterBufferH264. */ slice_param = (VASliceParameterBufferH264 *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_hevc.c b/libavcodec/vaapi_hevc.c index 62f783e..d63ac76 100644 --- a/libavcodec/vaapi_hevc.c +++ b/libavcodec/vaapi_hevc.c @@ -407,7 +407,7 @@ static int vaapi_hevc_decode_slice(AVCodecContext *avctx, if (sh->slice_type == I_SLICE) nb_list = 0; - ff_dlog(avctx, "vaapi_hevc_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_hevc_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Fill in VASliceParameterBufferH264. */ slice_param = (VASliceParameterBufferHEVC *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_mpeg2.c b/libavcodec/vaapi_mpeg2.c index 518fec0..c6cda3b 100644 --- a/libavcodec/vaapi_mpeg2.c +++ b/libavcodec/vaapi_mpeg2.c @@ -109,7 +109,7 @@ static int vaapi_mpeg2_decode_slice(AVCodecContext *avctx, const uint8_t *buffer GetBitContext gb; uint32_t quantiser_scale_code, intra_slice_flag, macroblock_offset; - ff_dlog(avctx, "vaapi_mpeg2_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_mpeg2_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Determine macroblock_offset */ init_get_bits(&gb, buffer, 8 * size); diff --git a/libavcodec/vaapi_mpeg4.c b/libavcodec/vaapi_mpeg4.c index b5b946d..a678a3d 100644 --- a/libavcodec/vaapi_mpeg4.c +++ b/libavcodec/vaapi_mpeg4.c @@ -125,7 +125,7 @@ static int vaapi_mpeg4_decode_slice(AVCodecContext *avctx, const uint8_t *buffer FFVAContext * const vactx = ff_vaapi_get_context(avctx); VASliceParameterBufferMPEG4 *slice_param; - ff_dlog(avctx, "vaapi_mpeg4_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_mpeg4_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Fill in VASliceParameterBufferMPEG4 */ slice_param = (VASliceParameterBufferMPEG4 *)ff_vaapi_alloc_slice(vactx, buffer, size); diff --git a/libavcodec/vaapi_vc1.c b/libavcodec/vaapi_vc1.c index 5ded5db..38a387e 100644 --- a/libavcodec/vaapi_vc1.c +++ b/libavcodec/vaapi_vc1.c @@ -318,7 +318,7 @@ static int vaapi_vc1_decode_slice(AVCodecContext *avctx, const uint8_t *buffer, FFVAContext * const vactx = ff_vaapi_get_context(avctx); VASliceParameterBufferVC1 *slice_param; - ff_dlog(avctx, "vaapi_vc1_decode_slice(): buffer %p, size %d\n", buffer, size); + ff_dlog(avctx, "vaapi_vc1_decode_slice(): buffer %p, size %d\n", (void *)buffer, size); /* Current bit buffer is beyond any marker for VC-1, so skip it */ if (avctx->codec_id == AV_CODEC_ID_VC1 && IS_MARKER(AV_RB32(buffer))) { diff --git a/libavcodec/vorbisdec.c b/libavcodec/vorbisdec.c index f773afa..3075223 100644 --- a/libavcodec/vorbisdec.c +++ b/libavcodec/vorbisdec.c @@ -663,7 +663,7 @@ static int vorbis_parse_setup_hdr_floors(vorbis_context *vc) ff_dlog(NULL, "floor0 number of books: %u\n", floor_setup->data.t0.num_books); ff_dlog(NULL, "floor0 book list pointer: %p\n", - floor_setup->data.t0.book_list); + (void *)floor_setup->data.t0.book_list); { int idx; for (idx = 0; idx < floor_setup->data.t0.num_books; ++idx) { diff --git a/libavfilter/avfilter.c b/libavfilter/avfilter.c index 8b1b7d2..841373a 100644 --- a/libavfilter/avfilter.c +++ b/libavfilter/avfilter.c @@ -47,7 +47,7 @@ void ff_tlog_ref(void *ctx, AVFrame *ref, int end) av_unused char buf[16]; ff_tlog(ctx, "ref[%p buf:%p data:%p linesize[%d, %d, %d, %d] pts:%"PRId64" pos:%"PRId64, - ref, ref->buf, ref->data[0], + (void *)ref, (void *)ref->buf, (void *)ref->data[0], ref->linesize[0], ref->linesize[1], ref->linesize[2], ref->linesize[3], ref->pts, av_frame_get_pkt_pos(ref)); @@ -321,7 +321,7 @@ void ff_tlog_link(void *ctx, AVFilterLink *link, int end) if (link->type == AVMEDIA_TYPE_VIDEO) { ff_tlog(ctx, "link[%p s:%dx%d fmt:%s %s->%s]%s", - link, link->w, link->h, + (void *)link, link->w, link->h, av_get_pix_fmt_name(link->format), link->src ? link->src->filter->name : "", link->dst ? link->dst->filter->name : "", @@ -332,7 +332,7 @@ void ff_tlog_link(void *ctx, AVFilterLink *link, int end) ff_tlog(ctx, "link[%p r:%d cl:%s fmt:%s %s->%s]%s", - link, (int)link->sample_rate, buf, + (void *)link, (int)link->sample_rate, buf, av_get_sample_fmt_name(link->format), link->src ? link->src->filter->name : "", link->dst ? link->dst->filter->name : "", diff --git a/libavformat/asfdec_f.c b/libavformat/asfdec_f.c index 7c31cfa..538c965 100644 --- a/libavformat/asfdec_f.c +++ b/libavformat/asfdec_f.c @@ -1327,7 +1327,7 @@ static int asf_parse_packet(AVFormatContext *s, AVIOContext *pb, AVPacket *pkt) /* read data */ av_log(asf, AV_LOG_TRACE, "READ PACKET s:%d os:%d o:%d,%d l:%d DATA:%p\n", s->packet_size, asf_st->pkt.size, asf->packet_frag_offset, - asf_st->frag_offset, asf->packet_frag_size, asf_st->pkt.data); + asf_st->frag_offset, asf->packet_frag_size, (void *)asf_st->pkt.data); asf->packet_size_left -= asf->packet_frag_size; if (asf->packet_size_left < 0) continue; diff --git a/libavformat/rtpdec_h264.c b/libavformat/rtpdec_h264.c index b399be4..a556fda 100644 --- a/libavformat/rtpdec_h264.c +++ b/libavformat/rtpdec_h264.c @@ -171,7 +171,7 @@ static int sdp_parse_fmtp_config_h264(AVFormatContext *s, ret = ff_h264_parse_sprop_parameter_sets(s, &codec->extradata, &codec->extradata_size, value); av_log(s, AV_LOG_DEBUG, "Extradata set to %p (size: %d)\n", - codec->extradata, codec->extradata_size); + (void *)codec->extradata, codec->extradata_size); return ret; } return 0; diff --git a/libavformat/utils.c b/libavformat/utils.c index 689473e..9a25b77 100644 --- a/libavformat/utils.c +++ b/libavformat/utils.c @@ -1096,7 +1096,7 @@ static void compute_pkt_fields(AVFormatContext *s, AVStream *st, av_log(s, AV_LOG_TRACE, "IN delayed:%d pts:%s, dts:%s cur_dts:%s st:%d pc:%p duration:%"PRId64" delay:%d onein_oneout:%d\n", presentation_delayed, av_ts2str(pkt->pts), av_ts2str(pkt->dts), av_ts2str(st->cur_dts), - pkt->stream_index, pc, pkt->duration, delay, onein_oneout); + pkt->stream_index, (void *)pc, pkt->duration, delay, onein_oneout); /* Interpolate PTS and DTS if they are not present. We skip H264 * currently because delay and has_b_frames are not reliably set. */ diff --git a/libswscale/swscale.c b/libswscale/swscale.c index 1769348..2567390 100644 --- a/libswscale/swscale.c +++ b/libswscale/swscale.c @@ -421,10 +421,10 @@ static int swscale(SwsContext *c, const uint8_t *src[], srcStride[2] <<= c->vChrDrop; DEBUG_BUFFERS("swscale() %p[%d] %p[%d] %p[%d] %p[%d] -> %p[%d] %p[%d] %p[%d] %p[%d]\n", - src[0], srcStride[0], src[1], srcStride[1], - src[2], srcStride[2], src[3], srcStride[3], - dst[0], dstStride[0], dst[1], dstStride[1], - dst[2], dstStride[2], dst[3], dstStride[3]); + (void *)src[0], srcStride[0], (void *)src[1], srcStride[1], + (void *)src[2], srcStride[2], (void *)src[3], srcStride[3], + (void *)dst[0], dstStride[0], (void *)dst[1], dstStride[1], + (void *)dst[2], dstStride[2], (void *)dst[3], dstStride[3]); DEBUG_BUFFERS("srcSliceY: %d srcSliceH: %d dstY: %d dstH: %d\n", srcSliceY, srcSliceH, dstY, dstH); DEBUG_BUFFERS("vLumFilterSize: %d vLumBufSize: %d vChrFilterSize: %d vChrBufSize: %d\n", -- 2.6.2
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel