Skip to contentSkip to conversation

audio/sdlaudio: remove unused variable

PATCH Applied Serge Ziryukin 2 participants 2 messages Apr 22, 2010

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 audio/sdlaudio: remove unused variable

Remove unused 'shift' variable spotted by clang. Also clean up aud_to_sdlfmt which used to get the value of shift.

---
 audio/sdlaudio.c |   11 ++---------
 1 files changed, 2 insertions(+), 9 deletions(-)
audio/sdlaudio.c +2 -9
index 8e7e5cb..c353016 100644
@@ -115,23 +115,19 @@ static int sdl_unlock_and_post (SDLAudioState *s, const char *forfn)
     return sdl_post (s, forfn);
 }
 
-static int aud_to_sdlfmt (audfmt_e fmt, int *shift)
+static int aud_to_sdlfmt (audfmt_e fmt)
 {
     switch (fmt) {
     case AUD_FMT_S8:
-        *shift = 0;
         return AUDIO_S8;
 
     case AUD_FMT_U8:
-        *shift = 0;
         return AUDIO_U8;
 
     case AUD_FMT_S16:
-        *shift = 1;
         return AUDIO_S16LSB;
 
     case AUD_FMT_U16:
-        *shift = 1;
         return AUDIO_U16LSB;
 
     default:
@@ -326,16 +322,13 @@ static int sdl_init_out (HWVoiceOut *hw, struct audsettings *as)
     SDLVoiceOut *sdl = (SDLVoiceOut *) hw;
     SDLAudioState *s = &glob_sdl;
     SDL_AudioSpec req, obt;
-    int shift;
     int endianess;
     int err;
     audfmt_e effective_fmt;
     struct audsettings obt_as;
 
-    shift <<= as->nchannels == 2;
-
     req.freq = as->freq;
-    req.format = aud_to_sdlfmt (as->fmt, &shift);
+    req.format = aud_to_sdlfmt (as->fmt);
     req.channels = as->nchannels;
     req.samples = conf.nb_samples;
     req.callback = sdl_callback;
Signature · 1 line
1.7.0.5
Applied
#1 lore raw
Quoted 4 lines · Serge Ziryukin

Remove unused 'shift' variable spotted by clang. Also clean up aud_to_sdlfmt which used to get the value of shift.

Applied. Thank you.

Signature · 1 line
mailto:[email protected]