WM: add support for drag events

This allows for a single key to be mapped to both release and drag,
useful for pie menus to share a key with a different action.

Fix key repeat events resetting the click timer

Cleanup: trailing space for windowmanager

WM: check for release instead of not pressed

Makes reasoning about events more predictable.

Fix crash due to missing init of new bAnimContext bmain member in transform code.

From own previous G.main-busting commit.

Fix double free in dyntopo-sculpt mode undo

Fix sculpt assert on initialization

Cleanup: remove moar G.main usages.

Notes:
* Really need to address RNA setters case, end up adding way too much
G.main here these days... :/
* Added Main pointer into bAnimContext, helps a lot in anim code ;)

Revert "Cycles: Cleanup: Don't use return on function returning void"

Not sure why exactly it is called a cleanup, the code was much more clear
and robust against possible missing return statements which are MANDATORY.

Missing return statement will:

- Cause two different BVH traversals to be run.

  Not is happening currently, but if more BVH layouts are added, it will
  become a problem.

- It is already causing assert() statements to fail, since functions are
  no longer returning when they are supposed to.

If there is any measurable reason to keep this change, let me know.
Otherwise just stick to reliable/tested/robust code.

This reverts commit ba65f7093b39a8e5f1fb869cbc347fb810a05ab9.

Cleanup: Nuke moar G.main usages...

Python API: Initial 'imbuf' API

Support only basic operations new/load/write & resize.

Add now so we can extend as needed & more easily accept patches.

Fix BLI_ASSERT_UNIT macro w/ non-finite numbers

UI: NLA: Influence should be a factor (RNA)

Alembic export: only free duplilists when not NULL

Fix error using freed bmain

Regression in 481cdb08ed6f3

make.bat : move all experimental options to their own section in the help.

make.bat : bring the help up to date with recent additions to the build script.

make.bat : check for existence of ninja before using it.

make.bat: change the way the vs buildtools are detected.

The recent change also used the buildtools instead of the regular compiler, you now have to explicitly state what you want to use :

2017 - the standard msvc compiler
2017pre - the msvc compiler from the preview installation
2017b - the msvc compiler from the buildtools installation

Fix crash in owmn previous commit.

Cleanup: use new accessors to blendfile path (Main.name).

Cleanup: add hleper functions to get filepath from Main.

This helps making things clearer and cleaner. Func returning filepath of
G.main is separate, so that we can easily track its usages, and
hopefully deprecate it at some point. Though that usage of G.main is
likely the less evil one, you nearly always want current blendfile path
in those cases anyway.

Cleanup: pep8

Use 'autopep8 --ignore E721,E722' on our UI code, only minor changes.

make.bat: cache the vcredist directory.

When run from make.bat the environment is setup correctly and the VCToolsRedistDir environment variable exists, on later invocations of cmake this may no longer be the case and a warning was emitted about the missing runtime. we can't rely on InstallRequiredSystemLibraries.cmake here since it uses the compiler version to figure out the correct location and it doesn't know how to deal with clang.

make.bat : Fix clang+asan msbuild project generation

Cleanup: correct menu name

Cleanup: strip ghost trailing space

Cleanup: strip tests trailing space

Particle System: move data creation into RNA update

Relying on evaluation to initialize data causes issues w/ 2.8.

Fix T55260: load Text File with Python from GUI Button results in 0 User

Let's just always ensure user_one when loading text from operator...

Cleanup: strip trailing space in editors

 Cleanup: strip trailing space in GPU module

Cleanup: newlines at EOF for Python modules

Cleanup: strip trailing space in Python module

Cleanup: strip trailing space in bmesh module

Cycles: Cleanup: Don't use return on function returning void

Cleanup: correct variable name, doxy sections

Cleanup: add argument names to screen callbacks

Cleanup: remove blockscale & handler

Replace with link_flag, currently unused,
needed for dynamic space types which is planned.

build_environment: support for msvc2017 and newer cmake.

-expanded build_deps.cmd with 2017 support, it can't locate msvc2017 so needs to be run from developer prompt.
-Newer cmake was unhappy with openal's cmakelists.txt
-collada has warning as error on and errored out on new msvc2017 warnings.

build_environment: fix paths in osl.diff

--debug-gpu-shader: Dump GLSL shaders to disk

This is really convenient for development. Either for profiling the
generated shaders or to check if the generated code is correct.

It writes the shaders to the temporary blender session folder.

(ported over from blender2.8)

Cleanup: warning

Cleanup: trailing whitespace (comment blocks)

Strip unindented comment blocks - mainly headers to avoid conflicts.

Cleanup: some more G.main removal from editor code.

Add Asan support for clang on windows.

