On 11/03/2019 06:37, Alex Peshkoff via Firebird-devel wrote:
> On 3/10/19 1:56 AM, Adriano dos Santos Fernandes wrote:
>> Here is a patch that changes FB_MESSAGE alignment to match Firebird
>> definitions.
>>
>> I have purposely didn't tested for C++11 to use alignas. I think it's
>> better to have an error when problematic types are used and this keyword
>> is not available.
>>
>> There is a question about others 32-bit non-Windows/non-Linux.
>>
>> That code assumes only Linux is problematic now, but maybe all
>> non-Windows are?
>
> For non-windows you may use configure to check for actual alignment.
> Unfortunately does not guarantee what will happen when other compiler
> is used on that platform.
>
We are talking about public header files, to be used by applications.

Using configure here means the distributes include files should be
different between platforms (not good) or ineffective (#ifdef
NEVER_DEFINED_MACRO by user application).


Adriano



Firebird-Devel mailing list, web interface at 
https://lists.sourceforge.net/lists/listinfo/firebird-devel

Reply via email to