diff options
author | James Dong <jdong@google.com> | 2012-02-02 15:07:52 -0800 |
---|---|---|
committer | James Dong <jdong@google.com> | 2012-02-02 15:13:56 -0800 |
commit | 8bcc65c753085fe3328592cceda0cf0e8f8b0a45 (patch) | |
tree | f7f9cff24dbf50e02f135b65bc1041e534c39d9c /media/libstagefright/AMRWriter.cpp | |
parent | 2af955b7437ac5345d75a5e528ebbb749667af7e (diff) | |
download | frameworks_av-8bcc65c753085fe3328592cceda0cf0e8f8b0a45.zip frameworks_av-8bcc65c753085fe3328592cceda0cf0e8f8b0a45.tar.gz frameworks_av-8bcc65c753085fe3328592cceda0cf0e8f8b0a45.tar.bz2 |
Don't call virtual functions in destructors for the writer classes
Have not found any concrete bugs related to these calls yet, but we should avoid
calling virtual functions in destructors, regardless.
Change-Id: I2d47b79d3fb2d29f418619bee83aa147d232a5d4
Diffstat (limited to 'media/libstagefright/AMRWriter.cpp')
-rw-r--r-- | media/libstagefright/AMRWriter.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/media/libstagefright/AMRWriter.cpp b/media/libstagefright/AMRWriter.cpp index 6c4e307..59b4ca7 100644 --- a/media/libstagefright/AMRWriter.cpp +++ b/media/libstagefright/AMRWriter.cpp @@ -52,7 +52,7 @@ AMRWriter::AMRWriter(int fd) AMRWriter::~AMRWriter() { if (mStarted) { - stop(); + reset(); } if (mFd != -1) { @@ -152,7 +152,7 @@ status_t AMRWriter::pause() { return OK; } -status_t AMRWriter::stop() { +status_t AMRWriter::reset() { if (!mStarted) { return OK; } |