Re: bgpd: cleanup mrt.c

2023-07-14 Thread Theo Buehler
On Fri, Jul 14, 2023 at 11:43:38AM +0200, Claudio Jeker wrote: > On Thu, Jul 13, 2023 at 11:36:22AM +0200, Theo Buehler wrote: > > On Thu, Jul 13, 2023 at 10:04:33AM +0200, Claudio Jeker wrote: > > > This is a follow-up to use more of the new ibuf API to write the mrt > > > message. > > > > > > T

Re: bgpd: cleanup mrt.c

2023-07-14 Thread Claudio Jeker
On Thu, Jul 13, 2023 at 11:36:22AM +0200, Theo Buehler wrote: > On Thu, Jul 13, 2023 at 10:04:33AM +0200, Claudio Jeker wrote: > > This is a follow-up to use more of the new ibuf API to write the mrt > > message. > > > > This removes all of the DUMP_XYZ macros and replaces them with > > ibuf_add_

Re: bgpd: cleanup mrt.c

2023-07-13 Thread Theo Buehler
On Thu, Jul 13, 2023 at 10:04:33AM +0200, Claudio Jeker wrote: > This is a follow-up to use more of the new ibuf API to write the mrt message. > > This removes all of the DUMP_XYZ macros and replaces them with > ibuf_add_nX() calls. Also unify the error handling by using > goto fail; in all cases

bgpd: cleanup mrt.c

2023-07-13 Thread Claudio Jeker
This is a follow-up to use more of the new ibuf API to write the mrt message. This removes all of the DUMP_XYZ macros and replaces them with ibuf_add_nX() calls. Also unify the error handling by using goto fail; in all cases and use a more generic log_warn() there once. -- :wq Claudio Index: mrt