expose long double in _ffi (which is _rawffi.alt) and ctypes

[gitlab-importer]

In GitLab by @mattip on Oct 4, 2020, 00:26

Support longdouble in _rawffi.alt, which will make it available to ctypes

[gitlab-importer]

In GitLab by @arigo on Oct 4, 2020, 08:50

Class LongDoubleArg still contains a raise NotImplementedError that looks like it can be reached at runtime.

[gitlab-importer]

In GitLab by @mattip on Oct 4, 2020, 09:30

How can I write a test that would hit that code path? Wouldn't we need a space.w_longdouble?

[gitlab-importer]

In GitLab by @mattip on Oct 4, 2020, 09:34

added 1 commit

• 65877468 - add assert, but why is it needed?

Compare with previous version

[gitlab-importer]

In GitLab by @mattip on Oct 4, 2020, 23:33

added 1 commit

• ab304eea - fix check

Compare with previous version

[gitlab-importer]

In GitLab by @mattip on Oct 7, 2020, 11:23

Translations and own tests are passing for windows and linux 64

[gitlab-importer]

In GitLab by @mattip on Oct 7, 2020, 13:20

.. and macos too (clang). I think that class can only be reached if there is a space.w_longdouble

[gitlab-importer]

In GitLab by @arigo on Oct 7, 2020, 17:57

OK, but then why is there a class LongDoubleArg at all, if the TYPE can never be rffi.LONGDOUBLE? That is unreachable and untested code, and so it shouldn't be added at all, no?

[mattip]

Closing as "not needed"