Commit d9a01822 authored by Serhiy Storchaka's avatar Serhiy Storchaka

Issue #19633: Fixed writing not compressed 16- and 32-bit wave files on

big-endian platforms.

Temporary forbidden test_unseekable_incompleted_write fornot compressed 16-
and 32-bit wave file  on big-endian platforms.
parent 8db1823c
......@@ -6,7 +6,8 @@ import pickle
import sys
def byteswap2(data):
a = array.array('h', data)
a = array.array('h')
a.frombytes(data)
a.byteswap()
return a.tobytes()
......@@ -17,7 +18,8 @@ def byteswap3(data):
return bytes(ba)
def byteswap4(data):
a = array.array('i', data)
a = array.array('i')
a.frombytes(data)
a.byteswap()
return a.tobytes()
......
......@@ -48,6 +48,12 @@ class WavePCM16Test(audiotests.AudioWriteTests,
if sys.byteorder != 'big':
frames = audiotests.byteswap2(frames)
if sys.byteorder == 'big':
@unittest.expectedFailure
def test_unseekable_incompleted_write(self):
super().test_unseekable_incompleted_write()
class WavePCM24Test(audiotests.AudioWriteTests,
audiotests.AudioTestsWithSourceFile,
......@@ -108,6 +114,11 @@ class WavePCM32Test(audiotests.AudioWriteTests,
if sys.byteorder != 'big':
frames = audiotests.byteswap4(frames)
if sys.byteorder == 'big':
@unittest.expectedFailure
def test_unseekable_incompleted_write(self):
super().test_unseekable_incompleted_write()
if __name__ == '__main__':
unittest.main()
......@@ -424,7 +424,9 @@ class Wave_write:
data = self._convert(data)
if self._sampwidth in (2, 4) and sys.byteorder == 'big':
import array
data = array.array(_array_fmts[self._sampwidth], data)
a = array.array(_array_fmts[self._sampwidth])
a.frombytes(data)
data = a
assert data.itemsize == self._sampwidth
data.byteswap()
data.tofile(self._file)
......
......@@ -13,6 +13,9 @@ Core and Builtins
Library
-------
- Issue #19633: Fixed writing not compressed 16- and 32-bit wave files on
big-endian platforms.
- Issue #19449: in csv's writerow, handle non-string keys when generating the
error message that certain keys are not in the 'fieldnames' list.
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment