ShortByteString's constructor is exported by Data.ByteString.Short but maybe shouldn't

[sergv]

I was a bit surprised that constructor of ShortByteString, the one showing the unboxed byte array within, is exported by the non-internal module Data.ByteString.Short. For other bytestring types the modules strive to hide internals, especially if internals are not intended to be used directly. It seems that unboxed byte array shouldn't be inteded to be used by ShortByteString clients by default. Perhaps I'm missing something?

[clyring]

Exposing the constructor was intentional. See #313 for some brief discussion.

[sergv]

Makes sense, I'll close this one then.