[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.


> 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.

Thanks,
Jason.


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

Reply via email to