On Thu, Nov 30, 2023 at 02:23:24PM +1100, patrick.oppenlan...@gmail.com wrote:
> +++ b/leapdb.c
> @@ -0,0 +1,147 @@
> +/*
> +  chronyd/chronyc - Programs for keeping computer clocks accurate.
> +
> + **********************************************************************
> + * Copyright (C) Patrick Oppenlander 2023

When code is moved, the copyright should be assigned to the author of
the code as reported by git blame.

> +void
> +LDB_Finalise(void)
> +{
> +  /* nothing to do */

Just a nit, comments start with capital letter. 

> --- a/main.c
> +++ b/main.c
> @@ -57,6 +57,7 @@
>  #include "smooth.h"
>  #include "tempcomp.h"
>  #include "util.h"
> +#include "leapdb.h"

Try to keep the inclusion in alphabetical order.

> +++ b/reference.c
> @@ -36,6 +36,7 @@
>  #include "logging.h"
>  #include "local.h"
>  #include "sched.h"
> +#include "leapdb.h"

Same here.

-- 
Miroslav Lichvar


-- 
To unsubscribe email chrony-dev-requ...@chrony.tuxfamily.org with "unsubscribe" 
in the subject.
For help email chrony-dev-requ...@chrony.tuxfamily.org with "help" in the 
subject.
Trouble?  Email listmas...@chrony.tuxfamily.org.

Reply via email to