| commit | 90461df0cad131abf2abb8924360bab9dbc7f54b | |
| author | Eric Blake <[email protected]> | |
| Mon, 7 Oct 2013 19:18:56 +0000 (7 13:18 -0600) | ||
| committer | Eric Blake <[email protected]> | |
| Mon, 14 Oct 2013 20:34:37 +0000 (14 14:34 -0600) | ||
| tree | 5d706a6f3650c2d290cc2fbfdaaed7d5f49883b2 | treesnapshot (tar.gz zip) |
| parent | bc116872291690bc18e475d51a65748122ddec44 | commitdiff |
maint: avoid 'const fooPtr' in python bindings
'const fooPtr' is the same as 'foo * const' (the pointer won't
change, but it's contents can). But in general, if an interface
is trying to be const-correct, it should be using 'const foo *'
(the pointer is to data that can't be changed).
Fix up offenders in the python bindings.
* python/generator.py (py_types): Drop useless conversions.
* python/libvirt-override.c (getPyVirTypedParameter)
(setPyVirTypedParameter): Use intended type.
Signed-off-by: Eric Blake <[email protected]>
'const fooPtr' is the same as 'foo * const' (the pointer won't
change, but it's contents can). But in general, if an interface
is trying to be const-correct, it should be using 'const foo *'
(the pointer is to data that can't be changed).
Fix up offenders in the python bindings.
* python/generator.py (py_types): Drop useless conversions.
* python/libvirt-override.c (getPyVirTypedParameter)
(setPyVirTypedParameter): Use intended type.
Signed-off-by: Eric Blake <[email protected]>
| generator.py | diffblobblamehistory | |
| libvirt-override.c | diffblobblamehistory |