[Bf-committers] [Bf-blender-cvs] SVN commit: /data/svn/bf-blender [53146] trunk/blender/source/blender/ makesrna/intern/rna_scene.c: BGE: Removing the VBO raster_storage option from the UI.

[Dalai Felinto]

Hi Mitchell,
First of all, congrats on the gsoc  commits.

Regarding the vbo part of it: did you run any benchmark in memory usage?

If I recall it right, vbo are supposed to be slower but to have a smaller
memory footprint. I can foresee them beating DL only in scene with massive
numbers of skinned objects (because technically they would be competing
again VA here).

I had an old test file sent to me back in the original VBO patch days. I
will try yo find it.

--
Dalai

On Dec 18, 2012 7:19 PM, "Mitchell Stokes" <mogurijin at gmail.com> wrote:
>
> Revision: 53146
>
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=53146
> Author:   moguri
> Date:     2012-12-18 21:19:15 +0000 (Tue, 18 Dec 2012)
> Log Message:
> -----------
> BGE: Removing the VBO raster_storage option from the UI.
>
> Modified Paths:
> --------------
>     trunk/blender/source/blender/makesrna/intern/rna_scene.c
>
> Modified: trunk/blender/source/blender/makesrna/intern/rna_scene.c
> ===================================================================
> --- trunk/blender/source/blender/makesrna/intern/rna_scene.c
 2012-12-18 20:56:25 UTC (rev 53145)
> +++ trunk/blender/source/blender/makesrna/intern/rna_scene.c
 2012-12-18 21:19:15 UTC (rev 53146)
> @@ -2447,8 +2447,9 @@
>         static EnumPropertyItem storage_items[] ={
>                 {RAS_STORE_AUTO, "AUTO", 0, "Auto Select", "Chooses the
best supported mode"},
>                 {RAS_STORE_IMMEDIATE, "IMMEDIATE", 0, "Immediate Mode",
"Slowest performance, requires OpenGL (any version)"},
> -               {RAS_STORE_VA, "VERTEX_ARRAY", 0, "Vertex Arrays",
"Moderate performance, requires at least OpenGL 1.1"},
> -               {RAS_STORE_VBO, "VERTEX_BUFFER_OBJECT", 0, "Vertex Buffer
Objects", "Best performance, requires at least OpenGL 1.4"},
> +               {RAS_STORE_VA, "VERTEX_ARRAY", 0, "Vertex Arrays",
"Better performance, requires at least OpenGL 1.1"},
> +               /* VBOS are currently disabled since they cannot beat
vertex array with display lists in performance. 8?
> +               /*{RAS_STORE_VBO, "VERTEX_BUFFER_OBJECT", 0, "Vertex
Buffer Objects", "Best performance, requires at least OpenGL 1.4"},*/
>                 {0, NULL, 0, NULL, NULL}};
>
>         srna = RNA_def_struct(brna, "SceneGameData", NULL