Sequencer: switching to preview mode works fine again

Move function out of mathutils to: BLI_math_rotation --- single_axis_angle_to_mat3(mat3, axis, angle), copied out from mathutils, axis arg is a char 'X/Y/Z' rather then a vector like axis_angle_to_mat3().

/blender/editors: Removed final points in UI strings and messages.
Plus a few cuts in very long lines…

remove Buffer.list, deprecated before 2.59 release.

Curve back/front fill changes:

- Use enum instead of back/front flags combinations.
- This flags behaves differently for 2d/3d curves so use different enums for them.
- This commit shouldn't change existing files.

Fix for add hook/vertex parent for meshes

The only way to achieve this is to re-load editMesh.
After this commit you can't iterate through vertices and insert
hooks/parents from script anymore -- this operators are now
treated as topology-changing and shouldn't be used when iterating
via geometries.

replace WORDS_BIGENDIAN with __BIG_ENDIAN__

cleanup endian handling
- define __BIG_ENDIAN__ or __LITTLE_ENDIAN__ with cmake & scons.
- ENDIAN_ORDER is now a define rather than a global short.
- replace checks like this with single ifdef: #if defined(__sgi) || defined (__sparc) || defined (__sparc__) || defined (__PPC__) || defined (__ppc__) || defined (__hppa__) || defined (__BIG_ENDIAN__)
- remove BKE_endian.h which isn't used

remove WITH_* defines from image formats, instead just dont build the files at all.

update glew from 1.6 to 1.7:

from their site:
  adds support for OpenGL 4.2, new extensions, fixes bugs

remove ChangeLog, its missing from the 1.7 bundle, also remove html doc dir.

add back GPL2+ header to paint_utils.c, copied from editface.c since some of its functions trace back to that file from 2.4x. (imapaint_tri_weights for eg).

add old style nan GPL2+ header to sound.c since this file is from rev2 and should have had this header added.

