2018-02-14 1:00 GMT+01:00 Muhammad Faiz <mfc...@gmail.com>: > On Tue, Feb 13, 2018 at 4:22 AM, Carl Eugen Hoyos <ceffm...@gmail.com> wrote: >> Hi! >> >> Attached patch marks the RTPDynamicProtocolHandler as const. >> >> Please comment, Carl Eugen >> >> From e219322aa7396db6ecb6ab02fd5b42b42271f27f Mon Sep 17 00:00:00 2001 >> From: Carl Eugen Hoyos <ceffm...@gmail.com> >> Date: Mon, 12 Feb 2018 22:21:06 +0100 >> Subject: [PATCH] lavf/rtpdec*: Constify all RTPDynamicProtocolHandler. > > >> +const extern RTPDynamicProtocolHandler ff_theora_dynamic_handler; >> +const extern RTPDynamicProtocolHandler ff_vc2hq_dynamic_handler; >> +const extern RTPDynamicProtocolHandler ff_vorbis_dynamic_handler; >> +const extern RTPDynamicProtocolHandler ff_vp8_dynamic_handler; >> +const extern RTPDynamicProtocolHandler ff_vp9_dynamic_handler; > > Cosmetics: probably extern const? > > Rest LGTM.
Applied with "extern const". Thank you, Carl Eugen _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel