forgot return value

xiongbinsh · Mar 22, 2014 · a10c196 · a10c196
1 parent 8f12354
commit a10c196
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/src/AVOutput.cpp b/src/AVOutput.cpp
@@ -149,7 +149,7 @@ void AVOutput::setStatistics(Statistics *statistics)
 
 bool AVOutput::installFilter(Filter *filter)
 {
-    onInstallFilter(filter);
+    return onInstallFilter(filter);
 }
 
 bool AVOutput::onInstallFilter(Filter *filter)
@@ -168,7 +168,7 @@ bool AVOutput::onInstallFilter(Filter *filter)
  */
 bool AVOutput::uninstallFilter(Filter *filter)
 {
-    onUninstallFilter(filter);
+    return onUninstallFilter(filter);
 }
 
 bool AVOutput::onUninstallFilter(Filter *filter)

diff --git a/src/VideoOutput.cpp b/src/VideoOutput.cpp
@@ -348,16 +348,18 @@ void VideoOutput::setStatistics(Statistics* statistics)
 bool VideoOutput::onInstallFilter(Filter *filter)
 {
     DPTR_D(VideoOutput);
-    d.impl->onInstallFilter(filter);
+    bool ret = d.impl->onInstallFilter(filter);
     d.filters = d.impl->filters();
+    return ret;
 }
 
 bool VideoOutput::onUninstallFilter(Filter *filter)
 {
     DPTR_D(VideoOutput);
-    d.impl->onUninstallFilter(filter);
+    bool ret = d.impl->onUninstallFilter(filter);
     // only used internally for AVOutput
     //d.pending_uninstall_filters =
+    return ret;
 }
 
 void VideoOutput::onAddOutputSet(OutputSet *set)
@@ -391,6 +393,7 @@ bool VideoOutput::onHanlePendingTasks()
     if (!d.impl->onHanlePendingTasks())
         return false;
     d.filters = d.impl->filters();
+    return true;
 }
 
 } //namespace QtAV