diff options
author | James Dong <jdong@google.com> | 2010-08-18 19:10:39 -0700 |
---|---|---|
committer | James Dong <jdong@google.com> | 2010-08-19 13:33:13 -0700 |
commit | 37187916a486504acaf83bea30147eb5fbf46ae5 (patch) | |
tree | b011eba352b6e60ae459f88ef590b45b2e351110 /media/libmediaplayerservice | |
parent | 34c9ad88d3d827cca0c7b5f2fb1b0d4a1a20a2d1 (diff) | |
download | frameworks_av-37187916a486504acaf83bea30147eb5fbf46ae5.zip frameworks_av-37187916a486504acaf83bea30147eb5fbf46ae5.tar.gz frameworks_av-37187916a486504acaf83bea30147eb5fbf46ae5.tar.bz2 |
Make MediaWriter stop and pause return errors if necessary
o Make the API consistent with SF framework, which the MediaSource
provides a return status for stop
o Also, helps to convey errors that occurred right when a
premature stop() is called, leading to a potentially
mal-formed output file.
Change-Id: I52a932345f38570fdf8ea04d67d73dd94ccd30ef
Diffstat (limited to 'media/libmediaplayerservice')
-rw-r--r-- | media/libmediaplayerservice/StagefrightRecorder.cpp | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/media/libmediaplayerservice/StagefrightRecorder.cpp b/media/libmediaplayerservice/StagefrightRecorder.cpp index 94448c1..3c6d01b 100644 --- a/media/libmediaplayerservice/StagefrightRecorder.cpp +++ b/media/libmediaplayerservice/StagefrightRecorder.cpp @@ -1067,8 +1067,9 @@ status_t StagefrightRecorder::pause() { status_t StagefrightRecorder::stop() { LOGV("stop"); + status_t err = OK; if (mWriter != NULL) { - mWriter->stop(); + err = mWriter->stop(); mWriter.clear(); } @@ -1090,7 +1091,7 @@ status_t StagefrightRecorder::stop() { mOutputFd = -1; } - return OK; + return err; } status_t StagefrightRecorder::close() { |