Re: [PATCH 1/2] factor: add option for printing in x^y format

[Pádraig Brady]

On 04/05/2022 08:28, Rasmus Villemoes wrote:
> On 27/04/2022 19.21, Bernhard Voelker wrote:
> > On 4/27/22 16:14, Pádraig Brady wrote:
> > > On 27/04/2022 11:07, Rasmus Villemoes wrote:
>
> > > Though for interactive use, the new option is a benefit.
> > >
> > > Given the interactive benefit and a previous request to add this functionality¹
> > > I'm tentatively leaning towards adding this.
> >
> > Re. the patch:
> > the implementation is quite clean in 2 places - the LONGINT_OK and the MPZ case 
> > - and
> > doesn't seem to get into the way of any future extensions (null-separated 
> > output, whatever).
> >
> > Therefore, I'm also 70:30 for adding it.
> > Of course, we'd also need a long option and texinfo documentation.
>
> Thanks. I don't care much about what the short option is, but it's quite
> important for my use case that it does have a short form. Given the
> NetBSD precedent [1], I think -h is fine. We can make the long option
> --human-readable or --exponents, I don't have any opinion on that.
>
> Should I respin with -h instead of -e?

I'll apply the attached later.

thanks again,
Pádraig

factor-h.patch
Description: Text Data