Blender Git Loki

Feature Collada: Prepare for exporting all actions of a scene

bc_getSceneActions(Object *ob, bool all_actions)

This function is supposed to return the list of actions
to be exported for the given object. However the function
is not yet used. Also it needs an improvement such that only
actions which match the active object will be exported.
This is still work in progress.

Cleanup Collada: changed import stage names for better reading.

The Collada Importer processes the imported files twice. It is still
not clear to me why it needs to do that. However i renamed the
Import stages for now to clarify a bit what really happens:

General -> Fetching_Scene_data
Controller -> Fetching_Controller_data

This change makes it more clear that when the controller data is
fetched these methods are not needed. Before it was much harder
to understand what this is all about.

Fix: Collada Exporter/Import break Object hierarchy

Exporter:
Children of Armatures where not exported as children but as sibblings.

Importer:
When an object is linked to an armature then the Importer
unconditionally makes the child a direct child of Armature.
Actually, if the child is already a child of the armature, then
there is no need to reparent it.

cleanup: Collada remove unused function

This was a function introduced earlier,
but it is no longer needed.

Cleanup: Added comment to clarify

The exporter does not export invisible and hidden objects!
This may cause issues. I added the comment to make this clear in
the code. This behavior may later be improved.

Cleanup Collada: Added new low level functions for later usage

bc_get_action_id() : To create unique Collada conform identifiers

The following new functions handle conversions of matrices
to various forms:

bc_copy_darray_m4d() : Copy matrix to an array of floats
bc_copy_v44_m4d() : Copy matrix to a vector of vectors of floats
bc_copy_m4d_v44() : opposite of previous function

Note: The vector<vector<double>> matrix is needed so that i later can
easily store matrix data in another vector (containing matrix curves)

Collada: Added Option "All actions" for exporting animation libs

This is work in progress. The follow up changes in
the exporter will eventually make use of the new options.
Currently the new option does nothing.

Cleanup: renamed transformName to channel_type

This makes it more clear what the parameter is about.
Especially it is not just a name but the paramter also
controls what type of data is exactly created by the exporter.

Cleanup: Added back explicit object type checks in Animation Exporter

For exporting Lamp Animation or Camera Animation i added utility functions
to fetch the related actions. And only if a matching action was found, the
export was processed for the object. However it turned out that the code
became less clear by this change. So i reverted it back to first check
the object type and only fetch the related action if the object type matches.

Cleanup: prepended bc_ to utility functions (satisfy name convention)

All utility functions from collada should contain the prependix bc_
Some of the new added functions did not comply to the convention.

Cleanup: calculating sample frames get scene action instead of object

Previously the functions for calculating the sample frames have used
the animated object from where they derived the active action. This was
not necessary. I changed the api so that the scene action is now
directly specified for better reading.

Cleanup: Added Helper functions to get Scene actions

This is mostly for cleaning up the AnimationExporter for
supporting the upcoming feature "export all animations".
I also added a few extra checks for existing animations,
which potentially avoid crashes when the calling functions
do not check for existing animations.

Removed const qualifier from EvaluationContext parameter and adjusted the class methods to reduce future merge conflicts in Blender2.8

Fix problem when change mode if no brushes

When chnage mode, create the default brushes if they are missing.

Fix problem with Brush Datablock icons

The icons of grease pencil brushes did not work because the poll return false.

WIP: more work moving datablock

All references to old bGPDbrush struct has been removed and basic functionality working again now.

Still the icons of the brushes are missing.

WIP: Fist step moving bGPDbrush to Brush datablock

Still a lot of work ahead, but this commit is to save the situation before remove bGPDbrush struct.

Merge branch 'master' into soc-2017-normal-tools

Partially revert "Fixed vertex groups not respected with keep sharp in modifier."

This reverts some changes from commit a7a4beecb208d12a00d550e3ebf4e2a37730fa6b.

Fixed vertex groups not respected with keep sharp in modifier.

Also fixed some other trivial issues.