Merge branch 'master' into blender2.8

Hide object center and other info when drawing sets

Alembic: use object-oriented approach in ABC_read_mesh()

This is easier to extend than the if/else if/else chain that was in place,
and allows for somewhat more granular error messages.

Alembic: Construct ISampleSelector once and pass along

Fix T51331: fixes for Alembic unit tests on Windows

Cycles: De-duplicate bit magic for decoding image options in OpenCL kernel

Cycles: Simplify code around maximum OpenCL info size allocation

Cycles: Cleanup, de-duplicate image packing of various types

Correct comment, we need wire_outline data in this case

Cycles: Quick (real) fix for broken textures on OpenCL

Previous fix did not work for mixed textures. This one will over-allocate
information array, but it's better than not being able to render at all.

Some more cleanup and improvement is coming.

Revert "Cycles: Fix image textures were completely broken since recent unlimited textures commit"

This reverts commit 8f4166ee495531fa38b676b0a5ef4c482e89f9a5.

The fix was not correct for cases when we've got float textures.

Draw Manager: pose-mode custom bone display

Cycles: Cleanup, spelling and braces

Cycles: Fix image textures were completely broken since recent unlimited textures commit

The indexing was totally wrong in both image packing code and image sampling in kernel.

Fixes T51341: Cycles OpenCL corruption in todays buildbot

Cycles: Cleanup, always use braces for blocks

Cycles: Cleanup, indentation in preprocessor

Cycles: Cleanup, use ccl::vector instead of std::vector

Cycles: Cleanup, indentaiton and trailing whitespace and wrapping

Recent commit state broke lattice vertex drawing

e4d856e3 caused lattice vertices not to draw in editmode.

Fix draw manager draw-modes interfering with eachother

Draw Manager: basic text overlay support

OpenGL: manage legacy state only when WITH_LEGACY_OPENGL=ON

- enabled lights
- alpha test
- texture environment
- point sprites (always enabled in modern GL)

Moved is_clip_plane for better struct packing, no functional change there.

Part of T51164

OpenGL: don't set texture LoD bias via glTexEnvf

Not supported in core profile. We could do this in GLSL if it's really needed.

Part of T51164

OpenGL: early exit from functions that don't mix with core profile

These parts will not be part of final viewport, but are called indirectly during the transition. To avoid runtime errors on core profile, exit early -- functions effectively do nothing.

I put the early exits inside the functions to avoid cluttering the code that calls these. But (long term) the calling functions need to change.

Basic shader's detect_options function was unused and full of old, so I deleted it.

Part of T51164

OpenGL: fix color logic op

GL_LOGIC_OP is a synonym for GL_INDEX_LOGIC_OP, from OpenGL 1.0's indexed color drawing. It's not part of modern GL and causes an error in core profile.

GL_COLOR_LOGIC_OP is still supported in the latest GL. It's the last stage before writing to the framebuffer.

Part of T51164

Add back text color for legacy drawing

Gawain: safely orphan immediate mode's VBO

There are multiple ways to orphan a buffer resource, and this code picks the oldest/safest way no matter which OpenGL version we're running.

TODO: use other (more recent) methods after thorough testing

Follow up to 8e0c57a812e8

OpenGL: don't glGet(GL_RED_BITS) in core profile

or GREEN or BLUE. These are deprecated and cause errors under core profile.

TODO: get color depth value another way

Part of T51164

OpenGL: stop enabling GL_TEXTURE

Texturing is always enabled in GLSL. Simply use a sampler in the shader.

Replaced gpu_generate_mipmap with glGenerateMipmap since the former just Enabled/Disabled the texture target and called the latter.

Part of T51164

OpenGL: stop using GL_NORMALIZE

With GLSL there is no need for GL_NORMALIZE. We explicitly normalize in the shader, or (better) send in unit vectors.

Part of T51164

Gawain: fix pointer arithmetic warning

void* + offset = ???

This Mac-only code works, but wasn't strictly correct. clang doesn't warn about this, gcc does.

fix Clay compatibility comment

When building WITH_LEGACY_OPENGL, Mac uses GL 2.1, Mesa uses GL 3.0. Has nothing to do with Intel!

Clay assumes GL 3.3 is available.

OpenGL: restore all 8 lights

fixes a copy/paste mistake

OpenGL: remove GL_MAP1_VERTEX_3

