[libdvdcss-devel] [PATCH 28/47] Replace GCC-specific bits in print_debug macro by standards-compliant code.
Jean-Baptiste Kempf
jb at videolan.org
Thu Oct 30 13:57:33 CET 2014
On 30 Oct, Diego Biurrun wrote :
> On Thu, Oct 30, 2014 at 08:02:45AM +0100, Reimar Döffinger wrote:
> > On 29.10.2014, at 21:33, Diego Biurrun <diego at biurrun.de> wrote:
> > > --- a/src/libdvdcss.h
> > > +++ b/src/libdvdcss.h
> > > @@ -87,27 +87,13 @@ struct dvdcss_s
> > > /*****************************************************************************
> > > * Functions used across the library
> > > *****************************************************************************/
> > > -#if defined( _MSC_VER )
> > > -#include <stdarg.h>
> > > -__forceinline void print_debug(dvdcss_t dvdcss, const char *msg,...)
> > > -{
> > > - va_list args;
> > > -
> > > - fprintf( stderr, "libdvdcss debug: " );
> > > - va_start( args, msg );
> > > - vfprintf( stderr, msg, args );
> > > - va_end( args );
> > > - fprintf( stderr, "\n" );
> > > -}
> > > -#else
> > > -#define print_debug(dvdcss,msg,args...) \
> > > +#define print_debug( dvdcss, ... ) \
> > > if( dvdcss->b_debug ) \
> > > { \
> > > fprintf( stderr, "libdvdcss debug: " ); \
> > > - fprintf( stderr, msg, ##args ); \
> > > + fprintf( stderr, __VA_ARGS__ ); \
> > > fprintf( stderr, "\n" ); \
> >
> > Probably nobody cares, but this most likely won't compile anymore on oldish MSVC versions.
>
> Quite possibly, yes. The current MSVC version is the first with sensible
> C99 support, so it feels like a good version to require :)
>
> j-b, do we care about old MSVC?
No.
With my kindest regards,
--
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device
More information about the libdvdcss-devel
mailing list