<p dir="ltr"><br>
On Jan 6, 2015 11:47 AM, "Jef Driesen" <<a href="mailto:jef@libdivecomputer.org">jef@libdivecomputer.org</a>> wrote:<br>
><br>
> These functions aren't used anywhere. Is that on purpose, or is there something missing in the patch?</p>
<p dir="ltr">Not missing, no. Just too much cut-and-paste. The case statement with the actual callbacks does the "warning" callbacks twice. The second set should be "alarm" (ie cases 0x0019 and 0x001a).</p>
<p dir="ltr">Why didn't I notice the compiler warning? Hmm. Probably because the build system is so verbose. There is a reason why the kernel (and subsurface) builds try to be quiet. Or maybe the warning flags aren't complete enough. I'm mobile, so can't test.</p>
<p dir="ltr">   Linus</p>