[FFmpeg-devel] [PATCH 3/5] all: use function pointer instead of void * data pointer

Ganesh Ajjanagadde gajjanag at mit.edu
Sat Oct 24 17:05:43 CEST 2015


On Sat, Oct 24, 2015 at 11:01 AM, wm4 <nfxjfg at googlemail.com> wrote:
> On Sat, 24 Oct 2015 09:28:52 -0400
> Ganesh Ajjanagadde <gajjanagadde at gmail.com> wrote:
>
>> ISO C does not guarantee that functions and data live in the same
>> address space. It is thus (strictly speaking) illegal to assign to a
>
> POSIX does.

Ok. You do not think that the removal of the casts improves
readability (ignoring the ISO C part)?

>
>> void * (which is a data pointer) from a function pointer.
>>
>> Found by enabling -Wpedantic on clang 3.7.
>>
>> Signed-off-by: Ganesh Ajjanagadde <gajjanagadde at gmail.com>
>
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel


More information about the ffmpeg-devel mailing list