Re: rpki-client: factor filename extension parsing into a function

2022-01-22 Thread Claudio Jeker
On Fri, Jan 21, 2022 at 01:36:30PM -0700, Bob Beck wrote: > > I like that.. LGTM > > ok beck@ Ditto > > On Fri, Jan 21, 2022 at 08:37:27PM +0100, Theo Buehler wrote: > > > Lets start with that and optimize this in tree. I think we can rename the > > > function to something like rtype_from_mft

Re: rpki-client: factor filename extension parsing into a function

2022-01-21 Thread Bob Beck
I like that.. LGTM ok beck@ On Fri, Jan 21, 2022 at 08:37:27PM +0100, Theo Buehler wrote: > > Lets start with that and optimize this in tree. I think we can rename the > > function to something like rtype_from_mftfile(). In that case I would move > > the function as well... > > Like this? >

Re: rpki-client: factor filename extension parsing into a function

2022-01-21 Thread Theo Buehler
> Lets start with that and optimize this in tree. I think we can rename the > function to something like rtype_from_mftfile(). In that case I would move > the function as well... Like this? Index: extern.h === RCS file: /cvs/src/usr.

Re: rpki-client: factor filename extension parsing into a function

2022-01-21 Thread Claudio Jeker
On Fri, Jan 21, 2022 at 04:49:47PM +0100, Theo Buehler wrote: > On Fri, Jan 21, 2022 at 02:58:57PM +0100, Claudio Jeker wrote: > > On Wed, Jan 19, 2022 at 06:01:38PM +0100, Theo Buehler wrote: > > > Not sure if it is that much of a win, but it saves some repetition and > > > makes sure we don't for

Re: rpki-client: factor filename extension parsing into a function

2022-01-21 Thread Theo Buehler
On Fri, Jan 21, 2022 at 02:58:57PM +0100, Claudio Jeker wrote: > On Wed, Jan 19, 2022 at 06:01:38PM +0100, Theo Buehler wrote: > > Not sure if it is that much of a win, but it saves some repetition and > > makes sure we don't forget checking the file name to be longer than 4 > > another time (misse

Re: rpki-client: factor filename extension parsing into a function

2022-01-21 Thread Claudio Jeker
On Wed, Jan 19, 2022 at 06:01:38PM +0100, Theo Buehler wrote: > Not sure if it is that much of a win, but it saves some repetition and > makes sure we don't forget checking the file name to be longer than 4 > another time (missed on review in main() and proc_parser_file()). I like the diff. It is

rpki-client: factor filename extension parsing into a function

2022-01-19 Thread Theo Buehler
Not sure if it is that much of a win, but it saves some repetition and makes sure we don't forget checking the file name to be longer than 4 another time (missed on review in main() and proc_parser_file()). Index: extern.h === RCS fil