fix(netwatch): various small improvments

[dignifiedquire]

• 2487d66: The parse_rib loop failed to advance the buffer pointer when encountering a message with a mismatched routing stack version, causing an infinite loop.
• 8785aa1:Iterator::zip silently stopped at the shorter iterator, so an interface gaining or losing addresses was not detected as a change.
• da51515: The comparison only iterated the old state's interfaces, so a newly appeared interface was never reported as a major change.
• 10eb2c8: The Option<bool> pending state allowed a normal network event to overwrite a pending time-jump signal before it was processed.
• f3986e3: Interval::reset_immediately() caused the debounce timer to fire on the next poll, defeating burst coalescing. A one-shot Sleep that extends its deadline on each event now provides a true quiet-period debounce.
• 68a334f: The unregister methods removed the Arc<Callback> from the map before calling CancelMibChangeNotify2, creating a window where Windows could invoke a callback through a dangling pointer.

[github-actions]

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/net-tools/pr/103/docs/net_tools/

Last updated: 2026-02-26T12:19:29Z

[flub]

generally looks good i think. glad to see the 250ms wakeup interval being gone. one nit/question more.

[matheus23]

Generally looks good, but have some nits.

Also - I still don't understand why the debouncing logic needs an interval instead of just a simple sleep that is bounced back via reset?
Wth I swear this wasn't changed before I reviewed!