Browse Source

bpo-22273: Disabled tests while investigating buildbot failures on ARM7L/PPC64. (GH-16377)

pull/16381/head
Vinay Sajip 7 years ago
committed by GitHub
parent
commit
57dc7d5ae8
No known key found for this signature in database GPG Key ID: 4AEE18F83AFDEB23
  1. 5
      Lib/ctypes/test/test_structures.py
  2. 2
      Modules/_ctypes/stgdict.c

5
Lib/ctypes/test/test_structures.py

@ -1,3 +1,4 @@
import platform
import unittest
from ctypes import *
from ctypes.test import need_symbol
@ -5,6 +6,8 @@ from struct import calcsize
import _ctypes_test
from test import support
MACHINE = platform.machine()
class SubclassesTest(unittest.TestCase):
def test_subclass(self):
class X(Structure):
@ -477,6 +480,8 @@ class StructureTestCase(unittest.TestCase):
self.assertEqual(s.first, got.first)
self.assertEqual(s.second, got.second)
@unittest.skipIf(MACHINE in ('armv7l', 'ppc64'),
'Test temporarily disabled on this architecture')
def test_array_in_struct(self):
# See bpo-22273

2
Modules/_ctypes/stgdict.c

@ -652,7 +652,7 @@ PyCStructUnionType_update_stgdict(PyObject *type, PyObject *fields, int isStruct
#define MAX_ELEMENTS 16
if (arrays_seen && (size <= 16)) {
if (arrays_seen && (size <= MAX_ELEMENTS)) {
/*
* See bpo-22273. Arrays are normally treated as pointers, which is
* fine when an array name is being passed as parameter, but not when

Loading…
Cancel
Save