From 36784bbc2c3faad776d9d12481a0ed9a6aabb646 Mon Sep 17 00:00:00 2001 From: Philippe Saint-Pierre Date: Mon, 25 Jun 2012 13:43:53 -0400 Subject: [PATCH] StringForRate(): adjustments * Make 1000.0f the default base * Also use sizeof() in MediaPlayer to get the string size rather than hardcode. Thanks Marcus --- headers/private/shared/StringForRate.h | 2 +- src/apps/mediaplayer/InfoWin.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/headers/private/shared/StringForRate.h b/headers/private/shared/StringForRate.h index bd1a0bd8d7..65e17407da 100644 --- a/headers/private/shared/StringForRate.h +++ b/headers/private/shared/StringForRate.h @@ -12,7 +12,7 @@ namespace BPrivate { const char* string_for_rate(double rate, char* string, size_t stringSize, - double base = 1024.0f); + double base = 1000.0f); } // namespace BPrivate diff --git a/src/apps/mediaplayer/InfoWin.cpp b/src/apps/mediaplayer/InfoWin.cpp index e6508b07fe..2316481d6f 100644 --- a/src/apps/mediaplayer/InfoWin.cpp +++ b/src/apps/mediaplayer/InfoWin.cpp @@ -410,7 +410,7 @@ printf("InfoWin::Update(0x%08lx)\n", which); float br = format.u.encoded_audio.bit_rate; char string[20] = ""; if (br > 0.0) - s << ", " << string_for_rate(br, string, 20, 1000.0); + s << ", " << string_for_rate(br, string, sizeof(string)); } s << "\n\n"; fContentsView->Insert(s.String());