[Bug 33722] New: Stereo IMA ADPCM encoding in WINE produces loud static instead of sound
http://bugs.winehq.org/show_bug.cgi?id=33722 Bug #: 33722 Summary: Stereo IMA ADPCM encoding in WINE produces loud static instead of sound Product: Wine Version: 1.3.8 Platform: x86 OS/Version: Linux Status: UNCONFIRMED Severity: normal Priority: P2 Component: msacm32 AssignedTo: wine-bugs(a)winehq.org ReportedBy: Nachanon_Vetjasit(a)hotmail.com Classification: Unclassified I've been using Microsoft Sound Recorder to convert PCM (44100Hz 16-bit Stereo) wave file to IMA ADPCM (44100Hz 4-bit Stereo), and saved to a wave file. When I played the file in MPlayer, instead of the desired sound, it played a loud static on the right channel, and a mashup of both original channels on the left channel. Mono encoding didn't affected though. Other codecs are fine, except PCM (crashes; bug #26388), MS ADPCM and MP3 (unavailable). To reproduce: - Start Microsoft Sound Recorder (use Windows XP version or below): $ wine sndrec32.exe - Open some PCM wave file that is 44100Hz 16-bit Stereo. - Go to File > Properties - Click "Convert Now" - Choose "IMA ADPCM" codec with 44100Hz 4-bit Stereo configuration. - Click "OK" - Save it to another wave file. (No matter play button is grayed out or not) - Play that wave file with any player that supports IMA ADPCM. You'll hear a loud static on the right channel, and mashed-up sound on the left. Another thing is after encoded, Sound Recorder saw a doubled length of audio, MPlayer also saw so. When played, playhead moved twice the normal speed. (Which is a strange behavior) Note: I filed this as msacm32 since Sound Recorder is using it directly for encoding. Using WINE 1.3.8 (source) on Debian GNU/Linux 5.0 (i386: Intel Pentium 4 2.66GHz) WINE Audio Driver: ALSA 44100Hz 16bit (Full Hardware Acceleration) WINE installed without GStreamer support. -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=33722 Andrew Eikum <aeikum(a)codeweavers.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |aeikum(a)codeweavers.com -- Configure bugmail: http://bugs.winehq.org/userprefs.cgi?tab=email Do not reply to this email, post in Bugzilla using the above URL to reply. ------- You are receiving this mail because: ------- You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #1 from Andrew Eikum <aeikum(a)codeweavers.com> --- Created attachment 47260 --> http://bugs.winehq.org/attachment.cgi?id=47260 imaadpcm32: Fix source offsets when converting stereo PCM to IMAADPCM This patch fixes the broken right channel. The audio still sounds pretty terrible though, so I guess there's another problem with the conversion. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
http://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #2 from Nachanon Vetjasit <Nachanon_Vetjasit(a)hotmail.com> --- On WINE 1.7.15 with attachment #47260 applied, encoded right channel is no longer producing static, and channel isolation seems to be okay. Bad-quality conversion and doubled-length timeline issue still persist, but I'll file another bug for that. WINE 1.7.15 (git) with attachment #47260 on Debian GNU/Linux 5.0 (i386: Intel Pentium 4 2.66GHz) WINE Windows Version: Windows ME WINE Audio Driver: winealsa.drv WINE installed without XInput2, OSMesa, OpenCL, CMS, GStreamer and OSS support Sound Recorder: sndrec32.exe from Microsoft Windows ME -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #3 from Austin English <austinenglish(a)gmail.com> --- This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.51 or newer) wine? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 super_man(a)post.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |super_man(a)post.com --- Comment #4 from super_man(a)post.com --- (In reply to Austin English from comment #3)
This is your friendly reminder that there has been no bug activity for over a year. Is this still an issue in current (1.7.51 or newer) wine?
The patch still applies cleanly against 1.7.51 so it haven't been merged. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #5 from super_man(a)post.com --- patching file dlls/imaadp32.acm/imaadp32.c against wine 1.9.7 -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 winetest(a)luukku.com changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |winetest(a)luukku.com -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Bruno Jesus <00cpxxx(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |00cpxxx(a)gmail.com --- Comment #6 from Bruno Jesus <00cpxxx(a)gmail.com> --- Andrew, do you mind sending your patch from comment 1? -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #7 from Andrew Eikum <aeikum(a)codeweavers.com> --- (In reply to Bruno Jesus from comment #6)
Andrew, do you mind sending your patch from comment 1?
Sure, this is in Wine now. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |z.figura12(a)gmail.com --- Comment #8 from Zebediah Figura <z.figura12(a)gmail.com> --- Fixed by https://source.winehq.org/git/wine.git/commit/0f0fb984985aa08146996e04baf2b3.... -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Gijs Vermeulen <gijsvrm(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |gijsvrm(a)gmail.com, | |Nachanon_Vetjasit(a)hotmail.c | |om --- Comment #9 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Nachanon Vetjasit from comment #0)
-snip- Could you confirm that it is fixed? If so, mark it fixed and fill the "fixed by sha-1" field with 0f0fb984985aa08146996e04baf2b3d6e58abf46
-- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 --- Comment #10 from Gijs Vermeulen <gijsvrm(a)gmail.com> --- (In reply to Zebediah Figura from comment #8)
Fixed by https://source.winehq.org/git/wine.git/commit/ 0f0fb984985aa08146996e04baf2b3d6e58abf46.
Now that you have the power, I guess this one can be marked fixed as well? ;) -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Zebediah Figura <z.figura12(a)gmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Fixed by SHA1| |0f0fb984985aa08146996e04baf | |2b3d6e58abf46 Resolution|--- |FIXED Status|UNCONFIRMED |RESOLVED --- Comment #11 from Zebediah Figura <z.figura12(a)gmail.com> --- Marking fixed. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Alexandre Julliard <julliard(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |CLOSED --- Comment #12 from Alexandre Julliard <julliard(a)winehq.org> --- Closing bugs fixed in 3.0-rc4. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|--- |2.0.x -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
https://bugs.winehq.org/show_bug.cgi?id=33722 Michael Stefaniuc <mstefani(a)winehq.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Target Milestone|2.0.x |--- --- Comment #13 from Michael Stefaniuc <mstefani(a)winehq.org> --- Removing the 2.0.x milestone from bugs included in 2.0.5. -- Do not reply to this email, post in Bugzilla using the above URL to reply. You are receiving this mail because: You are watching all bug changes.
participants (1)
-
wine-bugs＠winehq.org