Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update aac libraries for android #32

Merged
merged 3 commits into from
Oct 24, 2016

Conversation

madhu314
Copy link
Contributor

This PR will be addressing Issue #30

@madhu314
Copy link
Contributor Author

@tlenclos any update for this ? would you like to review this PR ?

@tlenclos
Copy link
Owner

I really want to test it against my app and will try tonight, sorry for the delay.

@madhu314
Copy link
Contributor Author

No worries @tlenclos , just wanted to hear an update. This definitely needs a solid testing to ensure we don't break anything on android

@tlenclos
Copy link
Owner

It appears that you have the commits from this PR also #29. I'll test all your dev and merge only this PR then.

@madhu314
Copy link
Contributor Author

Ahh my bad, that was unintended, however if both of these are ok, please merge this and I will revoke PR #29
otherwise, I can split these and send new PRs appropriately

@tlenclos
Copy link
Owner

Tested and approved, I'll just fix some code style and the example app on the master branch

@tlenclos tlenclos merged commit d23a722 into tlenclos:master Oct 24, 2016
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants