Adjust show method signature for CoefTable to include MIME"text/plain"

[andreasnoack]

The output is decorated so it shouldn't overload the two argument show method.

Also, have struct definition and show methods next to eachother in the source file.

[nalimilan]

Can you confirm you just changed the signature and that other changes just move the code without modifying it?

[andreasnoack]

Can you confirm you just changed the signature and that other changes just move the code without modifying it?

Actually, I made a single modification. I made the struct immutable. I think the struct migh predate immutable and I doubt you'd ever modify the fields of a Coeftable. Except for that, I just moved code around to have the definitions for the struct next to the struct definition.

[devmotion]

LGTM.

I wonder if a custom show method with a single-line representation should be added?

[andreasnoack]

I wonder if a custom show method with a single-line representation should be added?

I think it is hard to do better than the default one.

[nalimilan]

Maybe it would be safer to make a separate PR that makes the struct immutable, and merge it before the next breaking release?

[andreasnoack]

Fair point. I've factored that change out, so the only change is now the show method signature.