Hi,

On Tue, Aug 24, 2021 at 9:07 AM Olivier Crête <olivier.cr...@collabora.com>
wrote:

> Hi Andreas,
>
> > Generally, one should forward error codes and not make up some on the
> > fly; in this case, the callee does not return proper error codes, so
> > this should be fixed, too (but not in the same commit and not
> > necessarily by you). Even more importantly, the callee emits an error
> > message, stating that an error will be ignored, which will no longer
> > true with this patch.
>
> If I understand correctly, the "ignoring" refers to the block being
> ignored because of errors, not to the error being ignored.
>

No, the opposite. He suggests to try:

ret = try_decode_wma();
if (ret < 0) return ret;

instead of:

if (ret < 0) return AVERROR_INVALIDDATA;

Ronald
_______________________________________________
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".

Reply via email to