[FFmpeg-devel] [PATCHv2] all: fix enum definition for large values

Clément Bœsch u at pkh.me
Wed Oct 28 16:44:38 CET 2015


On Wed, Oct 28, 2015 at 10:34:37AM -0400, Ronald S. Bultje wrote:
[...]
> You could do the same for the last 4 values of the cinething changes, they
> are clearly not enums, but flags. Flags are always unsigned, so if enums
> are unsigned and that causes an issue (as it does here), it makes sense to
> fix that at its root, i.e. by making the value unsigned and it thus also
> being a macro instead of an enum. Again, only the last 4 elements, the
> first 5 appear to be genuine enums.

I support this (or just remove the entries altogether since unused).

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 473 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151028/621505a6/attachment.sig>


More information about the ffmpeg-devel mailing list