[Ffmpeg-devel] [PATCH] const AVOption
Tue Oct 31 08:39:17 CET 2006
On 10/31/06, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Mon, Oct 30, 2006 at 10:44:50PM +0100, Guillaume POIRIER wrote:
> > On 10/23/06, Steve Lhomme <slhomme at divxcorp.com> wrote:
> > >AVOption data usually don't need to be modified. This patch ensures that.
> > This patch doesn't seem to break anything here. Should I apply it Michael?
> hmm, but what good does it do? are we applying patches just because they
> dont break anything?
Mmm... no :-)
> if adding these consts protects against some bugs then ill review it and
> wont object to it being applied but steve didnt claim that, so iam not
> sure ...
Well in theory compilers are able to do some smarter things when some
values are made const...
In practice, I don't know what GCC or MSVC is able to do though.
Steve, did you find any practical advantage of your patch?
With DADVSI (http://en.wikipedia.org/wiki/DADVSI), France finally has
a lead on USA on selling out individuals right to corporations!
Vive la France!
More information about the ffmpeg-devel