<pre style='margin:0'>
Christopher Nielsen (mascguy) pushed a commit to branch master
in repository macports-legacy-support.
</pre>
<p><a href="https://github.com/macports/macports-legacy-support/commit/1849c3ac55df0862a2f7c3f11c83235dd77b23a6">https://github.com/macports/macports-legacy-support/commit/1849c3ac55df0862a2f7c3f11c83235dd77b23a6</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit 1849c3ac55df0862a2f7c3f11c83235dd77b23a6
</span>Author: Fred Wright <fw@fwright.net>
AuthorDate: Thu Jun 6 19:59:35 2024 -0700
<span style='display:block; white-space:pre;color:#404040;'> sincos: Add missing include.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> The comment for including the main MacportsLegacySupport.h header was
</span><span style='display:block; white-space:pre;color:#404040;'> not actually followed by the include, though it was using
</span><span style='display:block; white-space:pre;color:#404040;'> __MP_LEGACY_SUPPORT_COSSIN__. It's unclear how it ever worked, though
</span><span style='display:block; white-space:pre;color:#404040;'> the tests pass both before and after the fix.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> This appears to have been true ever since the code was moved here from
</span><span style='display:block; white-space:pre;color:#404040;'> a header-based version, which also lacked the main include.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> TESTED:
</span><span style='display:block; white-space:pre;color:#404040;'> Tested on 10.4-10.5 ppc, 10.5-10.6 ppc (i386 Rosetta), 10.4-10.6 i386,
</span><span style='display:block; white-space:pre;color:#404040;'> 10.5-12.x x86_64, and 11.x-14.x arm64. Tested against all 10.4-14.x
</span><span style='display:block; white-space:pre;color:#404040;'> SDKs, using the headerinfo manual test (from a subsequent commit).
</span><span style='display:block; white-space:pre;color:#404040;'> All target/SDK combinations where the SDK supports the CPU
</span><span style='display:block; white-space:pre;color:#404040;'> architecture work correctly, including all SDKs on Intel
</span><span style='display:block; white-space:pre;color:#404040;'> architectures. Only 10.4-10.6 SDKs support ppc, and only 11.x+ SDKs
</span><span style='display:block; white-space:pre;color:#404040;'> support arm64.
</span>---
src/sincos.c | 2 ++
1 file changed, 2 insertions(+)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/src/sincos.c b/src/sincos.c
</span><span style='display:block; white-space:pre;color:#808080;'>index 903dedc..315ecbf 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/src/sincos.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/src/sincos.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -15,6 +15,8 @@
</span> */
/* MP support header */
<span style='display:block; white-space:pre;background:#e0ffe0;'>+#include "MacportsLegacySupport.h"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span> #include <math.h>
#if __MP_LEGACY_SUPPORT_COSSIN__
</pre><pre style='margin:0'>
</pre>