clang Warning: Padding Class...

[Jean-Marc Lasgouttes]

It is a good idea to have correct padding by reordering members, even if the memory gains are minimal for buffer::impl . Do you have that for other classes ?

Note that padding needs are not the same for 32bist and 64bits, and probably not the same for Linux and windows.

JMarc

Le 19 février 2020 21:24:15 GMT+01:00, Richard Kimberly Heck <rikiheck at lyx.org> a écrit :
>I'm seeing a lot of warnings in QtCreator of the form:
>
>Buffer.cpp:219:11: warning: padding class 'lyx::Buffer::Impl' with 3
>bytes to align 'filename'
>
>I take it that this is because there were five bools right before this
>declaration. Do we care about this?