[Pkg-sugar-commit] [sugar-record-activity] 01/09: Delay recording processing after shutter sound

[Jonas Smedegaard]

This is an automated email from the git hooks/post-receive script.

js pushed a commit to branch master
in repository sugar-record-activity.

commit 41f8faa8eafe0136319d71216d7628bf43986bb1
Author: James Cameron <quozl at laptop.org>
Date:   Mon Aug 24 15:57:22 2015 +1000

    Delay recording processing after shutter sound
    
    Heavy processing by GStreamer of the audio recording pipeline interferes
    with playing of the shutter sound effect.  One period of the sound (960
    samples at 48000 Hz) is repeated for a time proportional to the
    compression time of the recorded audio data, because GStreamer is too
    busy to service the audio device.
    
    Delay processing until the shutter sound effect completion callback
    occurs.
    
    Affects XO-1.75 with a recent kernel.
    
    See also https://dev.laptop.org/ticket/12867#comment:10
    
    Signed-off-by: James Cameron <quozl at laptop.org>
---
 glive.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/glive.py b/glive.py
index 9f8c5fa..940868a 100644
--- a/glive.py
+++ b/glive.py
@@ -349,7 +349,9 @@ class Glive:
         # pipeline while manipulating it.
         # http://dev.laptop.org/ticket/10183
         self._pipeline.set_state(gst.STATE_NULL)
-        self.model.shutter_sound()
+        self.model.shutter_sound(self._stop_recording_audio)
+
+    def _stop_recording_audio(self):
         self._pipeline.remove(self._audiobin)
 
         audio_path = os.path.join(Instance.instancePath, "output.wav")

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-sugar/sugar-record-activity.git