Adjust code style for pointer types and casts
Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
This commit is contained in:
parent
2986accd20
commit
b70c4e07d0
2 changed files with 7 additions and 7 deletions
|
@ -322,7 +322,7 @@ def gen_function_wrapper(name, local_vars, args_dispatch):
|
||||||
:param name: Test function name
|
:param name: Test function name
|
||||||
:param local_vars: Local variables declaration code
|
:param local_vars: Local variables declaration code
|
||||||
:param args_dispatch: List of dispatch arguments.
|
:param args_dispatch: List of dispatch arguments.
|
||||||
Ex: ['(char *)params[0]', '*((int *)params[1])']
|
Ex: ['(char *) params[0]', '*((int *) params[1])']
|
||||||
:return: Test function wrapper.
|
:return: Test function wrapper.
|
||||||
"""
|
"""
|
||||||
# Then create the wrapper
|
# Then create the wrapper
|
||||||
|
@ -488,7 +488,7 @@ def parse_function_argument(arg, arg_idx, args, local_vars, args_dispatch):
|
||||||
typ, _ = m.groups()
|
typ, _ = m.groups()
|
||||||
if typ in SIGNED_INTEGER_TYPES:
|
if typ in SIGNED_INTEGER_TYPES:
|
||||||
args.append('int')
|
args.append('int')
|
||||||
args_dispatch.append('((mbedtls_test_argument_t*)params[%d])->sint' % arg_idx)
|
args_dispatch.append('((mbedtls_test_argument_t *) params[%d])->sint' % arg_idx)
|
||||||
return 1
|
return 1
|
||||||
if typ in STRING_TYPES:
|
if typ in STRING_TYPES:
|
||||||
args.append('char*')
|
args.append('char*')
|
||||||
|
@ -498,7 +498,7 @@ def parse_function_argument(arg, arg_idx, args, local_vars, args_dispatch):
|
||||||
args.append('hex')
|
args.append('hex')
|
||||||
# create a structure
|
# create a structure
|
||||||
pointer_initializer = '(uint8_t *) params[%d]' % arg_idx
|
pointer_initializer = '(uint8_t *) params[%d]' % arg_idx
|
||||||
len_initializer = '((mbedtls_test_argument_t*)params[%d])->len' % (arg_idx+1)
|
len_initializer = '((mbedtls_test_argument_t *) params[%d])->len' % (arg_idx+1)
|
||||||
local_vars.append(' data_t data%d = {%s, %s};\n' %
|
local_vars.append(' data_t data%d = {%s, %s};\n' %
|
||||||
(arg_idx, pointer_initializer, len_initializer))
|
(arg_idx, pointer_initializer, len_initializer))
|
||||||
args_dispatch.append('&data%d' % arg_idx)
|
args_dispatch.append('&data%d' % arg_idx)
|
||||||
|
|
|
@ -487,8 +487,8 @@ class ParseFuncSignature(TestCase):
|
||||||
self.assertEqual(local, '')
|
self.assertEqual(local, '')
|
||||||
self.assertEqual(arg_dispatch,
|
self.assertEqual(arg_dispatch,
|
||||||
['(char *) params[0]',
|
['(char *) params[0]',
|
||||||
'((mbedtls_test_argument_t*)params[1])->sint',
|
'((mbedtls_test_argument_t *) params[1])->sint',
|
||||||
'((mbedtls_test_argument_t*)params[2])->sint'])
|
'((mbedtls_test_argument_t *) params[2])->sint'])
|
||||||
|
|
||||||
def test_hex_params(self):
|
def test_hex_params(self):
|
||||||
"""
|
"""
|
||||||
|
@ -500,10 +500,10 @@ class ParseFuncSignature(TestCase):
|
||||||
self.assertEqual(args, ['char*', 'hex', 'int'])
|
self.assertEqual(args, ['char*', 'hex', 'int'])
|
||||||
self.assertEqual(local,
|
self.assertEqual(local,
|
||||||
' data_t data1 = {(uint8_t *) params[1], '
|
' data_t data1 = {(uint8_t *) params[1], '
|
||||||
'((mbedtls_test_argument_t*)params[2])->len};\n')
|
'((mbedtls_test_argument_t *) params[2])->len};\n')
|
||||||
self.assertEqual(arg_dispatch, ['(char *) params[0]',
|
self.assertEqual(arg_dispatch, ['(char *) params[0]',
|
||||||
'&data1',
|
'&data1',
|
||||||
'((mbedtls_test_argument_t*)params[3])->sint'])
|
'((mbedtls_test_argument_t *) params[3])->sint'])
|
||||||
|
|
||||||
def test_unsupported_arg(self):
|
def test_unsupported_arg(self):
|
||||||
"""
|
"""
|
||||||
|
|
Loading…
Reference in a new issue