Re: [PATCH 00/15] lib: Add support for a decimal 0m prefix for numbers

Wolfgang Denk
2021-07-21T07:53:03Z

Hi,

In message <20210720160547.GM9379@bill-the-cat> you wrote:
>
> > So for example (10)123 would mean decimal 123? I don't know how we
> > would parse brackets separately from expressions though.
>
> (123)10 would be "123" in decimal. Which is indeed a mouthful. But it
> would also be generic and (123)16 would be 0x123. So the parsing
> shouldn't be too hard, for most commands. But then yes, expressions
> become quite hard.

Come on, guys, be serious! This is a boot loader. Size matters.

Do we _really_ need all this, and is it worth the code size?

Simon's patches include some cleanup, which probably even reduces
the size, so good.

But whether it's 0m123 or 0t123 or 0!123 or ... is pretty much
irrelevant - chose one symbol, use it, and be done with that.

Best regards,

Wolfgang Denk

--
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd@denx.de
Do you suppose the reason the ends of the `Intel Inside' logo don't
match up is that it was drawn on a Pentium?