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 /include | |
parent | 5768fa034ede834656697d3612c525595ff85ef9 (diff) | |
download | frameworks_av-37007147edde731cd3dc74febd7bb7e093a35020.zip frameworks_av-37007147edde731cd3dc74febd7bb7e093a35020.tar.gz frameworks_av-37007147edde731cd3dc74febd7bb7e093a35020.tar.bz2 |
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
Diffstat (limited to 'include')
-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 0ecc348..a211ed9 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() { |