patch [#28684] Image pack/unpack() implementation.
from Bill Currie (taniwha)

replace strnlen with BLI_strnlen & some style changes.

fix for crash loading a file saved with fluidsim when blenders compiled with it disabled.
also remove unneed class prefix on function name for itasc.

blf - further shrink drawing functions & some style changes.

blf code - no functional changes.
- remove saniy checks from blf_font.c, the callers now check instead.
- move duplicate code into defines (may move into static functions).
- move kerning checks into const values set at the start of the function, rather then checking on every character.

Fix for [#28672] Blender segfault after exiting a game that was opened with autoplay on (Blender, not blenderplayer)

Autoplay misses uninitialisation, I'm just fixing the reported crash, it still misses all other frees, but as the program exits, we don't care?

Minor: Other UI strings typos and tweaks.

Console Scrolling - reset while typing.
patch from Damir Prebeg with some edits.

Also made it so resizing the console view keeps the lower part of the text in view (could be annoying when you needed to scroll because of a resized view).

image button was hard coded to draw the splash screen which it loaded from PNG data on every draw.

now pass the ImBuf when callign the image button so we could have different images in buttons later on.

Recast/detour: fix some property enum identifiers to follow conventions.

Nodes: fix nodes in groups not getting socket templates verified, and reserve
some socket flags for cycles to avoid conflicts.

Speedup for font drawing, every letter was character was calling:
  glGetIntegerv(GL_TEXTURE_2D_BINDING_EXT, &cur_tex);

... with shadow enabled glGetFloatv(GL_CURRENT_COLOR, color) was called twice per character as well.

Now only call glGetFloatv(GL_CURRENT_COLOR, ...) once per string and only when drawing with shadow or blur, texture bind is stored in the font.

Gives 8% overall FPS speedup when displaying heavy UI in my test.

Minor: fixing other UI typos.

use macros RAD2DEG & DEG2RAD rather then multiplying by 180.0/M_PI or M_PI/180.0

Minor: fixing an UI typo.

use const and array size in function definitions, no functional change.

correction to recent commit, wouldnt have given any troubles but was assigning the 4th component of a float[3].

more mini optimizations - don't call UI_ThemeColor 4 times per curve handle, instead get all colors at the start and index them when drawing curves in editmode.

also remove redundant NULL check.

micro optimization for circle drawing.
- use vertex array for drawcircball()
- add circball_array_fill() and call from drawcircball().
- for object center's rather than drawing 2 circles, create the array and reuse it.

use less confusing array syntax for circle drawing, no functional changes.

BGE Animations: Fixing some refcount issues with KX_Scene::m_animatedlist (fixes m_animatedlist crashes) and some whitespace issues with KX_GameObject::GetActionManager().

BGE fix: Font Objects not showing up in the dome mode
geez, who coded the font object? or even worse, who did the dome code?
Don't coders talk? tsc tsc ...

Now seriously, KX_KetsjiEngine::RenderFonts() could be moved to inside the KX_Scene class. It probably should (so I could call it from inside KX_Dome::RenderDomeFrame()). Not critical, so not changing it for now.

SVN maintenance.

WeightVG: added WeightVG icon for outliner (don’t know when that where lost...).

- fix for memory leak in findFreeNavPolyIndex()
- also correct own script for running cppcheck.

Fix #28663: All "unit" properties show a value of 0 (on WinXP&MinGW&scons)

Initially problem was caused by updated version of mingw-runtime which
changed behavior of snprintf and vsnprintf so %lf isn't anymore valid
for doubles.

According to manpages, %f is a correct format for snprintf for doubles.

correction for my fix for [#28668], would crash when there were no editbones.

use replace 0 with NULL for pointers, set some functions static
also fixed own errors in recent static check commit.

new convenience makefile targets for static source code cheching: check_splint/check_sparse/check_cppcheck

replace macros with math lib functions

- fix for access past the buffer size (paint / sculpt used some 2d vecs as 3d)
- remove redundant NULL checks on old code where it would crash if the result was NULL later on.
- add some missing NULL checks.

- remove deprecated pose channel members
- change short -> char for flags that support it.
- add pose 'temp' pointer to use for outliner drawing (was using 'prev' and restoring which seems dodjy)

fix [#28668] Crashes entering edit mode on Armature

WeightVG: Made Edit and Proximity also use the new weightvg_update_vg MDeformWeight** parameter (to avoid another vgroup searching).
Also added to Proximity a check in case vgroup would have no vertices in it.
Plus a few minor edits...

SVN maintenance.

- include enum names and descriptions in sphinx generated documentation
- add descriptions for operator bl_options

WeightVG utils, weightvg_update_vg func updates.
* Added an optional array of MDeformModifier pointers, to avoid another search based on defgrp_idx.
* Split out "add/remove verts from vgroup" functions, preparing their move to deform.c (if their current form is validated!).

Use static context trick for all platforms.

Should be safe until modifier stack is not threaded.
Solves issues with mingw and older glibc version (like used in release environment).

WeightVG Mix modifier: updated code to use defgrp_find_index, and make MDeformWeights be searched only once.
Also fixed a bug: when another set mode than "All Vertices" was used and resulting set of verts was empty, all vertices was used, instead of just returning org, unmodified data!

- Whitespace fixes (was commiting from windows where text editor wasn't configured, pardon)
- Fixing typo in description of GP paint mode.

replace BLI_strncpy with BLI_strncpy_utf8 where input isnt ensured to be valid.
also replace strcpy's which copy using "" with str[0]='\0'

fix [#28658] python can assign non utf8 and crash because of string lenth limits.

add BLI_strncpy_utf8() which which ensures there are no partially copied UTF8 characters, limited by the buffer size.

Update build rules to deal with new gettext libraries.

Fix [#28654] Warp modifier does not support negative strength when Vertex Group is used.
The vg weight was multiplied by org strength (i.e. neg strength was always skiping all verts!), now multiplying it with abs value of strength.

New C/Py api utility function PyC_Err_Format_Prefix() which raises an error with the existing error as a suffix.

Use this to raise errors when assigning a string property fails even though the value to assign *is* a string.

Before:
  TypeError: bpy_struct: item.attr= val: Object.name expected a string type, not str

After:
  TypeError: bpy_struct: item.attr= val: Object.name error assigning string, UnicodeEncodeError('utf-8' codec can't encode character '\udce9' in position 23: surrogates not allowed)

move utf8 string.c functions into their own file, also add python tip for printing operators.

minor edits to ascii draw function, unused var warning.

move ED_object_pose_armature --> object_pose_armature_get to so we dont get bad level calls in the weight paint branch.

removed nodes from CMake's BLENDER_LINK_LIBS, rewrote find-pcre using own template.

- removed some duplicate library links from cmake (which were needed because of bad level calls)
- FindXML2 we had copied from another project was always running and not using cached value, rewrote based on template used for most of our other find modules which makes use of 'FindPackageHandleStandardArgs'
- mark statuc collada libs as advanced.

resolve bad level calls from blenkenel/ into editors/ & remove editors from the include path from CMake & SCons.

* ED_curve_editnurbs --> curve_editnurbs
* ED_sculpt_modifiers_changed --> object_sculpt_modifiers_changed

fix compilation for MinGW by substituting qsort_r with qsort. What aversion do MinGW guys have for including '_r' variants of functions anyway?
Warning: a clean build will be needed probably to account for recent merge changes, or link errors will occur.

Fix typo in own commit for raycast library

Enable FFTW3 library for buildslaves

fix [#28635] Mirror Modifier - Clipping still active when modifier is disabled

Fixes #28599: Wrong re-assigning of layers in RenderLayer comp nodes when deleting a render layer.
Also added the check of comp nodetree of all scenes, as others might also use that scene in their compositing!

correct bad maximum value, for wile-loop node, outside the range of a short.

fix for opencollada crash with r40164, missing NULL check.

fix for error linking opencollada on linux.

Fix [#28614] Collada Exporter does not export Ambient term
reported by Steiner Bernhard

ma->ambX is calculated only on a render, so instead of relying on those values compute them manually.

use vector size and const args where possible (no functional change)

Code shuffle so function has only one exit point.

Fix for bug #28332: Smoke Simulation rendering artifacts.

Volume pre-caching altered shared data simultaneously in multiple threads, causing invalid scattering results when "Asymmetry" value was used. The view vector is now passed as a function argument.

Fix #28613: SEGFAULT: When setting Best-Quality on a Bump Map in GLSL viewport

Problem was caused by CRLF line ending instead of LF line ending.
Re-generate pu_shader_material to use LF line ending and made gpu codegen
treat \r as space character.

fix for changing font sizes with recent utf8 speedup

Fixed "bug" #28611: Subdiv value for icosphere should not be below 1.

Recast: attempt to fix compilation problem in Linux.

- fix for mesh_get_mapped_verts_nors copying short normals to floats without scaling.
- BL_SkinDeformer also did this though for that case its not a problem because the normals are later accumulated anyway.

replace VECCOPY -> copy_v3_v3, added copy_v*_v*_short too for typesafe copying, some parts of the code are copying float -> short normals without scaling. fix coming next.

cleanup for object color theme (no functional changes), could set the opengl color 2-3 times per object.

quiet -Wundef warnings for cmake/gcc/elbeem

fix for passing NULL to strstr() in visualkey_can_use()

fix for ntreeGPUMaterialNodes(), using uninitialized exec pointer.
also commented some set-but-unused variables.

== Sequencer ==

Fixed silly bug, that activated proxies at the wrong time.

pep8 edits

Found other problems similar to bug #28619 in other primitive_add ops… Will continue to chek!

Fixed bug #28619: values of X/Y grid size were allowed to be below 3, when adding a grid primitive.

Fixes for Win+SCons and disable recastnavigationif game engine is disabled in SCons

Recast: fix bad level calls, Recast compiled out if BGE not enabled. SCons updated but not tested.

fix for 'blender -E' crashing.

Audio:
* Fix for high quality upsampling which was wrong.
* Fix for doppler effects which were calculated wrong for scenes.
* Improved animation evaluation at the beginning and end of a scene.

move maxseq define into DNA.

speedup for editmesh drawing.
- avoid needless context switching quad/tri, flat/smooth.
- dont call glNormal3vf() lighting is disabled.

gives ~2x speedup with a subdivided cube, but thats probably the best case, quad/tri smooth/flat mix will slow down a bit.

Game Engine:
Fixed some more abbreviations in the UI.

partial fix for blenderplayer linking problems (decast retour)
This fix the problem of linking the extern components for the KX module.
The remaining linking issues are related to actual bad level access so the fix is a bit more complex.

Remaining linking issues (osx cmake gcc)
###########
Undefined symbols:
  "_buildNavMeshDataByDerivedMesh", referenced from:
      KX_NavMeshObject::BuildVertIndArrays(float*&, int&, unsigned short*&, int&, unsigned short*&, float*&, int&, unsigned short*&, int&, int&)in libge_logic_ketsji.a(KX_NavMeshObject.cpp.o)
      _applyModifier in libbf_modifiers.a(MOD_navmesh.cpp.o)
  "_polyIsConvex", referenced from:
      _applyModifier in libbf_modifiers.a(MOD_navmesh.cpp.o)
  "_polyNumVerts", referenced from:
      KX_NavMeshObject::BuildVertIndArrays(float*&, int&, unsigned short*&, int&, unsigned short*&, float*&, int&, unsigned short*&, int&, int&)in libge_logic_ketsji.a(KX_NavMeshObject.cpp.o)
  "_polyFindVertex", referenced from:
      KX_NavMeshObject::BuildVertIndArrays(float*&, int&, unsigned short*&, int&, unsigned short*&, float*&, int&, unsigned short*&, int&, int&)in libge_logic_ketsji.a(KX_NavMeshObject.cpp.o)
      KX_NavMeshObject::BuildVertIndArrays(float*&, int&, unsigned short*&, int&, unsigned short*&, float*&, int&, unsigned short*&, int&, int&)in libge_logic_ketsji.a(KX_NavMeshObject.cpp.o)
      KX_NavMeshObject::BuildVertIndArrays(float*&, int&, unsigned short*&, int&, unsigned short*&, float*&, int&, unsigned short*&, int&, int&)in libge_logic_ketsji.a(KX_NavMeshObject.cpp.o)
############

speedup font drawing:
 for ascii characters in a utf8 string use glyph_ascii_table lookup rather than call blf_glyph_search(), otherwise fallback to blf_utf8_next() and blf_glyph_search().

fix for error in own commit r40108.

replace magic numbers for flags for uiSetRoundBox();