[FFmpeg-devel] [PATCH] Change type of spherical stuff to uint32_t
Vittorio Giovara
vittorio.giovara at gmail.com
Tue Mar 14 17:37:54 EET 2017
On Sun, Mar 12, 2017 at 11:21 PM, Michael Niedermayer
<michael at niedermayer.cc> wrote:
> On Sun, Mar 12, 2017 at 08:54:07PM -0400, Ronald S. Bultje wrote:
>> Hi,
>>
>> On Sun, Mar 12, 2017 at 7:32 PM, James Almer <jamrial at gmail.com> wrote:
>>
>> > On 3/12/2017 6:16 PM, Michael Niedermayer wrote:
>> > > Using the same type across platforms is more robust and avoids platform
>> > > specific issues from differences in range.
I still think you are curing the symptom rather than the illness.
Besides, you can't just change types on a whim, you should wait for
the major bump (if at all).
>> > > Also fixed point integers are on a semantical level not size_t
This is only theoretical, and, since we're talking about semantics,
you're breaking ABI by using sizeof(AVSphericalMapping) outside of
libavutil.
>> > > Previous version reviewed-by: James Almer <jamrial at gmail.com>
>> > > Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
>> > > ---
>> > > ffprobe.c | 2 +-
>> > > libavformat/dump.c | 6 +++---
>> > > libavutil/spherical.c | 6 +++---
>> > > libavutil/spherical.h | 16 ++++++++--------
>> > > 4 files changed, 15 insertions(+), 15 deletions(-)
>> >
>> > Needs FATE update.
>>
>>
>> Since this is Vittorio's code, it's probably good to ask him [CC]? From
>
> yes,
>
> semi on topic: but i think people interrested in FFmpeg
> development and the direction the code goes should subscribe to
> ffmpeg-devel. This CC-ing is not too practical
My subscription to ffmpeg-security was rejected, if you can rectify
that, I'll subscribe to ffmpeg-devel too.
--
Vittorio
More information about the ffmpeg-devel
mailing list