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

Fix issue #70 - WAV encoding for mono recording #71

Closed
wants to merge 1 commit into from

Conversation

andpor
Copy link

@andpor andpor commented Mar 2, 2016

Correct the WAV encoding for single channel or mono recording. byteRate field in the WAV header was incorrectly computed

@noahjerreel
Copy link

Hello @andpor does this audio work when recording to stereo type device, like laptops?

I created an issue regarding the audio got choppy when trying to record on laptop with 1 audio channel setting for the recorder

https://github.com/streamproc/MediaStreamRecorder/issues/92

@andpor
Copy link
Author

andpor commented Jun 21, 2016

I think there was a bug for stereo recording - the right buffer was getting leftChannel data. I fixed that part on this PR which unfortunately has not been merged yet...

@noahjerreel
Copy link

@andpor but did you test it? Is it working now?

@muaz-khan muaz-khan closed this in 7f5bcd5 Jul 3, 2016
@andpor
Copy link
Author

andpor commented Jul 3, 2016

So you decided not to merge this in? Can you explain why...just curious...

@andpor
Copy link
Author

andpor commented Jul 3, 2016

@noahjerreel - my fix works fine.

@noahjerreel
Copy link

@andpor - I would love to test it but respect to @muaz-khan , will use his latest release. Sorry man!

@andpor
Copy link
Author

andpor commented Jul 13, 2016

Nothing to be sorry about - my fix in there...not directly merged but it is there...

@noahjerreel
Copy link

That's good then @andpor , right?

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.

2 participants