Blender Git Loki

USD: typedef for std::map<short, pxr::VtIntArray>

cleanup

refactor: extract method

USD: use BKE_object_visibility()

refactor: extract methods

extract functions

USD: Clarified some comments, distinguish Hydra from USDViewer

USD: removed dead code

USD: added TODO about checking depsgraph for time dependency

cleanup

GPencil: Fix error opening old files

USD: use `BKE_animdata_id_is_animated()`

`BKE_animdata_id_is_animated()` is now used to determine whether object data
has any animation. This also includes counting the fcurves+drivers; previously
the presense of a non-NULL AnimData pointer was enough to consider the
object data animated.

cleanup

Merge remote-tracking branch 'origin/master' into temp-sybren-usd-patch

Alembic export: don't assume transform is always animated

Instead of always writing the transform on every frame, it's now checked
whether the object is animated at all. This could be made stricter to
reduce false positives, for example by checking FCurves and drivers to
see whether translation/rotation/scale is animated. However, this
approach is already better than the `return true` we had before.

This commit adds the BKE_animdata_id_is_animated(id) function, which
returns true if the ID datablock has non-empty animation data. This is
determined by checking the the active action's fcurves, the drivers, and
NLA tracks.

Removed `BKE_object_is_animated()`

The function isn't used anywhere, and it's deceptively returning false
negatives. For example, `modifier_dependsOnTime()` will return `false`
for hook modifiers, even when the hook target is animated. Querying the
depsgraph for dependency on the time source would be a better approach.

Made ob param of `modifiers_getVirtualModifierList` const

The `modifiers_getVirtualModifierList()` function previously took a non-
const `Object *ob` parameter, preventing it from being called from more
restrictive functions. Since the function doesn't modify the passed
object, it could easily be made const.

No functional changes.

Clarified comment about `modifiers_getVirtualModifierList()`

The old comment was outdated and incorrect.

No functional changes.

use array instead of map

Clip Editor: Remove Image label from selector

Unfortunately, it didn't turn out to be as great as I've hoped to it
will.

The issue is: the label eats too much space, making selector buttons
and image name to be barely readable. Initial idea of making the panel
somewhat wider didn't work either: due to the sizing policy of label
it takes a lot of panel width to make image name readable.