Re: [FFmpeg-devel] [PATCH v2 0/1] [WIP] avutil/csp changes

2022-05-22 Thread Niklas Haas
Would it make sense to regroup the structs like this? typedef struct AVCIExy { AVRational x, y; } AVCIExy; struct AVPrimaryCoefficients { AVCIExy r, g, b; AVCIExy wp; }; typedef AVCIExy AVWhitepointCoefficients; /* optional */ On Sun, 22 May 2022 01:31:43 -0400 Leo Izen wrote: > Ch

[FFmpeg-devel] [PATCH v2 0/1] [WIP] avutil/csp changes

2022-05-21 Thread Leo Izen
Changes in v2: - increase precision for AVR() macro to 100k, at haasn's request - add #define AVUTIL_CSP_DENOM 10 - add 0.5 to the AVR macro definition to get exact values from truncation This patch is a work in progress example for swapping these structs from doubles to AVRationals. There's