Re: [MLton-commit] r7223

View: New views
1 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: [MLton-commit] r7223

by Matthew Fluet-5 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Sat, 5 Sep 2009, Wesley Terpstra wrote:
> Where there is a "to", there should be a "from".
>
> Converting an intinf to/from a byte-stream is a pretty important operation
> in cryptography. The basis library doesn't provide a way to convert to bytes
> aside from >> and WordX.fromLargeInt. It is not possible to convert an integer
> to bytes in linear time with these operations. Therefore, this patch extends
> the MLton extension to converts IntInf to/from it's underlying representation.

Looks good, especially with the checking for valid representations. Don't
forget to update  http://mlton.org/MLtonIntInf  with signature changes.

_______________________________________________
MLton mailing list
MLton@...
http://mlton.org/mailman/listinfo/mlton