<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/db8aec33c3c4e2ad76fa3c5a8c74f13af82256d9">https://github.com/macports/macports-legacy-support/commit/db8aec33c3c4e2ad76fa3c5a8c74f13af82256d9</a></p>
<pre style="white-space: pre; background: #F8F8F8"><span style='display:block; white-space:pre;color:#808000;'>commit db8aec33c3c4e2ad76fa3c5a8c74f13af82256d9
</span>Author: Fred Wright <fw@fwright.net>
AuthorDate: Mon Jun 16 19:43:40 2025 -0700
<span style='display:block; white-space:pre;color:#404040;'> test_clocks: Remove unused dstats args.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> The dstats pointer was being passed through/to some functions that
</span><span style='display:block; white-space:pre;color:#404040;'> don't use it. Surprisingly, this generated no warnings.
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> Also fixes a typo in a comment.
</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;'> Builds and passes on all platforms.
</span>---
test/test_clocks.c | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/test/test_clocks.c b/test/test_clocks.c
</span><span style='display:block; white-space:pre;color:#808080;'>index da3def2..0e8643c 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/test/test_clocks.c
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/test/test_clocks.c
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1201,7 +1201,7 @@ sandwich_samples(clock_idx_t clkidx, errinfo_t *ei)
</span> * In addition to the above, the "late" case is considered retriable.
*/
static int
<span style='display:block; white-space:pre;background:#ffe0e0;'>-compare_clocks(clock_idx_t clkidx, errinfo_t *ei, errinfo_t *eir, dstats_t *dp)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+compare_clocks(clock_idx_t clkidx, errinfo_t *ei, errinfo_t *eir)
</span> {
const ns_time_t *refbp = &refnsbuf[0];
const ns_time_t *testbp = nsbufp[clock_types[clkidx]];
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1313,8 +1313,7 @@ static int clock_replay_dual_ns(clock_idx_t clkidx, int quiet);
</span>
static int
check_clock_sandwich(clock_idx_t clkidx,
<span style='display:block; white-space:pre;background:#ffe0e0;'>- errinfo_t *ei, errinfo_t *eir, dstats_t *dp,
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- int quiet, int replay)
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ errinfo_t *ei, errinfo_t *eir, int quiet, int replay)
</span> {
int ret, tries = 0;
useconds_t sleepus = STD_SLEEP_US;
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1335,7 +1334,7 @@ check_clock_sandwich(clock_idx_t clkidx,
</span>
ret = check_samples(clkidx, nsbufp[clock_types[clkidx]], 1, ei);
if (ret) { if (ret < 0) break; continue; }
<span style='display:block; white-space:pre;background:#ffe0e0;'>- ret = compare_clocks(clkidx, ei, eir, dp);
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ ret = compare_clocks(clkidx, ei, eir);
</span> if (ret <= 0) break;
} while (!replay && ++tries < MAX_RETRIES);
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1489,7 +1488,7 @@ report_clock_compare(clock_idx_t clkidx, int dump, int dverbose,
</span>
if (vnq) printf(" Comparing %s to mach_absolute_time\n", name);
<span style='display:block; white-space:pre;background:#ffe0e0;'>- if (check_clock_sandwich(clkidx, &info, &refinfo, &dstats, quiet, replay)) {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if (check_clock_sandwich(clkidx, &info, &refinfo, quiet, replay)) {
</span> if (replay && info.retries < 0) return 0; /* Just skip nonex replay */
if (refinfo.errnum) {
report_clock_err(clock_idx_mach_absolute, &refinfo, 0, verbose);
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1542,7 +1541,7 @@ report_all_clock_compares(int dump, int dverbose, int verbose, int quiet, int re
</span> * "multiply first" approach on PowerPC can be observed.
*
* If and when the overflow occurs from using the "multiply first" approach
<span style='display:block; white-space:pre;background:#ffe0e0;'>- * on PowrerPC, the computed nanosecond value wraps around, after which its
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ * on PowerPC, the computed nanosecond value wraps around, after which its
</span> * value is never more than half the correct value. Thus, it can be detected
* on the basis of a very large negative error in the scale (at least 0.5).
*/
</pre><pre style='margin:0'>
</pre>