[PATCH] D33440: clang-format: better handle statement macros
Francois Ferrand via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Nov 9 05:00:47 PST 2017
Typz added a comment.
In https://reviews.llvm.org/D33440#920205, @djasper wrote:
> Out of curiosity, will this be able to fix the two situations that you get for python extension?
> There, you usually have a PyObject_HEAD with out semicolon in a struct and than a PyObject_HEAD_INIT(..) in a braced init list. More info:
> http://starship.python.net/crew/mwh/toext/node20.html
`PyObject_HEAD` is defined (by defaut) as:
Py_ssize_t ob_refcnt;
PyTypeObject *ob_type;
so declaring it as a statement macro should allow clang-format to process it correctly. (though this is a variable-like macro, so I need to check if this case is supported)
`PyObject_HEAD_INIT` ends with a comma, which is not supported yet. We would need to add another kind of macro to support that case.
https://reviews.llvm.org/D33440
More information about the cfe-commits
mailing list