cvs commit: fptools/libraries/network/Network URI.hs

Sven Panne Sven.Panne at aedion.de
Thu Oct 28 16:42:40 EDT 2004


Graham Klyne wrote:
> Oops, well spotted, thanks.  That's another small change I made 
> recently.  I should have mentioned in the CVS comment, but forgot, that 
> I changed the order of parameters to escapeString.  Does anyone have any 
> objection?

Yes, I do! :-) It breaks old programs and introduces just another #ifdef...

> [[
> diff -r1.13 -r1.12
> 95c95
> <     , escapeString              -- :: (Char->Bool) -> String -> String
> ---
>  >     , escapeString              -- :: String -> (Char->Bool) -> String
> ]]
> 
> (Rationale:  the revised parameter order makes it easier to create 
> specialized string-escape functions by currying with the 
> escape-character-testing function.)

I really like the new order much more and I've never understood the old one,
to be honest. Nevertheless, I think we should keep the old version (deprecated)
and be creative about a new name for the version with swapped parameters (incl.
escapeChar). That's library business...

Cheers,
    S.


More information about the Cvs-libraries mailing list