[FFmpeg-devel] [PATCH 001/244] Add a new channel layout API

Anton Khirnov anton at khirnov.net
Tue Dec 10 13:57:28 EET 2019


Quoting Nicolas George (2019-12-09 20:34:45)
> Michael Niedermayer (12019-12-09):
> > mixing unsigned and signed int of course works if one is aware of
> > * what is signed, what is unsigned
> > * exact semantics of expressions mixing them
> > * does not miss any corner cases
> 
> I think it is reasonable to demand from FFmpeg developers that they know
> enough C to be at ease with this.

I disagree. I think I have a fair amount of skill and experience as a C
developer, but I still get hit by those issues frequently. It's extra
trouble for only theoretical gain.

Will reply to the rest of your remarks later.

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list