[FFmpeg-cvslog] r17168 - in trunk/libavcodec: vdpau.c vdpau.h vdpau_pixfmt.h xvmc.c xvmc_pixfmt.h xvmc_render.h
Sat Feb 14 14:38:07 CET 2009
On Fri, Feb 13, 2009 at 10:21:00AM +0100, Diego Biurrun wrote:
> On Thu, Feb 12, 2009 at 11:54:14PM +0100, Aurelien Jacobs wrote:
> > Diego Biurrun wrote:
> > > On Thu, Feb 12, 2009 at 02:09:02AM +0100, Aurelien Jacobs wrote:
> > > > diego wrote:
> > > >
> > > > > Log:
> > > > > Rename [...] and vdpau.h --> vdpau_pixfmt.h
> > > >
> > > > This is broken. Make install won't work anymore.
> > > > Moreover, strictly speaking, you are breaking public API, and
> > > > thus this would require a major version bump.
> > >
> > > Hmmm....
> > This don't really tells me much...
> > Anyway, the situation is quite clear:
> > - either you consider this header part of public API, and then
> > you can't rename it without a major version bump
> > - or you don't consider it part of public API, and then it
> > must not be installed
> > I can't see any other possibility.
> I do consider it part of the public API, how else are people supposed
> to use vdpau/xvmc right now? When the patch was approved, the header
> was installed on purpose for this very reason.
> So all in all it's clearly my bad, I'll rename it back later today.
More information about the ffmpeg-cvslog