Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(1418)

Issue 6067044: GLSL Routines for Blender BGL wrapper (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by dfelinto
Modified:
12 years ago
Reviewers:
ideasman42, bf-codereview
Base URL:
https://svn.blender.org/svnroot/bf-blender/trunk/blender/
Visibility:
Public.

Description

This patch adds the bare minimum necessary functions needed to use GLSL with bgl. There are other flags that could make into it, but I'm yet to list them all. Please note that glSheaderSource was implemented manually to account for the need of char ** in the original code (which our macros couldn't handle). I followed what PyGL does, instead of a function with 4 arguments, the wrapper takes only 2, so the user can create only one at a time, what I find reasonable. Images: [1] - http://www.dalaifelinto.com/ftp/bgl_glsl/glsl_3dview.jpg [2] - http://www.dalaifelinto.com/ftp/bgl_glsl/glsl_3dview2.jpg [3] - http://www.dalaifelinto.com/ftp/bgl_glsl/glsl_3dview3.jpg Test python script: [4] - http://www.dalaifelinto.com/ftp/bgl_glsl/bgl_glsl.zip - commited with changeson rev. 46444

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+134 lines, -1 line) Patch
source/blender/python/generic/bgl.h View 2 chunks +11 lines, -1 line 0 comments Download
source/blender/python/generic/bgl.c View 19 chunks +123 lines, -0 lines 0 comments Download

Messages

Total messages: 3
dfelinto
12 years, 1 month ago (2012-04-19 06:43:46 UTC) #1
dfelinto
I talked with Mitchell and realized that there will always be something else we want ...
12 years ago (2012-04-26 14:32:12 UTC) #2
dfelinto
12 years ago (2012-05-09 13:53:50 UTC) #3
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b