Improve the _repr_ of IntegerRange

[hivert]

The actual printing was discussed on this thread of sage-combinat-devel.

I also took the chance of normalizing the input to improve equality.

Component: combinatorics

Keywords: integer range

Author: Florent Hivert

Reviewer: Mike Hansen

Merged: sage-4.4.4.alpha0

Issue created by migration from https://trac.sagemath.org/ticket/8704

[hivert]

Changed keywords from none to integer range

[hivert]

comment:2

I fixed a doctest failure... All tests should pass now.

[hivert]

comment:3

I fixed the following issues raised by Nicolas on sage-combinat-devel

> >> The output of IntegerRange is much nicer now. I was about to put a
> >> positive review, when I had a last doubt about the consistency between:
> >>
> >>      sage: I = IntegerRange(2,100,5); I
> >>      {2, 7 .. 97}
> >>      sage: I = IntegerRange(54,Infinity,3); I
> >>      {54, 57, ..}
> >>
> >> Should there be a comma in both cases, in none, or is it good as is?
> >
> > I would say {2, 7 .. 97} should be replaced by {2, 7, .., 97} for
> > consistency.

[hivert]

Attachment: trac_8704-integer_range_print-fh.patch.gz

[mwhansen]

Reviewer: Mike Hansen

[mwhansen]

comment:4

Looks good to me.

[mwhansen]

Merged: sage-4.4.4.alpha0