Warn about using Num instance instead of AdditiveGroup, VectorSpace etc

music-suite / music-pitch

This repo has been merged into: https://github.com/music-suite/music-suite

http://music-suite.github.io

BSD 3-Clause "New" or "Revised" License

9 stars 9 forks source link

Warn about using Num instance instead of AdditiveGroup, VectorSpace etc #27

Closed hanshoglund closed 10 years ago

hanshoglund commented 10 years ago

There are a lot of Num instances which only exist for fromInteger.

We want to retain them for fromInteger, but discourage the use of +, * in favour of ^+^ etc. Warn in docs, or consider removing the instance (replace by ad-hoc function Integer -> Pitch etc).