On Thu, Dec 7, 2023 at 12:48 AM Miroslav Lichvar <mlich...@redhat.com> wrote:
>
> On Thu, Nov 30, 2023 at 02:23:28PM +1100, patrick.oppenlan...@gmail.com wrote:
> > +/* Leap second database */
> > +struct leapdb {
> > +  long long updated;
> > +  long long expiry;
> > +  size_t len;
> > +  struct leap {
> > +    long long when;
> > +    int tai_offset;
> > +  } leap[];
> > +};
>
> Please avoid saving the data in memory. I think the requested leap can
> be found in one pass over the file. Simpler is better.

Agreed, simple is good. Thanks for the feedback, I'll prepare a v3.

Patrick

> --
> 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.
>

--
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