Leo Izen:
>
> On 4/15/22 07:34, Andreas Rheinhardt wrote:
>> Leo Izen:
>>> +static int jpegxl_probe(const AVProbeData *p)
>>> +{
>>> + const uint8_t *b = p->buf;
>>> +
>>> + /* ISOBMFF-based container */
>>> + /* 0x4a584c20 == "JXL " */
>>> + if (AV_RL64(b) == FF_JPEGXL_CONTAINER_SIGNATURE_LE)
>>> + return AVPROBE_SCORE_EXTENSION + 1;
>>> + /* Raw codestreams all start with 0xff0a */
>>> + if (AV_RL16(b) != FF_JPEGXL_CODESTREAM_SIGNATURE_LE)
>>> + return 0;
>>> + if (ff_jpegxl_verify_codestream_header(p->buf, p->buf_size) >= 0)
>> This will give a linking failure if the image_jpegxl_pipe_demuxer is
>> disabled.
>>
> I thought of that, and I tested it, and it doesn't. It produces a
> compiler warning that the static function jpegxl_probe is never called,
> which means the linker probably throws away the function entirely, thus
> eliminating a linking error. Though I admit that's a guess.
That only works when optimizations are enabled. Try again with -O0.
- Andreas
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".
