[GLES] Port cube plugin to use GLVertexBuffer rather than GLWindow::Geometry

Bug #999017 reported by Sam Spilsbury
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Compiz
Fix Released
Medium
Compiz Linaro Team

Bug Description

Cube is currently using the old unsupported GLWindow::Geometry API - it should be ported to use GLVertexBuffer

Tags: gles
Changed in compiz-core:
status: New → Invalid
status: Invalid → Confirmed
importance: Undecided → Medium
assignee: nobody → Compiz Linaro Team (compiz-linaro-team)
milestone: none → 0.9.8.0
Changed in compiz-core:
milestone: 0.9.8.0 → gles-0.9.8
Changed in compiz:
assignee: nobody → Compiz Linaro Team (compiz-linaro-team)
importance: Undecided → Medium
status: New → Confirmed
Changed in compiz:
milestone: none → 0.9.8.0
summary: - Port cube plugin to use GLVertexBuffer rather than GLWindow::Geometry
+ [GLES] Port cube plugin to use GLVertexBuffer rather than
+ GLWindow::Geometry
tags: added: gles
no longer affects: compiz-core
Changed in compiz:
status: Confirmed → In Progress
Revision history for this message
Frederic Plourde (fredinfinite23) wrote :

Hi guys, I don't know if this could help, but before leaving the gles team with Linaro in March, I had come up with a nice working CUBE plugin under GLES. (without skydome, though).

So here's the quick patch I had then, for what it's worth.
Hope this helps !

cheers !

Revision history for this message
Sam Spilsbury (smspillaz) wrote :

Thanks fredric.

I have cube working in the gles2 branch as it stands, but it lacks skydome support (like this patch does). Were you able to get caps working using the normal GLVertexBuffer calls? I had to add some custom code there to make it work.

Revision history for this message
Frederic Plourde (fredinfinite23) wrote :

>> Thanks fredric.
>> I have cube working in the gles2 branch as it stands,

good :-)

>> but it lacks skydome support (like this patch does). Were you able to get caps working using the normal GLVertexBuffer calls?

same here !... no skydome support (because it involves more rewritting)
and no caps working (nor any attempt to make them work yet)

>> I had to add some custom code there to make it work.

Revision history for this message
Daniel van Vugt (vanvugt) wrote :

This appears to be done already.

Changed in compiz:
status: In Progress → Triaged
status: Triaged → Fix Committed
Changed in compiz:
milestone: 0.9.8.0 → none
Changed in compiz:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.