This will currently only work for the RelWithDebInfo configuration since asan
does not support the debug crt. for source line information in the reports,
you need a copy of llvm-symbolizer in the blender folder or set the
ASAN_SYMBOLIZER_PATH environment variable to point to it. Currently (as of
6.0.0) llvm-symbolizer does not ship with the binary clang/llvm distribution.

Reviewers: campbellbarton

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

Cleanup: Remove G.main from some editor files.

Cleanup: remove G.main from BKE mball code.

Cleanup: get rid of last G.main usages in BKE library code.

Cleanup: remove G.main from BKE modifier.

Cleanup: remove G.main from BKE object

Had to add some G.main to modifiers, but in 2.8 we do not need that
anymore, so it's not that bad! ;)

Cleanup: nuke G.main out of BKE PackedFile code.

Cleanup: use doxy sections for space types

Also use struct names in enum/define comments.

Cleanup: return types in stubs

Fix OpenCL compilation error - BPT without SSS.

make.bat: fix support for building with just the build tools installed.

Fix missing movieclip ID type in allowed ones for Outliner...

Edit Mode: add success return value

Also remove fix for T6614, since BKE_object_obdata_is_libdata
no longer checks proxy.

DNA: add OB_DATA_SUPPORT_EDITMODE macro

Cleanup: Get rid of G.main in BKE_material.

Note that in some cases, this only moves the G.main case to somne other
places - in particular, RNA getters/setters are becoming annoying here...

make.bat : fix release build not getting the right settings.

make.bat : improve error message while detecting the msvc runtime when using clang

make.bat : add support for building with ninja and clang together.

Windows: Add support for building with clang.

This commit contains the minimum to make clang build/work with blender, asan and ninja build support is forthcoming

Things to note:

1) Builds and runs, and is able to pass all tests (except for the freestyle_stroke_material.blend test which was broken at that time for all platforms by the looks of it)

2) It's slightly faster than msvc when using cycles. (time in seconds, on an i7-3370)

victor_cpu
msvc:3099.51
clang:2796.43

pavillon_barcelona_cpu
msvc:1872.05
clang:1827.72

koro_cpu
msvc:1097.58
clang:1006.51

fishy_cat_cpu
msvc:815.37
clang:722.2

classroom_cpu
msvc:1705.39
clang:1575.43

bmw27_cpu
msvc:552.38
clang:561.53

barbershop_interior_cpu
msvc:2134.93
clang:1922.33

3) clang on windows uses a drop in replacement for the Microsoft cl.exe (takes some of the Microsoft parameters, but not all, and takes some of the clang parameters but not all) and uses ms headers + libraries + linker, so you still need visual studio installed and will use our existing vc14 svn libs.

4) X64 only currently, X86 builds but crashes on startup.

5) Tested with llvm/clang 6.0.0

6) Requires visual studio integration, available at https://github.com/LazyDodo/llvm-vs2017-integration

7) The Microsoft compiler spawns a few copies of cl in parallel to get faster build times, clang doesn't, so the build time is 3-4x slower than with msvc.

8) No openmp support yet. Have not looked at this much, the binary distribution of clang doesn't seem to include it on windows.

9) No ASAN support yet, some of the sanitizers can be made to work, but it was decided to leave support out of this commit.

Reviewers: campbellbarton

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

Fix Cycles + OSL build error, pass main to node editing functions.

UI: center align number buttons w/o text

This makes supporting split properties and text possible, see T54951

Cleanup: Nuke most of remaining evil G.main from RNA.

The few ones in getters/setters we cannot remove as easily, for now we
can live with those I think...

Cycles: Fix problems in the IES loader when rendering with no file selected

UI: replace BLI_strncpy w/ memcpy

Size is already checked.

Recently added IES conflicts w/ EEVEE

3D View: minor change to NDOF view orbit

This change is needed for 2.8, where the NULL check isn't a reliable way
of testing if dynamic offset is needed.

Fix restrict error in BLI_str_format_byte_unit

Don't use sprintf to append a string to it's self.

Also correct BLI_str_rstrip_float_zero's return value.

make.bat : Fix builtime.txt being written in the wrong folder.

Cycles: Add Support for IES files as textures for light strength

This patch adds support for IES files, a file format that is commonly used to store the directional intensity distribution of light sources.
The new IES node is supposed to be plugged into the Strength input of the Emission node of the lamp.

Since people generating IES files do not really seem to care about the standard, the parser is flexible enough to accept all test files I have tried.
Some common weirdnesses are distributing values over multiple lines that should go into one line, using commas instead of spaces as delimiters and adding various useless stuff at the end of the file.

