Re: [PATCH v5 2/3] cmdline: add floating point support

[Stephen Hemminger <stephen@networkplumber.org>]

On Fri, 23 May 2025 12:21:15 +0200
"Burakov, Anatoly" <anatoly.burakov@intel.com> wrote:

> On 5/21/2025 4:05 PM, Stephen Hemminger wrote:
> > On Wed,  7 May 2025 16:22:11 +0100
> > Anatoly Burakov <anatoly.burakov@intel.com> wrote:
> >   
> >> +NUMERIC_TYPES = {
> >> +    "UINT8": "uint8_t",
> >> +    "UINT16": "uint16_t",
> >> +    "UINT32": "uint32_t",
> >> +    "UINT64": "uint64_t",
> >> +    "INT8": "int8_t",
> >> +    "INT16": "int16_t",
> >> +    "INT32": "int32_t",
> >> +    "INT64": "int64_t",
> >> +    "FLOAT_SINGLE": "float",
> >> +    "FLOAT_DOUBLE": "double",
> >> +}
> >>     
> > 
> > Is it really worth having both single and double versions.
> > Seems like unnecessary complexity at this point.  
> 
> I'd argue the same about the various int types. I was just being 
> consistent. I can remove the single (or double) if that's desired.


The int types have different ranges, but all float values have same
range, just different number of bits. For now choose one version
for floating point.