I'd have no objection to a well-worded warning.
On Mon, Oct 20, 2014 at 4:57 AM, Herbert Valerio Riedel
On 2014-10-19 at 23:24:25 +0200, Edward Kmett wrote:
Malcolm's reasoning matches mine perfectly here.
The instance should not be standard.
It should never go into Prelude.
It standardizes on one of many viable choices.
However, it is one of those things that folks, rightly or wrongly, define fairly often and removing it would increase conflicts among orphans doing the same thing.
I'd strongly advocate holding your nose and leaving it there.
Fair enough. But what about adding a WARNING pragma such as (modulo wording):
module Text.Show.Functions {-# WARNING "Here Be Dragons, RTFM!" -# } where
and adding a bit more documentation pointing out the dos and donts of using that module (like explaining it is not be advisable import that module in a public library package to be placed on Hackage due to the global namespace issue of such instances)
Cheers, hvr