[gst-devel] [gst-cvs] tpm gstreamer: gstreamer/ gstreamer/gst/

[Stefan Kost]

Hi tim,

tpm at kemper.freedesktop.org schrieb:
> CVS Root:       /cvs/gstreamer
> Module:         gstreamer
> Changes by:     tpm
> Date:           Mon Mar 03 2008  14:43:41 UTC
> 
> Log message:
> 	* configure.ac:
> 	* gst/gstconfig.h.in:
> 	* gst/gstregistry.h:
> 	  Fix broken use of config.h-defined preprocessor directive in a public
> 	  header file. Add a corresponding define to gstconfig.h, since we can't
> 	  really remove those function declarations from the header file now
> 	  (or can we? and why are they there in the first place?).
> 
> Modified files:
>     .               : ChangeLog configure.ac
>     gst             : gstconfig.h.in gstregistry.h
> 

you are right. headers should not use config.h values. With you change though I
get a build failure becasue of
> more docs/gst/gstreamer-unused.txt
GST_USING_BINARY_REGISTRY
Easy to fix. Should I file a core-blocker with patch?

Stefan

> Links:
> http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gstreamer/ChangeLog.diff?r1=1.3700&r2=1.3701
> http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gstreamer/configure.ac.diff?r1=1.528&r2=1.529
> http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gstreamer/gst/gstconfig.h.in.diff?r1=1.28&r2=1.29
> http://freedesktop.org/cgi-bin/viewcvs.cgi/gstreamer/gstreamer/gst/gstregistry.h.diff?r1=1.40&r2=1.41
> 
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> _______________________________________________
> gstreamer-cvs mailing list
> gstreamer-cvs at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/gstreamer-cvs