Re: [chrony-dev] [PATCH v2 3/6] leapdb: move source check into separate function

[ Thread Index | Date Index | More chrony.tuxfamily.org/chrony-dev Archives ]


On Thu, Nov 30, 2023 at 02:23:26PM +1100, patrick.oppenlander@xxxxxxxxx wrote:
>  /* ================================================== */
>  
> +typedef NTP_Leap (*GetLeapFn)(time_t when, int *tai_offset);
> +

I don't like function pointers much. More difficult to follow the
code. Can it be replaced with a condition in LDB_GetLeap?

-- 
Miroslav Lichvar


-- 
To unsubscribe email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "unsubscribe" in the subject.
For help email chrony-dev-request@xxxxxxxxxxxxxxxxxxxx with "help" in the subject.
Trouble?  Email listmaster@xxxxxxxxxxxxxxxxxxxx.


Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/