The user interface of the node is similar to the script node, the user can either select an internal Text or load a file.
Internally, IES files are handled similar to Image textures: They are stored in slots by the LightManager and each unique IES is assigned to one slot.

The local coordinate system of the lamp is used, so that the direction of the light can be changed. For UI reasons, it's usually best to add an area light,
rotate it and then change its type, since especially the point light does not immediately show its local coordinate system in the viewport.

Reviewers: #cycles, dingto, sergey, brecht

Reviewed By: #cycles, dingto, brecht

Subscribers: OgDEV, crazyrobinhood, secundar, cardboard, pisuke, intrah, swerner, micah_denn, harvester, gottfried, disnel, campbellbarton, duarteframos, Lapineige, brecht, juicyfruit, dingto, marek, rickyblender, bliblubli, lockal, sergey

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

Threads: add spinlock hit for hyperthreading processors on Windows.

Suggested by Percy Ross Tiglao.

make.bat: Add support for building with ninja.

ninja is an alternative to msbuild designed for fast rebuilds. However there is no IDE support, builds only from the command line.

Comparison between msbuild and ninja for a full build, build time in seconds.

Full Clean Build
msbuild     867.5
Ninja       801.2
Difference  -66.3 (-7.6%)

Minor Change
msbuild      43.0
Ninja        14.9
Difference  -28.1 (-64.4%)

No Changes
msbuild      23.0
Ninja         6.1
Difference  -16.9 (-73.5%)

make.bat/cleanup: removed stray echo in make.bat

make.bat: refactor make.bat

make.bat was starting to become hard to maintain, this refactors it into separate batch files for each stage of the process.

-Improved detection of msvc2013/2015
-Improved failure handling.
-Added check for working msbuild and C++ compiler
-Added verbose switch to ease trouble shooting.
-Added Check if svn/cmake/git are in the path before using them
-Display the build configuration before asking to download the libraries
-Offer an option to recover an interrupted checkout of the libraries.
-Automatically check out sub-modules in-case they are missing.

Add number and memory size formatting throughout the UI

This commit adds number formatting (thousands separator) to the baking panel. It also adds a new function to format memory sizes (KB/GB/etc) and applies it to the baking panel and scene stats. The new function is unit tested.

Reviewers: Severin
Tags: #user_interface
Differential Revision: https://developer.blender.org/D1248

[windows/make.bat] use a more reliable way of locating visual studio 2017.

The registry hack we were using wasn't very reliable, the recommended way to locating visual studio is using vswhere (15.2 and up), using it also allows to switch between the regular and pre-release versions.

WM: check modal handlers for keymap lookups

Keep in sync with 2.8x

UI: fix assert

Replace hard-coded button size check with UI_UNIT_X.

Caused icon-only buttons to have strings assigned based on UI-scale.

3D View: add pixelsize function w/o UI scale

Cycles: Cleanup: Remove duplicated atan2f definition for OpenCL

Turns out that atan2f was already defined for OpenCL.

Cycles Denoising: Don't use atomics in the accumulation kernel on CPUs

The GPU kernel needs to use atomics for accumulation since all offsets are processed in
parallel, but on CPUs that's not the case, so we can disable them there for a considerable speedup.

Cycles/Compositor: Add arctan2 operation to the Math node

The Math node currently has the normal atan() function, but for
actual angles this is fairly useless without additional nodes to handle the signs.

Since the node has two inputs anyways, it only makes sense to add an arctan2 option.

Reviewers: sergey, brecht

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

3D View: remove poll 3D view for copy/paste

These operators only need selected objects.

Fix T55034: Setting duplication group for multiple selected items only
affects one item

UI editing multiple selected items missed the case of PROP_POINTER
properties

Reviewed By: campbellbarton

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

Fix Extend property of Lasso select tool in Mask editor not working

Reviewed By: campbellbarton

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

Fix T54336: Extend property of Lasso select tool in Node editor does not
work

Reviewed By: campbellbarton

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

Fix T55137: Compilation failing on non-x86-64 architectures

Some conversion helper functions were (most likely by accident) contained
inside an ifdef for SSE2 support, so on e.g. ARM they would be undefined
and therefore cause compilation to fail.

Cleanup: strip trailing space from interface files

Fix incorrect size in aligned lockfree realloc

Thanks to @alikendarfen for finding.

blenderplayer: add stubs

Fix T55093: Bisect + fill crash

Memory allocator: use lockfree calls internally

Was already used in some areas.

CMake: only include licences for enabled libs

Cleanup: use const for transform internal API

RNA: support for PARM_OUTPUT & PARM_RNAPTR

Fix too much memory usage for Cycles attribute map.

Thanks to Thomas Krebs for identifying the problem and solution.