Blender Git Loki

OSX/Cmake: workaround for missing OpenGL headers, make sure sdk lookup

Fix IRC reported by elubie: windows crash with audio animation

There was a bug in how the iterators of STL list was used when erasing during iteration of a list, which was triggered by the STL implementation of MSVC, but hid well with gcc.

Fix for addons submodule which was pointing to a not existing revision

Pretty much likely happened because of arc utility included this
changes by accident and there were local commits in a local tree.

Not sure about how to prevent this from happening yet, just be
careful for now.

Blender Internal:
* Remove "Edge" post processing effect and the corresponding render layer.

Since we have freestyle, this is not needed anymore and was a very simple effect anyway (Zbuffer filter effect, could be added to the compositor if really needed again).

Reviewed By: brecht, ton
Differential Revision: https://developer.blender.org/D14

Code Cleanup: style, spelling and pep8 edits

Extra unwanted change from the previous commit

Buildinfo fixes

- Use -M suffix if working tree does have uncommitted
modifications.
- Local commits are considered local changes as well

Code Cleanup: move trans-verts out of snap code into their own file.

Developer Note:
* minmax & centroid was being calculated when creating trans-verts but never used.
* static vars removed, replace with TransVertStore stack var.

Fix: tab completing a filepath name in file browsers asks to create a new
directory if name was not fully matched

When hitting tab to complete a directory name in the filepath field in the
filebrowser Blender shows a "create new directory?" popup, if the beginning
of directory name typed in the field matches many entries. For example if you
have directories in the open directory called "test123" and "test456", typing
"te", tab would complete up to "test", but ask to create a new folder with the
name "test".

This patch unsets the boolean storing the info about changing filepath if the
folder with the completed name does not exist.

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D10

Fix: Game Engine regression drawing text from recent cleanup

Buildbot: update configuration file

Consider this configuration final-1 for the migration,
Linux and OSX slaves seems to be fine, Windows slave
would need some changes from the slave side.

Remove Surface modifier when removing force field from object

Summary:
Before this adding Surface type of force field and removing
this field would leave Surface modifier alive in the stack.

This might be really misleading and annoying.

Now removing force field will ensure no modifiers needed for
it are remained in the stack.

This also fixes missing notifier to redraw modifier stack
when changing force field type.

Reviewers: brecht, campbellbarton

Reviewed By: brecht

Differential Revision: https://developer.blender.org/D13

Cleanup: Renamed compositor executePixel functions and their 'read' wrappers in SocketReader.

Distinguish the 3 different methods for acquiring pixel color values (executePixel, executePixelSampled, executePixelFiltered).
This makes it easier to keep track of the different sampling methods (and works nicer with IDEs that do code parsing).

Differential Revision: https://developer.blender.org/D7

Buildbot: fix path got pack step

Fix T37535: Hebrew charset was missing two chars.

Buildbot: tweaks for source directory

Need this so checkout of existing svn does not
interferes with new git checkout which would
need to be done.

Initial tweaks to buildbot configuration

Pretty much sure more tweaks would be needed,
but need this to get started.

Potential fix for T37525: Viewer node causes crash

LOCK_DRAW_IMAGE used to be unlocked twice. Now made it
so all the thread-unsafe code is inside a single lock/
unlock section.

View3D: Add menu in the header is now context sensitive (like shift+a).

Fix T37399: Crash with wireframe (wasn't clearing edge tag)