diff options
| author | Thiago Macieira <thiago.macieira@intel.com> | 2018-01-28 21:59:10 -0800 |
|---|---|---|
| committer | Thiago Macieira <thiago.macieira@intel.com> | 2018-01-30 23:12:52 +0000 |
| commit | d22d786d94540d50fde776c5f44ccdb041824343 (patch) | |
| tree | da0ea5d018247259a26eabe958368164629752b6 | |
| parent | a9f4b0561ba5cf372fb49076ef8013f0de13d30b (diff) | |
Initialize the unused member
GCC 7 complains that it is used uninitialized. The line number points to
the opening brace of the class, so I suppose it's the implicit copy
constructor.
Loading undefined floating point values is probably a bad idea. Could
generate an FP exception.
animationutils_p.h:126:8: error: ‘<anonymous>.Qt3DAnimation::Animation::ChannelNameAndType::pad’ may be used uninitialized in this function [-Werror=maybe-uninitialized]
Change-Id: I11d489f37e7e424b971dfffd150e3268d4be9294
Reviewed-by: Paul Lemire <paul.lemire@kdab.com>
| -rw-r--r-- | src/animation/backend/animationutils_p.h | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/src/animation/backend/animationutils_p.h b/src/animation/backend/animationutils_p.h index a0ab218d4..9778745c4 100644 --- a/src/animation/backend/animationutils_p.h +++ b/src/animation/backend/animationutils_p.h @@ -142,6 +142,7 @@ struct ChannelNameAndType , jointIndex(-1) , mappingId() , jointTransformComponent(NoTransformComponent) + , pad(0) {} ChannelNameAndType(const QString &_name, @@ -154,6 +155,7 @@ struct ChannelNameAndType , jointIndex(_jointIndex) , mappingId(_mappingId) , jointTransformComponent(NoTransformComponent) + , pad(0) {} ChannelNameAndType(const QString &_name, @@ -165,6 +167,7 @@ struct ChannelNameAndType , jointIndex(invalidIndex) , mappingId() , jointTransformComponent(_jointTransformComponent) + , pad(0) {} bool operator==(const ChannelNameAndType &rhs) const |