aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Robillard <d@drobilla.net>2022-12-15 12:12:47 -0500
committerDavid Robillard <d@drobilla.net>2022-12-15 12:16:36 -0500
commit9dc04ad3c5b62d0018eae14dc177be2993dbcbd4 (patch)
tree6537453f2a6af1956a4712bf9107d4581f924ab3
parentf4fe410e75c0fd3aec2fd6ee20be1571221a1700 (diff)
downloadlv2-9dc04ad3c5b62d0018eae14dc177be2993dbcbd4.tar.xz
Remove double size check entirely
It turns out that, since this header is widely included in dependants, removing the conditional in the previous commit triggers undocumented warnings downstream. Since the check is pretty esoteric anyway, and has been widely included across most of the LV2 ecosystem for a decade and never once come up, I think it's safe to just delete this and be done with the hassle.
-rw-r--r--NEWS5
-rw-r--r--include/lv2/atom/atom.h4
2 files changed, 3 insertions, 6 deletions
diff --git a/NEWS b/NEWS
index c64ff13..d75a2ae 100644
--- a/NEWS
+++ b/NEWS
@@ -3,8 +3,9 @@ lv2 (1.18.11) unstable; urgency=medium
* Allow LV2_SYMBOL_EXPORT to be overridden
* Fix pylint warning in test script
* Override pkg-config dependency within meson
-
- -- David Robillard <d@drobilla.net> Sat, 10 Dec 2022 14:17:42 +0000
+ * Remove troublesome lv2_atom_assert_double_fits_in_64_bits
+
+ -- David Robillard <d@drobilla.net> Thu, 15 Dec 2022 17:15:58 +0000
lv2 (1.18.10) stable; urgency=medium
diff --git a/include/lv2/atom/atom.h b/include/lv2/atom/atom.h
index 59816e5..44f597a 100644
--- a/include/lv2/atom/atom.h
+++ b/include/lv2/atom/atom.h
@@ -62,10 +62,6 @@
extern "C" {
#endif
-// This expression will fail to compile if double does not fit in 64 bits
-typedef char lv2_atom_assert_double_fits_in_64_bits
- [((sizeof(double) <= sizeof(uint64_t)) * 2) - 1];
-
/**
Return a pointer to the contents of an Atom. The "contents" of an atom
is the data past the complete type-specific header.