-----Original Message----- From: ffmpeg-devel <ffmpeg-devel-bounces@ffmpeg.org> On Behalf Of Mark Thompson Sent: Monday, May 2, 2022 12:01 AM To: ffmpeg-devel@ffmpeg.org Subject: Re: [FFmpeg-devel] [PATCH 1/3] avutils/hwcontext: add derive- device function which searches for existing devices in both directions
[..]
There was a reason why this can't be used. IIRC, it was that the device count needs to be known at some other place where av_hwdevice_iterate_types() is not available.
Please see the previous discussion with Hendrik about this.
Where is that? The only place I see this used is the array of derived devices.
Sorry, it was with Lynne, not Hendrik: https://www.mail-archive.com/ffmpeg-devel@ffmpeg.org/msg127768.html
Two alternative implementations without the constant spring to mind:
* A shorter array indexed by av_hwdevice_iterate_types() which would not have empty entries for devices not compatible with the current platform.
* An array of type/reference pairs.
I would prefer not to over-engineer this. Everybody else seemed to be OK with the current approach. Best regards, softworkz