Hi all!

Perhaps I'm missing something but two_hop_covered_count below is used
uninitialized AFAICS.
----  snip  ----
/**
 *This function calculates the mpr neighbors
 *@return nada
 */
void
olsr_calculate_mpr(void)     
{  
  olsr_u16_t                   two_hop_covered_count;
  olsr_u16_t                   two_hop_count;
  int i;

  OLSR_PRINTF(3, "\n**RECALCULATING MPR**\n\n");
  OLSR_PRINTF(0, "two_hop_covered_count=%d\n", two_hop_covered_count);

  olsr_clear_mprs();
  two_hop_count = olsr_calculate_two_hop_neighbors();
  two_hop_covered_count += add_will_always_nodes();
----  snip  ----
According to the CVS, it is there since version 1.1.

Question 1: Andreas, this should probably better be " = ", not?
Question 2: Why doesn't gcc (both 3.4.6 and 4.1.1) detect this as
uninitialized usage?
Should we file a bug report there?

	Bernd
-- 
Firmix Software GmbH                   http://www.firmix.at/
mobil: +43 664 4416156                 fax: +43 1 7890849-55
          Embedded Linux Development and Services




