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/blenkernel/BKE_sound.h | |
| 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/blenkernel/BKE_sound.h')
| -rw-r--r-- | source/blender/blenkernel/BKE_sound.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/blender/blenkernel/BKE_sound.h b/source/blender/blenkernel/BKE_sound.h index f318c74258b..a4182b8405f 100644 --- a/source/blender/blenkernel/BKE_sound.h +++ b/source/blender/blenkernel/BKE_sound.h @@ -130,7 +130,7 @@ int sound_scene_playing(struct Scene *scene); void sound_free_waveform(struct bSound *sound); -void sound_read_waveform(struct bSound *sound, bool locked, short *stop); +void sound_read_waveform(struct bSound *sound, short *stop); void sound_update_scene(struct Main *bmain, struct Scene *scene); |