groff
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH v2] src/: ceil_prime(): Add function to get the lowest prime


From: James K. Lowden
Subject: Re: [PATCH v2] src/: ceil_prime(): Add function to get the lowest prime not smaller than n
Date: Tue, 12 Mar 2024 15:28:22 -0400

On Wed, 13 Mar 2024 06:52:10 -0500
"G. Branden Robinson" <g.branden.robinson@gmail.com> wrote:

> A hash table (or map) of size 1 is not a hash table, it's a list.

A container does not change its type based on its size.  

I don't know what you're trying to fix here, but it sounds
unimportant.  If the user is creating, let us say, unnecessary or
counterproductive overhead by calling for a map of size 1, is that ever
a problem in practice?  I would think the effect is undetectable.  

> The advantage to switching to STL containers for these hash tables
> would be to get groff out of the business of maintaining its own
> implementations in favor of those that have been more rigorously
> tested

100%

--jkl






reply via email to

[Prev in Thread] Current Thread [Next in Thread]