[vlc-devel] [vlc-commits] packetizer: mpegvideo: parse color details if present

[Ilkka Ollakka]

On Sat, Apr 23, 2016 at 09:57:14PM +0100, Filipe Cabecinhas wrote:

> Rémi is talking about the C99 standard. You’re converting a uint8_t (from 0x00 to 0xff) into an int8_t. The value in the uint8_t may be out of range of the new type, which would trigger Undefined Behaviour.
> It *might* seem to work, but that’s not enough.

Hi,

I was referring to C99 6.3.1.3, but it's not something that I feel that
strong and I'll push fix to change the variable to unsigned int.

-- 
Ilkka Ollakka
Let the people think they govern and they will be governed.
		-- William Penn, founder of Pennsylvania
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20160510/157c353c/attachment.sig>