On Mon, Sep 30, 2013 at 09:27:31PM +0300, Sergey Popovich wrote:
* Remove duplicate f_eval_asn() declaration: already in filter/filter.h
* Move tree_compare() from filter/filter.c to filter/tree.c and make it static as it only used in qsort(). Also remove definition from filter/filter.h.
* Make printing of zero prefix in trie_print() at filter/trie.c to print AFI specific value.
Applied, thanks.
* Move pm_format() from filter/filter.c to nest/a-path.c where other pm_* functions reside and also similar *_set_format() family already resides in nest/a-set.c.
* Move tree_node_print() and tree_print() from filter/filter.c to filter/tree.c Make tree_node_print() static. Add global comment to tree_print() and declare it in filter.h. This is similar to trie_print() from filter/trie.c
This is a good idea, but i have currently some significant changes to these functions in another branch, So i would postpone this to avoid unnecessary conflicts.
* Inline fprefix_get_bounds() from filter/filter.c in trie_add_prefix() at filter/filter.h as there is only one place where this code used.
I don't think this is good idea. Even if fprefix_get_bounds() should be inlined, then it should be done by converting it to static inline function to keep clean semantic separation. -- Elen sila lumenn' omentielvo Ondrej 'SanTiago' Zajicek (email: santiago@crfreenet.org) OpenPGP encrypted e-mails preferred (KeyID 0x11DEADC3, wwwkeys.pgp.net) "To err is human -- to blame it on a computer is even more so."