Blender Git Loki

Cleanup: Remove duplicated condition

Remove brush mode panel and add icon

The panel was not needed and was replaced by icons in the main panel.

Notice: I have reused standard icons, but maybe need custom icons in the future, but we will wait to the main redesign 2.8 interface task.

BMesh: fix bug where indices weren't set /w offset

Merge branch 'blender2.8' into greasepencil-object

Change Brush names to organize alphabetically

As the panel is sorted by name, it's better to have names that keep brushes grouped.

Change Eraser icon depending of eraser mode

Remove unused and duplicate code

After moving bGPDbrush to Brush, some data is duplicated or not used.

Merge branch '28' into temp-object-multi-mode

Merge branch 'blender2.8' into temp-object-multi-mode

Multi-Edit UV select

- (de)select all
- select linked
- mouse picking (vert/face/island)
(refactor selecting nearest UV so it can be used with many meshes)

Refactor Collada: Moved Classes into separate files

Renamed AnimationCurveCache to BCAnimationCurveContainer
because this is really not a cache but just a set of Data that
prepares the export of FCurves. This container only lives during
export and is otherwiose not used for anything else.

The BCAnimationCurveContainer is not depending on Collada in any way.
Theoretically it could be used also from Python, but this is another story.

Additions in more detail:

Classes:

* BCAnimationCurve
a wrapper for handling FCurves tied to an object within the
Animation Exporter

* BCAnimationCurveContainer
A container to keep all FCurves in one place. It also maintains
creation and deletion of temporary FCurves needed to export data
without interfering with the User's production data in Blender.

* BCMatrix
Handy class for making float mat[4][4] a primetime resident in
the code. Not necessary but nice.

Support functions:

* bc_get_children(std::vector<Object *> &child_set, Object *ob, Scene *scene)
returns the immediate children of an object into a set

* bc_string_after(const std::string& s, const char c)
returns the string after the last occurance of char

* bc_startswith(std::string const & value, std::string const & starting)
returns true if the string starts with the starting sequence

* bc_endswith(std::string const & value, std::string const & ending)
returns true if the string ends with the ending sequence

fix: Collada: changed scene update for sampling animation data

According to a comment in physics_fluid.c it sounds like the
function BKE_scene_update_for_newframe() does not work in all
cases. According to the comment it seems better to use
ED_update_for_newframe() instead.

This change only affects the export of Animation Data.
I make this change as a precaution. I am not aware that
there was anything wrong here before.

Feature Collada: Added AnimationCurveCache for exporting sampled animations

Details:
When exporting sampled Animations to Collada we need to create sample
data for every Animation curve. However it turns out to be extremely
slow when we repeatedly step over the entire animation for each curve.

The solution is to only step over the animation once but cache all needed
animation data in a cache. Once the animation has been processed,
all export information is located in the cache and can be easily
read from there.

The AnimationCurveCache is the first try to proof the concept.
This is work in progres and may still change a lot.

Feature Collada: Added new class files to CMakeLists.txt

AnimationClipExporter.cpp and AnimationClipExporter.h both need
to be added to CMakeLists.txt so they are actually built into
Blender

Feature Collada: Added new Enum and Mappings

* BC_animation_transform_type

Enum that Contains all animation types which can be exported

* BC_ANIMATION_NAME_FROM_TYPE

Map from transform_type to the name that is used in the exported data

* BC_ANIMATION_TYPE_FROM_NAME

Maps to get the animation transformtype from RNA Pathes
and from collada names (also needed for the Importer later)

These maps and Enums will later replace hard coded integer numbers
which are currently used all over the place and make it really hard
to understand what exactly is going on.

Cleanup Collada: Added #ifdef to avoid duplicate include

This avoids that the include file AnimationExporter.h
gets included more than once.

Feature Collada: Added new class for exporting Animation Clips.

This class is work in progres and not yet used. It is
also not yet largely tested and needs some improvements
regarding which actions will be exported for which objects.

Cleanup Collada: renamed parameters for better reading

I tested and commited each of the documented changes
separately and amended all changes into one single commit:

None of the changes makes a functional change. All changes are
in AnimationExporter.h and AnimationExporter.cpp

* Renamed from transformName to channel_type
* in create_sampled_animation():
renamed label to channel_type
* in export_sampled_transrotloc_animation()
in export_sampled_matrix_animation()
in export_sampled_animation_set()
in create_sampled_animation():
Renamed variable from times to frames
* Fixed Indentation in AnimationExporter.h
* break up function definition into multiple lines.

Feature Collada: Support for Animation Clip export

1.) Added one extra level to the exported animations
such that now all scene animations are enclosed:

<Animation name="id_name(ob)_Action">
<Animation>...</Animation>
...
</Animation>

2.) Added support functions for this extra level:

open_animation_container(bool has_container, Object *ob);
close_animation_container(bool has_container);

3.) Added Function for organizing unassigned exported animations

openAnimationWithClip() (not uesed yet)

4.) Added support function to check if a sampled matrix animation
is actually animating

bool is_flat_line() (not used yet)

Cleanup: Fixed indentation and added white space lines

Very minor change just for better reading