Hi;

On Mon, 2008-06-30 at 11:01 -0400, Jason Tackaberry wrote:
> [Resubmitting to list; I'm too used to lists that rewrite Reply-To :)]
> 
> On Mon, 2008-06-30 at 10:36 +0100, Matthew Allum wrote:
> > Ok, but if your concerned about this, your hooking into MPlayer in a
> > very inefficient way..
> 
> Presumably here you mean ClutterX11TexturePixmap specifically is
> inefficient?
> 
> ClutterGLXTexturePixmap seems to incur very little overhead indeed.
> Maybe this is true only with nvidia, or perhaps specifically where
> textured video is used.

I mean its going to be more efficient pushing frames from mplayer
directly into Clutter rather than going via the X server. 

> 
> 
> > Yes, that is because ClutterX11TexturePixmap updates the texture in a
> > slow but 'portable' manner. It 'manually' copies pixels from the X
> > drawable to the texture for damaged areas. Its really used as a fallback
> > for hardware that doesn't have the texture from pixmap extension.
> 
> Understood, thanks.
> 
> > Yep, see http://bugzilla.openedhand.com/show_bug.cgi?id=997 .If you
> > could test that patch there it would be helpful.
> 
> It breaks compilation:
> 
>         make[2]: *** [test-textures] Error 1
>         make[2]: *** Waiting for unfinished jobs....
>         gcc -pthread -I/usr/include/pango-1.0 -I/usr/include/cairo
>         -I/usr/include/glib-2.0 -I/usr/lib/glib-2.0/include
>         -I/usr/include/freetype2 -I/usr/include/libpng12
>         -I/usr/include/gtk-2.0 -g -O2 -g -pthread -Wl,--export-dynamic
>         -o .libs/test-events test-events.o  -lXfixes -lXdamage
>         -lXcomposite
>         -lXi /usr/lib/libGL.so /usr/lib/libpangocairo-1.0.so 
> /usr/lib/libpango-1.0.so /usr/lib/libcairo.so /usr/lib/libgthread-2.0.so -lrt 
> /usr/lib/libgdk_pixbuf_xlib-2.0.so /usr/lib/libgdk_pixbuf-2.0.so -lm 
> /usr/lib/libgobject-2.0.so /usr/lib/libgmodule-2.0.so -ldl 
> /usr/lib/libglib-2.0.so ../clutter/.libs/libclutter-glx-0.7.so  -Wl,--rpath 
> -Wl,/opt/clutter/svn-2008-06-30/lib
>         ../clutter/.libs/libclutter-glx-0.7.so: undefined reference to
>         `set_user_time'
>         collect2: ld returned 1 exit status
>         
> USE_XINPUT is set to 1 in config.h.  I didn't really spend much time
> looking into this compilation error.  I reverted clutter-event-x11.c and
> carried on.  I can look into it more closely if you wish.
> 
> As far as the new clutter_glx_texture_pixmap_new_with_window() API
> itself goes, it worked exactly as advertised.
> 

Try latest trunk, that patch is now integrated.

  == Matthew

-- 
To unsubscribe send a mail to [EMAIL PROTECTED]

Reply via email to