diff options
author | Nicholas Bishop <nicholasbishop@gmail.com> | 2015-01-15 14:28:08 +0300 |
---|---|---|
committer | Nicholas Bishop <nicholasbishop@gmail.com> | 2015-01-15 16:00:51 +0300 |
commit | b41ce0d1b95973c64bb4300a20c1b89506fe32d0 (patch) | |
tree | 7bdab570c8c2d8baf5ea9286c203fefa7ff864f1 /source/blender/editors/space_sequencer/sequencer_preview.c | |
parent | 588656a568a837d3d043c91b684ebbe2a006b2af (diff) |
Remove 'locked' parameter from sound_read_waveform()
This parameter was confusing in three ways:
1. It should have been named "lock" because it was used to take and
release the sound mutex, not to indicate whether it was locked.
2. In the one place this function gets called the locked argument was
set to "true", so not much point in having it optional.
3. I can't imagine that it would ever be a good idea to skip taking
and releasing the mutex.
Reviewed By: sergey
Differential Revision: https://developer.blender.org/D988
Diffstat (limited to 'source/blender/editors/space_sequencer/sequencer_preview.c')
-rw-r--r-- | source/blender/editors/space_sequencer/sequencer_preview.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/editors/space_sequencer/sequencer_preview.c b/source/blender/editors/space_sequencer/sequencer_preview.c index dd6349efb8f..da00b0ff6e1 100644 --- a/source/blender/editors/space_sequencer/sequencer_preview.c +++ b/source/blender/editors/space_sequencer/sequencer_preview.c @@ -85,7 +85,7 @@ static void preview_startjob(void *data, short *stop, short *do_update, float *p PreviewJobAudio *preview_next; bSound *sound = previewjb->sound; - sound_read_waveform(sound, true, stop); + sound_read_waveform(sound, stop); if (*stop || G.is_break) { BLI_mutex_lock(pj->mutex); |