diff options
author | Kévin Dietrich <kevin.dietrich@mailoo.org> | 2015-11-22 20:17:09 +0300 |
---|---|---|
committer | Kévin Dietrich <kevin.dietrich@mailoo.org> | 2015-11-22 20:18:43 +0300 |
commit | 3e1a9ee386b054912b540b7a061d76235f4049fa (patch) | |
tree | 4109341018601a6d9f70dfc2e32ffe4177e4bbce /release/scripts/startup/bl_ui | |
parent | 8623d75b468ff1a495a326cf74ca6d61f1ee4ce7 (diff) |
Smoke drawing code cleanups (and little refactor)
This patch contains the following changes:
- the vertices and edges arrays would be assigned default values, and
then reassigned new ones right away. It appears that those arrays were
once global and then made local
(rB06a2ee4afed4237398b69ddf253e29a730b2f9f0), so it makes sense now to
initialize them with the right values.
- the flame spectrum texture was created whether it was needed or not,
so now it's only created if there's flame to be drawn, also split the
code in a separate function.
- reduce the number of parameters to the main draw function, as most of
them are member of SmokeDomainSettings.
- some other minor cleanups: fold multiple operations into one to get
rid of one local variable, mark variables as `const` when necessary,
unecessary gl draw calls, reorder the code a bit...
Reviewers: campbellbarton, psy-fi
Differential Revision: https://developer.blender.org/D1368
Diffstat (limited to 'release/scripts/startup/bl_ui')
0 files changed, 0 insertions, 0 deletions