Commit Graph

31994 Commits

Author SHA1 Message Date
Campbell Barton
4e0492e3bc fix compiler error 2012-05-28 21:25:17 +00:00
Nicholas Bishop
3d98da9b0f Fix for bug [#31613] Cycles 3d viewport material display mode + skin modifier related crash
When in material display mode, mesh_calc_modifiers() calculates the
orco DerivedMesh, which uses a different CustomDataMask. In
particular, it does not necessarily include the current modifier's
requiredDataMask, so those layers might get set to NO_COPY. For the
skin modifier, this resulted in a crash when the modifier internally
copies the DerivedMesh and the output does not contain the expected
MVertSkin layer.

Fixed by adding the requiredDataMask to the orco DM's CustomDataMask.

Also added a debugging function to customdata.c:
customData_mask_layers__print(CustomDataMask mask);

This will print out the names of all the CD layer types in the mask.
2012-05-28 21:02:44 +00:00
Campbell Barton
4f90d757c7 add BLI_ghash_pop() which returns the value for a key and removes it at the same time, saves a lookup if you need to check if the item exists before removing. 2012-05-28 19:33:14 +00:00
Lukas Toenne
c2be2fd408 Fix 2 #29846. Effector point ave variable can be NULL. 2012-05-28 19:10:57 +00:00
Dalai Felinto
9fb4ca981e image.gl_touch: wasn't passing the parameters to gl_load 2012-05-28 18:32:18 +00:00
Campbell Barton
47826118d1 style cleanup: also fix for TIMEIT macro 2012-05-28 18:12:08 +00:00
Dalai Felinto
481658867e image.gl_load: moving error cleaning to before all gl calls" 2012-05-28 17:47:42 +00:00
Dalai Felinto
6323b32c08 image.gl_load(): clear glGetError buffer before getting a new one 2012-05-28 17:03:13 +00:00
Lukas Toenne
666667bd83 Fix #29846, Dynamic rotation of particles not affected by forcefields 2.61 64 bit. Patch by Arno Mayrhofer (azrael3000).
Minor fix by me to initialize the effector point angular velocity vector correctly.
2012-05-28 16:58:12 +00:00
Campbell Barton
e9d6b4db96 fix for own recent commit removing NULL checks from TREESTORE macro - this use needed it. 2012-05-28 15:37:43 +00:00
Thomas Dinges
d02b6707f9 * Some code optimizations to panel collapse code, to avoid calculations for each panel. Suggested by "Bao 2". 2012-05-28 14:57:38 +00:00
Lukas Toenne
3091c9d440 Fix for node linking operator, which could insert node link into the list twice and cause deadlock due to incomplete test. 2012-05-28 14:39:55 +00:00
Daniel Genrich
ee854f581a Smoke: Fix messed up shadow display with OpenMP. This needs fixing. For now disable openmp for that function. Shadow calculation would be better raytraced on GPU or using shaders for the future.
Part of my Blender Smoke Development Phase III.

(accidently commited this to smoke2 branch first)
2012-05-28 14:31:08 +00:00
Bastien Montagne
fd2f05be44 A few minor UI message fixes... 2012-05-28 14:01:42 +00:00
Sergey Sharybin
9ed75681b6 Sequencer: replace clamp+round of multiplied color value with FTOCHAR macro,
Should solve issues with building blender in release environment, should be
no functional changes.
2012-05-28 10:39:22 +00:00
Sergey Sharybin
e442adf9cf Fixed crash opening files with missed libraries. 2012-05-28 10:32:38 +00:00
Bastien Montagne
3a190396dd Fix [#31621] Face Angles Drawn in Strange Places
When do_global is True, don’t compute face angles/area positions based on ob-matrix-transformed coordinates!
2012-05-28 09:01:43 +00:00
Campbell Barton
ee15754e31 move mouse select options into their own property function - more operators should use these eventually 2012-05-28 05:45:19 +00:00
Thomas Dinges
492d7ace57 User Interface Panels:
* Add a solo mode for panels, CTRL+LMB on a panel header.
This closes all other panels.

Patch [#22233] by sneg negr (sneg) with modifications by myself to exclude panels with hidden header (like context)
2012-05-28 00:02:24 +00:00
Campbell Barton
46f1671c07 missing from previous commit 2012-05-27 21:56:19 +00:00
Peter Schlaile
4474a90d8b == Sequencer / Color balance ==
Make the color balance inner working more accurate in the byte -> byte
case, probably also fixing (can't check I'm on 32bit) the bug mentioned 
on yellow's blog:

http://blendervse.wordpress.com/2012/04/02/waiving-the-fullrange-flag/
2012-05-27 20:57:24 +00:00
Campbell Barton
032d83ecc4 style cleanup: defines with braces 2012-05-27 20:13:59 +00:00
Gaia Clary
b65c9711fe Collada exporter: Added name attribute to nodes in visual_scene elements (name = id) 2012-05-27 19:44:07 +00:00
Campbell Barton
b33f0ef0e3 style cleanup 2012-05-27 19:40:36 +00:00
Campbell Barton
295aa880e9 style cleanup: draw node 2012-05-27 19:38:10 +00:00
Daniel Genrich
1e13168183 Smoke:
a) Another boundary fix. Resulted in smoke getting "sucked" back into the domain
b) Disabling substeps (internal thing). Fixes arbitrary explosions/instabilities.

Part of my Blender Smoke Development.
2012-05-27 18:45:16 +00:00
Campbell Barton
d4b6927179 remove NULL check in TREESTORE macro, the return NULL value wasny checked for by any callers (ie - it would crash later if the arg was NULL anyway)
also comment on the speed of check_persistent()
2012-05-27 14:43:18 +00:00
Sergey Sharybin
8c51ecaf90 Use BKE_movieclip_get_size rather than BKE_movieclip_get_ibuf and use ibuf's resolution in tile's Movie Clip node 2012-05-27 14:30:45 +00:00
Sergey Sharybin
2a7b4adedc Fix mistmatch in usage of delete vs. delete [] in tiles reported by valgrind profiler. 2012-05-27 14:17:34 +00:00
Sergey Sharybin
a1091fbfa6 Fix #26876: Soft Body ignores Goal on Lattice
Patch by Arno Mayrhofer (aka azrael3000) with only slight code style modifications, thanks!
2012-05-27 13:33:09 +00:00
Campbell Barton
1e45042396 code cleanup: remove mode comments, these were applied very un-evenly across a few files. 2012-05-27 13:22:43 +00:00
Dan Eicher
bc4f98b16f rna_def_composite_nodetree_api() getting passed the wrong prop 2012-05-27 13:17:31 +00:00
Campbell Barton
2ab62ce126 code cleanup: defines with braces - end with '(void)0' so callers must end with ';' like normal function.
... without this some editors dont parse the source so well.
2012-05-27 12:21:13 +00:00
Campbell Barton
aa63016744 when SCREEN_OT_keyframe_jump fails dont send notifiers or call sound_seek_scene 2012-05-27 11:22:06 +00:00
Campbell Barton
405f6a7c77 code cleanup:
style and remove BM_face_find_longest/shortest_edge functions,
... instead use BM_face_find_longest/shortest_loop()->e
2012-05-27 11:10:27 +00:00
Campbell Barton
5b483932c7 style cleanup: nodes, also rename chunksize -> chunk_size 2012-05-27 11:08:47 +00:00
Sergey Sharybin
654ef111a7 Fix #31606: pg up/down does not work for changing falloff for proportional editing
Initially issue was caused by non-proportional edge slide patch in rev46927 which
had several wrong things:

- It introduced new events TFM_WHEEL_DOWN_EVT/TFM_WHEEL_UP_EVT which were defined
  as per-bit OR between edge slide and proportional editing modal events, but
  the issue is that modal events are not bit masks and that meant that new events
  were defined as 20|24 and 21|25 which is a bit strange.
- Another this was caused by the fact, that keymap wasn't creating for some specific
  transform modal keymap item -- it used to be tried to create single shortcut for
  two different modal events, which isn't supported by keymaps and resulted by
  artifacts in keymap editor interface (nodal event type was empty for wheel movements).
  That was caused by different events used by modal keymap and defining shortcuts.
- Actual issue which prevented PgUP/PGDOWN to work was mixing using that new events
  for wheel scrolling in switch statements which doesn't do bitmask check.

Solved by separating keymaps used for proportional editing and edge slide -- now
it's mouse Wheel for proportional editing and Alt-Wheel for edge slide. If someone
would want to have single shortcut for this two events it'll imply one of the follofing
things:

- Support poll function for shortcuts, so keymap would definitely know which
  modal event to send to operator.
- Generalize TFM_MODAL_PROPSIZE_{UP, DOWN} and TFM_MODAL_EDGESLIDE_{UP, DOWN} so
  they can share the same shortcut with current design of keymaps.
2012-05-27 10:08:37 +00:00
Campbell Barton
857dedbc58 style cleanup 2012-05-27 00:36:50 +00:00
Gaia Clary
822362189b [#31607] Collada: (Exporter) Implementation of 'use Object Instantiation' Option 2012-05-26 22:46:28 +00:00
Campbell Barton
54b64cfd61 code cleanup: use const float and define array size 2012-05-26 22:21:56 +00:00
Nicholas Bishop
ebdeed07e5 Delete skin customdata if all skin modifiers are removed.
Fix for bug [#31604] "Skin Modifier - Mark Root Bug?"

Essentially the same as multires customdata deletion.
2012-05-26 21:11:23 +00:00
Sergey Sharybin
ce9ee99c07 Fix #31538: Wrong colors after reloading file with exr image
Issue was caused by generated images had got no profile set (IB_PROFILE_NONE)
which confused OpenEXR saver (which makes sRGB conversion for profiles which
are not linear).

Actually this is much deeper issue which would require the whole color pipeline
workflow (which would happen at some point when currently urgent stuff is resolved),
but having correct profile set for generated images would still be helpful.
2012-05-26 17:04:51 +00:00
Campbell Barton
de6af75d4d code cleanup: use array size and const for vector args 2012-05-26 16:04:31 +00:00
Bastien Montagne
ae919658d4 Add frame_duration read-only member to Image RNA struct, returns movie's length in frames (or 1 for still images). 2012-05-26 14:53:33 +00:00
Campbell Barton
ed98de3398 code cleanup: dutch -> english (comments) 2012-05-26 13:31:57 +00:00
Campbell Barton
ba5e13912d fix for own error in applying patch 2012-05-26 13:08:42 +00:00
Campbell Barton
63788b47d6 add vector versions of hsv_to_rgb, rgb_to_hsv & rgb_to_hsv_compat 2012-05-26 11:01:01 +00:00
Campbell Barton
e727056c2e patch [#31265] Warning Fixes 02/05/2012
from Jason Wilkins (jwilkins) 

only applied some parts:
* const correctness
* moved a variable into a move local scope so it is also inside a #if/endif and does not end up conditionally unused
2012-05-26 10:05:22 +00:00
Sergey Sharybin
3cb166e0f2 Fix reading freed memory when opening file by click on it on splash screen
Issue was introduced by yesterday's commit 47021 and caused by some handler's
which is getting called from ui_handler_popup frees event.

Worked around a bit by storing return value for ui_handler_popup before running
other handlers, but this only means global refactor of even handling order is
getting closer and closer.
2012-05-26 09:49:54 +00:00
Campbell Barton
c895c0ee23 patch [#31579] Solidify modifier: Add new option "Flip Normals"
- previous commit was so similar to the patch Shinsuke wrote, better to add him as contributor, this commit will add to generated credits.
- this commit has no functional changes.
2012-05-26 09:39:04 +00:00