Quoting Diego Biurrun (2016-05-04 09:51:53)
> On Tue, May 03, 2016 at 08:44:49AM +0200, Anton Khirnov wrote:
> > Quoting Martin Storsjö (2016-05-03 08:38:19)
> > > On Tue, 3 May 2016, Anton Khirnov wrote:
> > > > --- a/configure
> > > > +++ b/configure
> > > > @@ -185,53 +185,62 @@ Individual component options:
> > > > +  --enable-libopencv         computer vision
> > > > +  --enable-libopenh264       H.264 video encoding/decoding
> > > 
> > > We only have the encoder half of OpenH264 hooked up
> > 
> > My intention was to describe what the library does, instead of what
> > parts of it we support (since that can and does change and then usually
> > nobody bothers to update the text).
> 
> That's a poor excuse for bad documentation.  Let's instead try to not
> neglect the documentation instead.

It's not an excuse and I don't think the new documentation is "bad". One
argument for making the change is that it's more practical to maintain.
Another is that the switch enables the use of the library, which does
not necessarily mean that all the code using it will actually end up
included in the build (e.g. if you manually disable some components, or
if there are additional deps that are not satisfied).

-- 
Anton Khirnov
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to