summaryrefslogtreecommitdiff
path: root/io/test
diff options
context:
space:
mode:
authorStephan Bergmann <sbergman@redhat.com>2019-12-11 16:36:42 +0100
committerStephan Bergmann <sbergman@redhat.com>2019-12-11 19:51:31 +0100
commit2157160b2665de61517973089b3bcd2a59416c1e (patch)
tree935990dffdc44a416b2929b178768de2203c9b96 /io/test
parent775a06361639ffc539fda20d44b9610c98005d9b (diff)
Recursively include unnamed inner classes in "layout heuristic"
This covers both cases where the inner class is defined in the declaration of a non-static data member, as needed by clang-cl for > bridges/source/cpp_uno/msvc_win32_x86-64/except.cxx(799,19): error: unused class member [loplugin:unusedmember] > PVOID pExceptionObject; > ~~~~~~^~~~~~~~~~~~~~~~ > bridges/source/cpp_uno/msvc_win32_x86-64/except.cxx(801,19): error: unused class member [loplugin:unusedmember] > PVOID pThrowImageBase; > ~~~~~~^~~~~~~~~~~~~~~ as well as anonymous structs (even if there appears to be no need for that across the LO code base; they are part of C11 and a widely supported C++ extension). The newly added TODO will eventually need fixing, but I didn't find a way in Clang to tell whether a RecordDecl is defined as part of a FieldDecl, and the issue appears to be of no practical relevance for the current LO code base. Change-Id: I08cecddb9b4a70c3ca480d2d2ab1ea4f198011b2 Reviewed-on: https://gerrit.libreoffice.org/84967 Tested-by: Jenkins Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Diffstat (limited to 'io/test')
0 files changed, 0 insertions, 0 deletions