darcs patch: more portabilization
Isaac Dupree
isaacdupree at charter.net
Wed Jan 16 19:16:19 EST 2008
Isaac Dupree wrote:
> Isaac Dupree wrote:
>> oh, darnit, you wanted to panic for invalid FastBools... which seems
>> reasonable and hard to work around.
>
> although if they're really supposed to be Fast, I question putting any
> extra checks than "is it equal to zero or not" for example, even to
> panic. chr# doesn't do checks the way Data.Char.chr does, so my fastChr
> doesn't do checks either, so that my patch would preserve GHC's speed.
> In fact that might make FastBool slower than Bool... maybe FastBool is
> not needed anyway and can be eliminated from the few places it's used?
okay, I'll do #ifdef DEBUG for the FastBool checks and leave it in a
separate module (I will make sure there are no warnings in either case!)
~Isaac
More information about the Cvs-ghc
mailing list