On Thu, 2025-05-15 at 14:44 +0200, Andreas Rheinhardt wrote:
> Martijn Otto:
> >  static av_always_inline const FFStream *cffstream(const AVStream
> > *st)
> >  {
> > -    return (FFStream*)st;
> > +    return (const FFStream*)st;
> >  }
> 
> Your patch should be applied on top of git master, not some old
> version.
> The above change is identical to
> 185d0acdc7a67b7d3d78d4c917334c4c3bf9accd.
> 
> Anyway, I remember that someone (IIRC Anton Khirnov) objected to the
> const when this was introduced.
> 
> - 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-request@ffmpeg.org with subject "unsubscribe".

I've updated the patch to apply to latest master, I've made it quite
some time in the past, and something went wrong with my submission
then.

I'm curious to hear what the arguments against these functions taking a
const-qualified stream are, given that we return a const-value and
don't modify it at all.