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/libstagefright/rtsp/ARTPWriter.cpp | |
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/libstagefright/rtsp/ARTPWriter.cpp')
-rw-r--r-- | media/libstagefright/rtsp/ARTPWriter.cpp | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/media/libstagefright/rtsp/ARTPWriter.cpp b/media/libstagefright/rtsp/ARTPWriter.cpp index d6dd597..d4eed7c 100644 --- a/media/libstagefright/rtsp/ARTPWriter.cpp +++ b/media/libstagefright/rtsp/ARTPWriter.cpp @@ -134,10 +134,10 @@ status_t ARTPWriter::start(MetaData *params) { return OK; } -void ARTPWriter::stop() { +status_t ARTPWriter::stop() { Mutex::Autolock autoLock(mLock); if (!(mFlags & kFlagStarted)) { - return; + return OK; } (new AMessage(kWhatStop, mReflector->id()))->post(); @@ -145,9 +145,11 @@ void ARTPWriter::stop() { while (mFlags & kFlagStarted) { mCondition.wait(mLock); } + return OK; } -void ARTPWriter::pause() { +status_t ARTPWriter::pause() { + return OK; } static void StripStartcode(MediaBuffer *buffer) { |