This is related to OpenGL evaluators (old feature). From my understanding, we don't seem to be actually using it.

Part of T51164

support OpenGL 3.0 on Linux only when WITH_LEGACY_OPENGL=ON

Part of T49012

Getting rid of setlinestyle: transform code.

Note that I also made 'dash anchor point' consistent (the static one,
not the mouse one), in previous code somtimes dashed were anchored to
the static center point, in others, to the moving mouse position, the
later was rather disturbing imho...

Fix: T50271: Bilateral/Directional blur's iterations is zero by default

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

Error in last commit

Cleanup: split cmake source/headers

Revert "Fix T51328: Add note to cmdline help that threads must be first"

The help message already notes that argument order matters,
and this doesn't need to be the first.

Merge branch 'master' into blender2.8

Cleanup: spelling

Cleanup: remove unused matrix arg

3D widgets were obscured by view geometry

Pose-mode drawing used alpha where it shouldn't

Cleanup: Some style and code tweaks to Image Code after changes.

Whitespace and order of switch/case etc. Let's try to stick to float4/byte4/half4/float/byte/half order as defined in "ImageDataType".

Fix gpuScaleUniform (caused strange pose-bone size)

Use pose-mode colors for pose-bone drawing

Also move static vars into structs

Unlimited number of textures for Cycles

This patch allows for an unlimited number of textures in Cycles where the hardware allows. It replaces a number static arrays with dynamic arrays and changes the way the flat_slot indices are calculated. Eventually, I'd like to get to a point where there are only flat slots left and textures off all kinds are stored in a single array.

Note that the arrays in DeviceScene are changed from containing device_vector<T> objects to device_vector<T>* pointers. Ideally, I'd like to store objects, but dynamic resizing of a std:vector in pre-C++11 calls the copy constructor, which for a good reason is not implemented for device_vector. Once we require C++11 for Cycles builds, we can implement a move constructor for device_vector and store objects again.

The limits for CUDA Fermi hardware still apply.

Reviewers: tod_baudais, InsigMathK, dingto, #cycles

Reviewed By: dingto, #cycles

Subscribers: dingto, smellslikedonkey

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

Cleanup: use doxy groups for armature

Fix basic pose-mode select drawing

Fix wrong scene assigned when using sets

Depsgraph was left with wrong set-scene.

Fix/workaround crash with non mesh geometry

eevee was crashing on curve grometry, draw without material for now.

Fix T51328: Add note to cmdline help that threads must be first

Cycles: Allow samples to finish in split kernel to avoid artifacts when canceling

Previously canceling a render done by the split kernel could cause artifacts
such as very bright or dark tiles. This was caused by unfinished samples
being included in the output buffer. To avoid this we now wait till all the
currently rendering samples have finished, up to a limit of twice the
expected time for them to finish (currently this is no more than 20 seconds,
but usually its much less). If samples still haven't finished by then we
stop anyways in case there's an endless loop occurring.

Cycles: Change OpenCL split kernel to use single program by default

Single program builds twice as fast as multi programs, so its better for
users to have it as the default.

Cycles: Enable Correlated Multi Jitter for OpenCL and split kernel

Testing showed no issues so there's no reason to not have this.

[msvc/make.bat] Provide a convenience rebuild.cmd in the build folder for easy rebuilds.

Getting rid of setlinestyle: Paint 'line' strokes.

Getting rid of setlinestyle: armature drawing code.

Firts usage of 3D version of new dashed line shader... Though not sure
whether that code will be kept for long?

Add 3D version of dashed line shader.

This is actually nearly same code as 2D version, maybe we can
deduplicate that later?

2D dashed line shader: changed to use viewport side instead of scale.

The scale version was working(ish), but it was not really extendable to
a 3D line version of the shader.

Also note that sequencer view still keeps its 'UI scale' adaptation
(dashes grow together with UI scale setting). Would be nice to do that
everywhere ultimately imho, but nothing urgent here.

Cleanup: rename select engine -> basic

This is used by depth drawing too and had no selection specific code.

Pass graph to depth functions

Draw Manager: cleanup, use doxy categories

Also add DRW_state for accessing current draw state.

Draw Manager: use engine drawing for depth drawing

View3D: move depth drawing into own function

No functional changes

Merge branch 'master' into 28

Merge commit 'master@6ed15c5a41130b55cb57a43a8a9470a91d38c3d5' into blender2.8

