diff options
author | Lars-Dominik Braun <PromyLOPh@lavabit.com> | 2009-11-22 16:39:26 +0100 |
---|---|---|
committer | Lars-Dominik Braun <PromyLOPh@lavabit.com> | 2009-11-22 16:39:26 +0100 |
commit | 927a878f171580c1fdeb4d01caf0a2c9c32f8b42 (patch) | |
tree | 13c1f7a23f90ccc28959ecaaa85101eb6ec199a5 /src | |
parent | 7d2fa5b1bcce7351b81b2d0d927e61035c2224d9 (diff) | |
download | pianobar-927a878f171580c1fdeb4d01caf0a2c9c32f8b42.tar.gz pianobar-927a878f171580c1fdeb4d01caf0a2c9c32f8b42.tar.bz2 pianobar-927a878f171580c1fdeb4d01caf0a2c9c32f8b42.zip |
Don't use int->float->int casts
Diffstat (limited to 'src')
-rw-r--r-- | src/player.c | 30 |
1 files changed, 15 insertions, 15 deletions
diff --git a/src/player.c b/src/player.c index ef3cd19..2878f79 100644 --- a/src/player.c +++ b/src/player.c @@ -150,9 +150,10 @@ static char BarPlayerAACCb (void *ptr, size_t size, void *stream) { ao_play (player->audioOutDevice, (char *) aacDecoded, frameInfo.samples * 2); /* add played frame length to played time, explained below */ - player->songPlayed += (float) frameInfo.samples * - (float) BAR_PLAYER_MS_TO_S_FACTOR / - (float) player->samplerate / (float) player->channels; + player->songPlayed += (unsigned long long int) frameInfo.samples * + (unsigned long long int) BAR_PLAYER_MS_TO_S_FACTOR / + (unsigned long long int) player->samplerate / + (unsigned long long int) player->channels; player->bufferRead += frameInfo.bytesconsumed; player->sampleSizeCurr++; /* going through this loop can take up to a few seconds => @@ -235,15 +236,14 @@ static char BarPlayerAACCb (void *ptr, size_t size, void *stream) { player->bufferRead += 4; player->sampleSizeCurr = 0; /* set up song duration (assuming one frame always contains - * the same number of samples); numbers are too huge => use - * float + * the same number of samples) * calculation: channels * number of frames * samples per * frame / samplerate */ /* FIXME: Hard-coded number of samples per frame */ - player->songDuration = (float) player->sampleSizeN * - 4096.0 * (float) BAR_PLAYER_MS_TO_S_FACTOR / - (float) player->samplerate / - (float) player->channels; + player->songDuration = (unsigned long long int) player->sampleSizeN * + 4096LL * (unsigned long long int) BAR_PLAYER_MS_TO_S_FACTOR / + (unsigned long long int) player->samplerate / + (unsigned long long int) player->channels; break; } else { player->sampleSize[player->sampleSizeCurr] = @@ -361,9 +361,9 @@ static char BarPlayerMp3Cb (void *ptr, size_t size, void *stream) { } /* calc song length using the framerate of the first decoded frame */ - player->songDuration = (float) player->waith.contentLength / - ((float) player->mp3Frame.header.bitrate / - (float) BAR_PLAYER_MS_TO_S_FACTOR / 8.0); + player->songDuration = (unsigned long long int) player->waith.contentLength / + ((unsigned long long int) player->mp3Frame.header.bitrate / + (unsigned long long int) BAR_PLAYER_MS_TO_S_FACTOR / 8LL); /* must be > PLAYER_SAMPLESIZE_INITIALIZED, otherwise time won't * be visible to user (ugly, but mp3 decoding != aac decoding) */ @@ -377,9 +377,9 @@ static char BarPlayerMp3Cb (void *ptr, size_t size, void *stream) { if (player->mode == PLAYER_RECV_DATA) { /* same calculation as in aac player; don't need to divide by * channels, length is number of samples for _one_ channel */ - player->songPlayed += (float) player->mp3Synth.pcm.length * - (float) BAR_PLAYER_MS_TO_S_FACTOR / - (float) player->samplerate; + player->songPlayed += (unsigned long long int) player->mp3Synth.pcm.length * + (unsigned long long int) BAR_PLAYER_MS_TO_S_FACTOR / + (unsigned long long int) player->samplerate; } QUIT_PAUSE_CHECK; |