Luca Olivetti wrote: > > well, it doesn't harm (it was in Martin's source withot me realizing it) > but it works well even without it. I also set the palette to 16 colors > now (not that it changes much, but...). > > Bye > > ------------------------------------------------------------------------ > > --- dxr3interface_spu_encoder.c.orig 2005-04-04 17:53:35.966868390 +0200 > +++ dxr3interface_spu_encoder.c 2005-04-04 17:57:36.107254910 +0200 > @@ -270,6 +270,8 @@ > > // set active area to 0 > //m_x0 = m_x1 = m_y0 = m_y1 = 0; > + //16 Colors max. > + m_palManager.SetBpp(4); You're right, that's the way it should be done. Another thing I made wrong is "dxr3tools.h" only gets included, if one sets -DUSE_XINE_SCALER in eg. the Makefile, otherwise not. Thus, the include-statement is at the wrong place (I don't think anyone wants to use the old scaling-algorithm, but you never know...) : ---- --- dxr3interface_spu_encoder.c 2005-04-04 18:45:09.526439552 +0200 +++ dxr3interface_spu_encoder.c.orig 2005-04-04 18:44:16.120558480 +0200 @@ -28,7 +28,7 @@ #include "dxr3interface_spu_encoder.h" #include "dxr3memcpy.h" -#include "dxr3tools.h" + /* ToDo: @@ -47,6 +47,7 @@ #include <signal.h> #include <string> +#include "dxr3tools.h" #include <vdr/plugin.h> namespace XineScaler
