Blender Git Loki

Refactopr Collada: Changed BCValueMap to contain Values+curve handles

This makes it easiuer to later construct the export curves.
This may later be removed again in favor of storing the data
within the FCurves themself. (I am still somewhat undecided here)

More work on opti edge split

Refactor Collada: Changed BCAnimationCurve methods to const methods

Refactor Collada: Added new Class BCKeyPoint

Refactor Collada: Removed unused parameter from method

BCAnimationCurve::add_value(const float val, const int frame_index)

Fix Collada: Keep existing FCurves available in Sampler

The FCurves are needed when we export Keyframes. In that case
we want to preserve the tangent data. Hence we want to keep
the original FCurves available while exporting.

Collada refactor: Added meaningful Constructor for BCAnimationSampler

The processing was unnecessarily split into creating the Sampler
and initialising it later. This can safely be done in one go
within a Sampler constructor

Also removed the not needed reference to the Sampler inside
the AnimationExporter instance. The Sampler is now a simple
local variable in AnimationExporter::exportAnimations()

Cleanup Collada: Removed unneeded Constructor

Refactor Collada: Renamed Enumerations to avoid confusion

The Curve animation type enumeration looked almost identical
to the enumeration that describes if the exporter shall
export sample frames or keyframes.

Details

old names:
BC_ANIMATION_TYPE_SAMPLE,
BC_ANIMATION_TYPE_KEYS

replaced by the more descriptive names:
BC_ANIMATION_EXPORT_SAMPLES,
BC_ANIMATION_EXPORT_KEYS

Refactor Collada: moved Object sampling code into its own private method

Refactor: Moved unrelated class methods to collada_utils

Refactor: Renamed classes, updated comments for better reading

DRW: Add UBO binding checking routine.

Cleanup: Fix compilation warnings.

DRW: Cleanup: Fix header naming.

GPUShader: Add name for debugging & identifying shaders.

Eevee: Fix missing UBO binds.

Some of them are unecessary and should be removed from the shader instead.

But for now we need a quick fix for the crashes happening on some platforms.

See T55475.

Cleanup collada: improved class description comment

Refactor Collada: Moved static class functions to collada_utils

Reason: The functions had nothing to do with the class

Cleanup Collada: moved typedefs to more intuitive locations