Does anybody use it?
Do any distros build with it enabled?
Should we add an "#warn untested" to the code?


static void
leap_smear_add_offs(l_fp *t) {
        t += leap_smear.offset;
}

clang 13 points out:
../../ntpd/ntp_proto.c:2210:27: warning: parameter 't' set but not used 
[-Wunused-but-set-parameter]

[If you don't see the bug right away, look harder.]

My normal testing doesn't include smearing.
This is tickled by option-tester which includes --enable-leap-smear


-- 
These are my opinions.  I hate spam.



_______________________________________________
devel mailing list
devel@ntpsec.org
https://lists.ntpsec.org/mailman/listinfo/devel

Reply via email to