ic
/
magnum
mirror of https://github.com/mosra/magnum.git
6
0
Fork 0
Code Issues Projects Releases Wiki Activity
Browse Source

Audio: minor cleanup, rename Buffer::length() to sampleCount(). 

* use Magnum types in public API
 * make getters const consistently to Source getters
 * Buffer::length() could be confused with size(), rename it to make it
   more explicit.
pull/300/head
Vladimír Vondruš 7 years ago
parent
8029abca07
commit
cdb69423ec
2 changed files with 37 additions and 43 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Split View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 53
      src/Magnum/Audio/Buffer.h
  2. 27
      src/Magnum/Audio/Test/BufferALTest.cpp

53
src/Magnum/Audio/Buffer.h
Unescape View File

@ -105,42 +105,44 @@ class MAGNUM_AUDIO_EXPORT Buffer {
}
/**
* @brief Get buffer size
* @return The buffer's size in bytes
* @brief Buffer size in bytes
*
* @see @ref channels(), @ref bitDepth(), @ref sampleCount()
*/
ALint size() {
ALint size;
Int size() const {
Int size;
alGetBufferi(_id, AL_SIZE, &size);
return size;
}
/**
* @brief Get buffer channels
* @return The buffer's number of channels
* @brief Buffer channel count
*
* @see @ref size(), @ref bitDepth(), @ref sampleCount()
*/
ALint channels() {
Int channels() const {
ALint channels;
alGetBufferi(_id, AL_CHANNELS, &channels);
return channels;
}
/**
* @brief Get buffer bit depth
* @return The buffer's bit depth
* @brief Buffer bit depth
*
* @see @ref size(), @ref channels(), @ref sampleCount()
*/
ALint bitDepth() {
Int bitDepth() const {
ALint bitDepth;
alGetBufferi(_id, AL_BITS, &bitDepth);
return bitDepth;
}
/**
* @brief Get buffer length
* @return The buffer's length in samples
* @brief Buffer sample count
*
* Calculated from @ref size(), @ref channels() and @ref bitDepth().
*/
ALint length() {
return size() * 8 / (channels() * bitDepth());
}
Int sampleCount() const { return size()*8/(channels()*bitDepth()); }
/**
* @brief Get buffer loop points
@ -148,8 +150,8 @@ class MAGNUM_AUDIO_EXPORT Buffer {
*
* @requires_al_extension Extension @al_extension{SOFT,loop_points}
*/
std::pair<ALint, ALint> loopPoints() {
std::pair<ALint, ALint> points{0, 0};
std::pair<Int, Int> loopPoints() const {
std::pair<Int, Int> points{0, 0};
alGetBufferiv(_id, AL_LOOP_POINTS_SOFT, &(points.first));
return points;
}
@ -160,8 +162,8 @@ class MAGNUM_AUDIO_EXPORT Buffer {
* @param loopEnd The loop's end point in samples
* @return Reference to self (for method chaining)
*
* The buffer needs to not be attached to a source for this operation to
* succeed.
* The buffer needs to not be attached to a source for this operation
* to succeed.
* @requires_al_extension Extension @al_extension{SOFT,loop_points}
*/
Buffer& setLoopPoints(Int loopStart, Int loopEnd);
@ -171,9 +173,8 @@ class MAGNUM_AUDIO_EXPORT Buffer {
* @param loopEnd The loop's end point in samples
* @return Reference to self (for method chaining)
*
* Equivalent to calling @ref setLoopPoints() with @p loopStart equal to
* 0.
*
* Equivalent to calling @ref setLoopPoints() with @p loopStart equal
* to @cpp 0 @ce.
* @requires_al_extension Extension @al_extension{SOFT,loop_points}
*/
Buffer& setLoopUntil(Int loopEnd) {
@ -186,8 +187,7 @@ class MAGNUM_AUDIO_EXPORT Buffer {
* @return Reference to self (for method chaining)
*
* Equivalent to calling @ref setLoopPoints() with @p loopEnd equal to
* @p INT_MAX.
*
* @cpp INT_MAX @ce.
* @requires_al_extension Extension @al_extension{SOFT,loop_points}
*/
Buffer& setLoopSince(Int loopStart) {
@ -198,9 +198,8 @@ class MAGNUM_AUDIO_EXPORT Buffer {
* @brief Resets the loop points
* @return Reference to self (for method chaining)
*
* Equivalent to calling @ref setLoopPoints() with @p loopStart equal to
* 0, and @p loopEnd equal to @p INT_MAX.
*
* Equivalent to calling @ref setLoopPoints() with @p loopStart equal
* to @cpp 0 @ce, and @p loopEnd equal to @cpp INT_MAX @ce.
* @requires_al_extension Extension @al_extension{SOFT,loop_points}
*/
Buffer& resetLoopPoints() {

27
src/Magnum/Audio/Test/BufferALTest.cpp
Unescape View File

@ -75,13 +75,12 @@ void BufferALTest::properties() {
CORRADE_COMPARE(buf.size(), 8);
CORRADE_COMPARE(buf.channels(), 1);
CORRADE_COMPARE(buf.bitDepth(), 8);
CORRADE_COMPARE(buf.length(), 8);
CORRADE_COMPARE(buf.sampleCount(), 8);
}
void BufferALTest::loopPoints() {
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>()) {
CORRADE_SKIP("Extension AL_SOFT_loop_points isn't supported on this system.");
}
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>())
CORRADE_SKIP(Extensions::AL::SOFT::loop_points::string() + std::string{" is not supported."});
Buffer buf;
constexpr char data[] { 25, 17, 24, 122, 67, 24, 48, 96 };
@ -91,9 +90,8 @@ void BufferALTest::loopPoints() {
}
void BufferALTest::setLoopPoints() {
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>()) {
CORRADE_SKIP("Extension AL_SOFT_loop_points isn't supported on this system.");
}
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>())
CORRADE_SKIP(Extensions::AL::SOFT::loop_points::string() + std::string{" is not supported."});
Buffer buf;
constexpr char data[] { 25, 17, 24, 122, 67, 24, 48, 96 };
@ -103,9 +101,8 @@ void BufferALTest::setLoopPoints() {
}
void BufferALTest::setLoopSince() {
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>()) {
CORRADE_SKIP("Extension AL_SOFT_loop_points isn't supported on this system.");
}
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>())
CORRADE_SKIP(Extensions::AL::SOFT::loop_points::string() + std::string{" is not supported."});
Buffer buf;
constexpr char data[] { 25, 17, 24, 122, 67, 24, 48, 96 };
@ -115,9 +112,8 @@ void BufferALTest::setLoopSince() {
}
void BufferALTest::setLoopUntil() {
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>()) {
CORRADE_SKIP("Extension AL_SOFT_loop_points isn't supported on this system.");
}
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>())
CORRADE_SKIP(Extensions::AL::SOFT::loop_points::string() + std::string{" is not supported."});
Buffer buf;
constexpr char data[] { 25, 17, 24, 122, 67, 24, 48, 96 };
@ -127,9 +123,8 @@ void BufferALTest::setLoopUntil() {
}
void BufferALTest::resetLoopPoints() {
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>()) {
CORRADE_SKIP("Extension AL_SOFT_loop_points isn't supported on this system.");
}
if(!_context.isExtensionSupported<Audio::Extensions::AL::SOFT::loop_points>())
CORRADE_SKIP(Extensions::AL::SOFT::loop_points::string() + std::string{" is not supported."});
Buffer buf;
constexpr char data[] { 25, 17, 24, 122, 67, 24, 48, 96 };

Write Preview
Loading…
Cancel
Save