Skip to content

Commit

Permalink
Audio: try to fix AppVeyor build.
Browse files Browse the repository at this point in the history 
I tried to "optimize" and broke it.
  • Loading branch information
@mosra
mosra committed Nov 14, 2015
1 parent 8cfbfd3 commit fd624d6
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/Magnum/Audio/Renderer.h
View file
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ class Renderer {
* @alc_extension{SOFT,HRTF}
*/
static bool isHrtfEnabled() {
Int enabled;
Int enabled = ALC_FALSE;
alGetIntegerv(ALC_HRTF_SOFT, &enabled);
return enabled == ALC_TRUE;
}
Expand All @@ -135,7 +135,7 @@ class Renderer {
if(!Context::current()->isExtensionSupported<Extensions::ALC::SOFT::HRTF>())
return isHrtfEnabled() ? HrtfStatus::Enabled : HrtfStatus::Disabled;

Int status;
Int status = ALC_HRTF_DISABLED_SOFT;
alGetIntegerv(ALC_HRTF_STATUS_SOFT, &status);
return HrtfStatus(status);
}
Expand Down

5 comments on commit fd624d6

@Squareys
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wondered about this, too. And I kinda never got hrtfs to actually work yet. I will fix this while working on the audio example simplification PR.

@Squareys
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Doh -_- Found it. This is the ALC_HRTF_SOFT extension, which obviously requires using alcGetIntegerv.

@Squareys
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR incoming after some testing ;)

@mosra
Copy link
Owner Author

@mosra mosra commented on fd624d6 Nov 14, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So I should revert this change, then. I guess.

@Squareys
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

#121 fixes everything related to this.

Please sign in to comment.