Skip to content
This repository was archived by the owner on Aug 31, 2021. It is now read-only.

[Bug 19630] Fix setting the iphoneSetAudioCategory#5446

Merged
livecodepanos merged 2 commits into
livecode:develop-8.1from
livecodepanos:bugfix-19630
May 10, 2017
Merged

[Bug 19630] Fix setting the iphoneSetAudioCategory#5446
livecodepanos merged 2 commits into
livecode:develop-8.1from
livecodepanos:bugfix-19630

Conversation

@livecodepanos
Copy link
Copy Markdown
Contributor

@livecodepanos livecodepanos commented May 10, 2017

The return value of MCSoundAudioCategoryFromString(*t_category_string) was not taken into account, and this resulted in not being able to actually set the iphoneSetAudioCategory

@livecodepanos livecodepanos added this to the 8.1.5-rc-1 milestone May 10, 2017
@mention-bot
Copy link
Copy Markdown

@livecodepanos, thanks for your PR! By analyzing the history of the files in this pull request, we identified @livecodeali and @livecodefraser to be potential reviewers.

@livecodeali
Copy link
Copy Markdown
Member

@livecode-vulcan review ok 59ef19f

@livecode-vulcan
Copy link
Copy Markdown
Contributor

💙 review by @livecodeali ok 59ef19f

@livecode-vulcan
Copy link
Copy Markdown
Contributor

😎 test success 59ef19f

  • try-community-armv6-android-api8: success
  • try-community-armv6-android-api9: success
  • try-community-js-emscripten-sdk1.35: success
  • try-community-universal-ios-iphoneos10.3: success
  • try-community-universal-ios-iphonesimulator10.3: success
  • try-community-universal-mac-macosx10.6: success
  • try-community-universal-mac-macosx10.9: success
  • try-community-x86-linux-debian7: success
  • try-community-x86-linux-debian8: success
  • try-community-x86_64-linux-debian7: success
  • try-community-x86_64-linux-debian8: success
  • try-community-x86-win32: success
  • try-community-x86_64-win32: success

@livecodepanos livecodepanos merged commit 77dfdf1 into livecode:develop-8.1 May 10, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants