diff options
| author | Christer Fletcher <christer.fletcher@sonyericsson.com> | 2012-09-13 10:28:39 +0200 |
|---|---|---|
| committer | Johan Redestig <johan.redestig@sonymobile.com> | 2012-11-30 15:32:18 +0100 |
| commit | 37007147edde731cd3dc74febd7bb7e093a35020 (patch) | |
| tree | 3e85f5a609e879554d48945b502c951f75f44724 | |
| parent | 5768fa034ede834656697d3612c525595ff85ef9 (diff) | |
Initialize and copy mRotationAngle in VideoFrame
When thumbnails were generated they could be generated at random
angles as the mRotationAngle variable was not initialized to any
value. This variable would have to be explicitly overwritten to not
cause random rotation. Changed the implementation to initialize the
value to 0 (no rotation). mRotationAngle was also missing in the
copy constructor.
Change-Id: I67a5340fdd807c6ab3a3da5eecb09b5b9d5f4666
| -rw-r--r-- | include/private/media/VideoFrame.h | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/include/private/media/VideoFrame.h b/include/private/media/VideoFrame.h index 0ecc3481..a211ed94 100644 --- a/include/private/media/VideoFrame.h +++ b/include/private/media/VideoFrame.h @@ -88,7 +88,8 @@ public: class VideoFrame { public: - VideoFrame(): mWidth(0), mHeight(0), mDisplayWidth(0), mDisplayHeight(0), mSize(0), mData(0) {} + VideoFrame(): mWidth(0), mHeight(0), mDisplayWidth(0), mDisplayHeight(0), mSize(0), mData(0), + mRotationAngle(0) {} VideoFrame(const VideoFrame& copy) { mWidth = copy.mWidth; @@ -105,6 +106,7 @@ public: mSize = 0; } } + mRotationAngle = copy.mRotationAngle; } ~VideoFrame() { |