Debug/Display for numeric types about subspace HOT 5 CLOSED

Adding support for fmtlib here: #244

There is so much to re-implement for display and fmt is high quality and widely used. I think it's the right path, so adding support for formatting of each type.

I need to look further into:

• display vs debug printing
• printing types without a formatter as a string of bytes
• so that Option<unknown> can print Some(...bytes...)
• add a PrintTo() for anything that is fmtlib formattable.
• maybe provide a stream impl for anything fmtlib formattable as well in a separate compat header.

from subspace.

There is no debug vs display differentiation in fmtlib.

We could add to parse() to look for a ? formatting character so you could write {:?} and get different behaviour. At the moment there's nothing that I would print differently and you don't get the distinction of a type being Debug but not Display. It would have to be Display in order to maybe also be Debug. In which case it's not worth doing anything with until we have types we want to print differently in a debug mode than in a "display" mode. But then.. those are normally types that are only Debug and in this case they'd have to be Display and just the user would not get nice stuff from them. Unless Debug was a long multiline thing and Display was a short thing? idk I don't see it yet, we can consider using the ? formatting character if we need it on a per-type basis or for all subspace types.

from subspace.

subspace/string/compat_stream.h will allow streaming of any (public) subspace type that is formattable.

from subspace.

For some reason the stream operator is not being picked up and used by GTest's ADL mechanisms on MSVC and it's breaking my brain. The decltype stuff for SFINAE all resolves when I reproduce it myself. Not sure if this is a blocker issue or what.

OK it was caching problems due to one TU seeing the operator<< template and one TU not, and compilers cacheing the first resulting GTest template for that type. Resolved by putting the operator<< template in prelude.h which it can do with just iosfwd.

Since we have operator<< there is no need for PrintTo.

from subspace.

I think #244 is sufficient to close this issue.

from subspace.