Re: nitpick about useless floating point division in gimme_edge_table

2019-05-23 Thread Tom Lane
Mark Dilger writes: > Hackers, > The return value of gimme_edge_table is not used anywhere in the > core code, so far as I can see. But the value is computed as > /* return average number of edges per index */ > return ((float) (edge_total * 2) / (float) num_gene); > which involves some flo

nitpick about useless floating point division in gimme_edge_table

2019-05-22 Thread Mark Dilger
Hackers, The return value of gimme_edge_table is not used anywhere in the core code, so far as I can see. But the value is computed as /* return average number of edges per index */ return ((float) (edge_total * 2) / (float) num_gene); which involves some floating point math. I'm not sure