# Conflicts:
# source/blender/alembic/intern/abc_exporter.cc

Cycles: Allow samples to finish in split kernel to avoid artifacts when canceling

Previously canceling a render done by the split kernel could cause artifacts
such as very bright or dark tiles. This was caused by unfinished samples
being included in the output buffer. To avoid this we now wait till all the
currently rendering samples have finished, up to a limit of twice the
expected time for them to finish (currently this is no more than 20 seconds,
but usually its much less). If samples still haven't finished by then we
stop anyways in case there's an endless loop occurring.

Cycles: Change OpenCL split kernel to use single program by default

Single program builds twice as fast as multi programs, so its better for
users to have it as the default.

Cycles: Enable Correlated Multi Jitter for OpenCL and split kernel

Testing showed no issues so there's no reason to not have this.

[msvc/make.bat] Provide a convenience rebuild.cmd in the build folder for easy rebuilds.

Cycles: Fix over-allocation of triangles storage for triangle primitive hair

Was also causing some bad memory access caused by read data from non-initialized
arrays.

Repoted by bzztploink in IRC, thanks!

Fix race condition invalidating object data's bounding box

Ensure task scheduler exists before any threading starts in Blender

Avoid race condition copying Scene for off-thread OpenGL image save

User preferences: Use checkbox for Cycles device selection

It was totally unclear whether the device is enabled or disabled.
Lots of people got fully lost in the current interface.

While the solution is not fully ideal, it is at least solves
ambiguity in the interface.

Threading: Remove thread lock which seems useless

Render data is never guarded by image drawe lock.

Move depth select loop into its own function

Useful for splitting out draw logic in 2.8x

Alembic export: support simple child hairs (Fix T51144)

Simple child hairs don't have a face index number assigned, so the
call to dm->getTessFaceData(dm, num, CD_MFACE) would cause a crash. To
work around this, UV and normal vectors are copied from the parent
hair.

I've also removed an unnecessary call to dm->getTessFaceArray(dm);

Reviewers: kevindietrich

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

Alembic: fixed indentation

Alembic: fixed memory leaks

Alembic: fixed refcount issue when duplicating imported objects

Duplicating an imported object didn't increment the cache reader's refcount,
whereas removing the duplicate did decrement it. This caused problems.

Fix T51280: Alembic: Crash when removing cache modifier

The calls to id_us_plus/min were unnecessary (and caused assertion
failures) as this is already taken care of by transformcache_id_looper().

Getting rid of setlinestyle: 3DView ruler.

Gawain: add immAttrib2fv helper.

Getting rid of setlinestyle: non-camera 3DView render border.

Also fixed wrong color for in-camera render border, and added a comment
to imm_cpack about expected color components order in integer parameter.

Fix material preview for Cycles and BI

I had to update the .blend files too, and get them to use collections over the nameless old layers.

Last commit broke object-armature select

Draw Manager: pose-bone selection

OSX buildbot: remove redundant option

OSX buildbot: disable unsupported calls for now and use quicktime off default

Avoid platform dependant PATH_MAX

Getting rid of setlinestyle: Squencer space.

Not much to add here, except that it needs the scale of its MVP matrix
to be taken into account here...

Reviewers: merwin, dfelinto

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

Alembic export: also export empties

Exporting an empty creates an Alembic XForm object. The empties can also
be animated.

Alembic export: renamed func object_is_shape → object_type_is_exportable

The function doesn't return whether the object is a shape at all, since
it also returns true for camera objects (and soon also for empties). It
returns true when objects of this type can be exported to Alembic at all.
This is now reflected in the name.

Getting rid of setlinestyle: 3DView camera view.

Use new 2D dashed line shader in 3DView camera view.

Note that this also involved converting UI_draw_safe_areas() to this
dashed shader, which means it cannot be used anymore with other shaders.

Part of D2647.

Move imm_draw_line_box_dashed to GPU_immediate_util.

Tweak 2D line dashed shader to take a scale parameter.

Even though in some cases this does not seems useful, in others (like
zommed 2D views) we have to correct MVP matrix scaling to get fixed
dashes size.

Note that we could do that differently (commented about it in shader),
would also have been cleaner to extract that MVP scale from within the
shader, but there does not seem to be a way to initialize uniform values
from within a shader, and would rather avoid recomputing the scale for
every run (especially since 3DViewport does not need it e.g.).

Part of D2647.