Skip to content

Remove instance Outputable Char, minor cleanup

Krzysztof Gogolewski requested to merge wip/cleanup-outputable into master

This removes the Outputable Char instance. Because strings are lists of characters, an SDoc such as ppr "hello" was rendered as "h,e,l,l,o" in several places. Instead, users should use char or text directly.

Minor cleanup in the first commit; replace renderWithContext with showSDocOneLine where the string is a one-liner. This documents the intent better and is more efficient.

Remove unused polyPatSig.

Edited by Krzysztof Gogolewski

Merge request reports