more inlining and a bug fix:
authorBernd Petrovitsch <bernd@firmix.at>
Mon, 5 May 2008 22:22:19 +0000 (00:22 +0200)
committerBernd Petrovitsch <bernd@firmix.at>
Mon, 5 May 2008 22:22:19 +0000 (00:22 +0200)
commitdfcd7feb61dcf5f45bcb6dd67513681cd5d85c5b
tree7cfbb9fe3f3ca121af2e113fc1bd152071994e67
parent1032e750f50fb533cc671b03a170cc4dda337a01
more inlining and a bug fix:
- inlined olsr_bump_routingtree_version()
  and made it a "void" function as no one uses the returned value
- inlined olsr_bool olsr_nh_change()
- inlined olsr_bool olsr_hopcount_change()
- commented out the unused olsr_fib_metric() function
- inlined olsr_get_nh()
- use ip6equal() in avl_comp_ipv6_prefix()
- commented in the - up to now unused - ipcmp() inline funtion
- use ipcmp() in olsr_cmp_rtp() instead of a memcmp()
  Actually I don't know if it buys that much from a performance
  point-of-viw.

The bug fix (at least IMHO):
- The ipcmp() result (which is equivalent to the memcmp() result)
  is compared "< 0" and not "== -1".
  Rationale: memcmp() (and others) return - according to the manual
  page - "== 0", "< 0" or "> 0". So the only 6 valid operations are
  the 6 combinations (and negations) thereof.
  Comparing the result to "== -1" is IMHO plain simply wrong as
  memcmp() may return validly "-2".
src/ipcalc.h
src/routing_table.c
src/routing_table.h