Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[ Issue #21 ] Adding Examples #23

Merged
merged 2 commits into from
Jun 13, 2018
Merged

[ Issue #21 ] Adding Examples #23

merged 2 commits into from
Jun 13, 2018

Conversation

localleon
Copy link
Contributor

Adding some Examples

As requested here's the Pull Request for Issue #21 . Its just basic playing and i tried to keep the sample mp3 as small as possible (6,2kib) . Suggestions for more Examples ?

Copy link
Owner

@faiface faiface left a comment

Choose a reason for hiding this comment

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

just a few pedant changes in the commends, otherwise good :)

)

func main() {
// Open first Sample File
Copy link
Owner

Choose a reason for hiding this comment

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

no need to capitalize Sample

// Open first Sample File
f, err := os.Open("sample1.mp3")

// Check for Errors when opening the file
Copy link
Owner

Choose a reason for hiding this comment

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

ditto about capitalizing Errors

// Channel, which will signal the end of the playback.
playing := make(chan struct{})

// Now we Play our Stream on the Speaker
Copy link
Owner

Choose a reason for hiding this comment

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

*Streamer

@faiface
Copy link
Owner

faiface commented Jun 12, 2018

Hi, thanks a lot for contributing this example! Other example suggestions would be:

  1. using Ctrl
  2. using Mixer
  3. using Resample in real-time
  4. similary for Volume, Pan, and perhaps Doppler

@localleon
Copy link
Contributor Author

Okay, i will correct the comments and add some more Examples. Maybe a specified folder for an example, so people can easily go build.

@localleon
Copy link
Contributor Author

Updated the Pull Reques and added a folder structure for the Examples

@faiface
Copy link
Owner

faiface commented Jun 13, 2018

Great! Thanks a lot for this.

@faiface faiface merged commit fe13953 into faiface:master Jun 13, 2018
@localleon
Copy link
Contributor Author

Thanks for merging, im now working on the ctrl example and effects example

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants