From d020c524bad9a9e26b124b85a356d3858b03cab7 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Tomasz=20Mi=C4=85sko?= Date: Mon, 7 Jan 2019 00:00:00 +0000 Subject: scanner: Flatten multi-dimensional arrays fields Provide partial support for multi-dimensional arrays by representing them as flattened one dimensional array with size that is equal to product of sizes in each dimension. Previously only the first dimension would be actually used. This should be sufficient to ensure that those fields have layout compatible with C, without using nested array types that are currently unsupported by vapigen Issue #255. --- tests/offsets/gitestoffsets.c | 11 ++++++++++ tests/offsets/offsets.h | 15 ++++++++++++++ tests/scanner/test_transformer.py | 42 +++++++++++++++++++++++++++++++++++++++ 3 files changed, 68 insertions(+) (limited to 'tests') diff --git a/tests/offsets/gitestoffsets.c b/tests/offsets/gitestoffsets.c index 9ce673e9..a84e8e8a 100644 --- a/tests/offsets/gitestoffsets.c +++ b/tests/offsets/gitestoffsets.c @@ -92,6 +92,16 @@ compiled (FILE *outfile) PRINT_MEMBER (OffsetsArray, some_ptrs); g_fprintf (outfile, "\n"); + PRINT_TYPE (OffsetsMultiDimArray); + PRINT_MEMBER (OffsetsMultiDimArray, ints); + PRINT_MEMBER (OffsetsMultiDimArray, chars); + PRINT_MEMBER (OffsetsMultiDimArray, floats); + PRINT_MEMBER (OffsetsMultiDimArray, pointers1); + PRINT_MEMBER (OffsetsMultiDimArray, pointers2); + PRINT_MEMBER (OffsetsMultiDimArray, pointers3); + PRINT_MEMBER (OffsetsMultiDimArray, dummy); + fprintf (outfile, "\n"); + PRINT_TYPE (OffsetsBasic); PRINT_MEMBER (OffsetsBasic, dummy1); PRINT_MEMBER (OffsetsBasic, field_int8); @@ -183,6 +193,7 @@ int main(int argc, char **argv) g_error ("Cannot open '%s': %s'", argv[1], g_strerror(errno)); introspected_struct (outfile, "Array"); + introspected_struct (outfile, "MultiDimArray"); introspected_struct (outfile, "Basic"); introspected_struct (outfile, "Enum"); introspected_struct (outfile, "Nested"); diff --git a/tests/offsets/offsets.h b/tests/offsets/offsets.h index 658601f5..7d89fe8d 100644 --- a/tests/offsets/offsets.h +++ b/tests/offsets/offsets.h @@ -124,6 +124,21 @@ struct _OffsetsArray gpointer some_ptrs[5]; }; +/* Test multi-dimensional arrays */ + +typedef struct _OffsetsMultiDimArray OffsetsMultiDimArray; + +struct _OffsetsMultiDimArray +{ + gint ints[10][2]; + gchar chars[255][10]; + float floats[11][13][17]; + gchar* pointers1[3][5]; + gpointer pointers2[7][9]; + double** pointers3[2][3][4]; + gchar dummy; +}; + /* Test object offsets */ typedef struct _OffsetsObj OffsetsObj; diff --git a/tests/scanner/test_transformer.py b/tests/scanner/test_transformer.py index 4731b5ce..92f21c67 100644 --- a/tests/scanner/test_transformer.py +++ b/tests/scanner/test_transformer.py @@ -483,5 +483,47 @@ class TestCallbacks(unittest.TestCase): self.assertTrue(isinstance(node, ast.Callback)) +class TestArrays(unittest.TestCase): + def setUp(self): + # Hack to set logging singleton + self.namespace = ast.Namespace('Test', '1.0') + logger = MessageLogger.get(namespace=self.namespace) + logger.enable_warnings((WARNING, ERROR, FATAL)) + + def test_multidimensional_arrays(self): + """Multidimensional arrays are flattend into a single dimension.""" + + load_namespace_from_source_string(self.namespace, """ + typedef struct { + int data[2][3][5][7][11]; + } TestMultiDimArray; + """) + + node = self.namespace.get('MultiDimArray') + self.assertIsNotNone(node) + + field = node.fields[0] + self.assertIsInstance(field, ast.Field) + self.assertIsInstance(field.type, ast.Array) + self.assertEqual(field.type.element_type, ast.TYPE_INT) + self.assertEqual(field.type.size, 2 * 3 * 5 * 7 * 11) + + def test_flexible_array_member(self): + """Flexible array members don't break transformer. + + They are generally unsupported, so nothing else is validated. + """ + + load_namespace_from_source_string(self.namespace, """ + typedef struct { + int length; + unsigned char data[]; + } TestFlexibleArray; + """) + + node = self.namespace.get('FlexibleArray') + self.assertIsNotNone(node) + + if __name__ == '__main__': unittest.main() -- cgit v1.2.1