# HG changeset patch # User Adam Chlipala # Date 1220804146 14400 # Node ID 6e665c7c96f6bc3c957db3e2543f517ffa1958f1 # Parent 550100a44ccab4344258132bb1f77da3790269b8 Error-parsing ints diff -r 550100a44cca -r 6e665c7c96f6 include/urweb.h --- a/include/urweb.h Sun Sep 07 11:56:53 2008 -0400 +++ b/include/urweb.h Sun Sep 07 12:15:46 2008 -0400 @@ -18,7 +18,6 @@ failure_kind lw_begin(lw_context, char *path); __attribute__((noreturn)) void lw_error(lw_context, failure_kind, const char *fmt, ...); -__attribute__((noreturn)) void lw_Basis_error(lw_context, lw_Basis_string); char *lw_error_message(lw_context); void *lw_malloc(lw_context, size_t); @@ -81,3 +80,5 @@ lw_Basis_int *lw_Basis_stringToInt(lw_context, lw_Basis_string); lw_Basis_float *lw_Basis_stringToFloat(lw_context, lw_Basis_string); lw_Basis_bool *lw_Basis_stringToBool(lw_context, lw_Basis_string); + +lw_Basis_int lw_Basis_stringToInt_error(lw_context, lw_Basis_string); diff -r 550100a44cca -r 6e665c7c96f6 lib/basis.urs --- a/lib/basis.urs Sun Sep 07 11:56:53 2008 -0400 +++ b/lib/basis.urs Sun Sep 07 12:15:46 2008 -0400 @@ -32,6 +32,8 @@ class read val read : t ::: Type -> read t -> string -> option t +val readError : t ::: Type -> read t -> string -> t +(* [readError] calls [error] if the input is malformed. *) val read_int : read int val read_float : read float val read_string : read string diff -r 550100a44cca -r 6e665c7c96f6 src/c/urweb.c --- a/src/c/urweb.c Sun Sep 07 11:56:53 2008 -0400 +++ b/src/c/urweb.c Sun Sep 07 12:15:46 2008 -0400 @@ -105,10 +105,6 @@ longjmp(ctx->jmp_buf, fk); } -__attribute__((noreturn)) void lw_Basis_error(lw_context ctx, const char *s) { - lw_error(ctx, FATAL, s); -} - char *lw_error_message(lw_context ctx) { return ctx->error_message; } @@ -793,3 +789,13 @@ else return NULL; } + +lw_Basis_int lw_Basis_stringToInt_error(lw_context ctx, lw_Basis_string s) { + char *endptr; + lw_Basis_int n = strtoll(s, &endptr, 10); + + if (*s != '\0' && *endptr == '\0') + return n; + else + lw_error(ctx, FATAL, "Can't parse int: %s", s); +} diff -r 550100a44cca -r 6e665c7c96f6 src/cjr_print.sml --- a/src/cjr_print.sml Sun Sep 07 11:56:53 2008 -0400 +++ b/src/cjr_print.sml Sun Sep 07 12:15:46 2008 -0400 @@ -554,7 +554,9 @@ space, string "tmp;", newline, - string "lw_Basis_error(ctx, ", + string "lw_error(ctx, FATAL, \"", + string (ErrorMsg.spanToString loc), + string ": %s\", ", p_exp env e, string ");", newline, diff -r 550100a44cca -r 6e665c7c96f6 src/mono_print.sml --- a/src/mono_print.sml Sun Sep 07 11:56:53 2008 -0400 +++ b/src/mono_print.sml Sun Sep 07 12:15:46 2008 -0400 @@ -62,11 +62,9 @@ string (#1 (E.lookupDatatype env n))) handle E.UnboundNamed _ => string ("UNBOUND_DATATYPE_" ^ Int.toString n)) | TFfi (m, x) => box [string "FFI(", string m, string ".", string x, string ")"] - | TOption t => - (case #1 t of - TDatatype _ => p_typ env t - | TFfi ("Basis", "string") => p_typ env t - | _ => box [p_typ env t, string "*"]) + | TOption t => box [string "option(", + p_typ env t, + string ")"] and p_typ env = p_typ' false env diff -r 550100a44cca -r 6e665c7c96f6 src/monoize.sml --- a/src/monoize.sml Sun Sep 07 11:56:53 2008 -0400 +++ b/src/monoize.sml Sun Sep 07 12:15:46 2008 -0400 @@ -64,6 +64,15 @@ | _ => poly () end +fun readType' (t, loc) = (L'.TFun ((L'.TFfi ("Basis", "string"), loc), + (L'.TOption t, loc)), loc) +fun readErrType (t, loc) = (L'.TFun ((L'.TFfi ("Basis", "string"), loc), + t), loc) +fun readType (t, loc) = + (L'.TRecord [("Read", readType' (t, loc)), + ("ReadError", readErrType (t, loc))], + loc) + fun monoType env = let fun mt env dtmap (all as (c, loc)) = @@ -86,8 +95,7 @@ | L.CApp ((L.CFfi ("Basis", "show"), _), t) => (L'.TFun (mt env dtmap t, (L'.TFfi ("Basis", "string"), loc)), loc) | L.CApp ((L.CFfi ("Basis", "read"), _), t) => - (L'.TFun ((L'.TFfi ("Basis", "string"), loc), - (L'.TOption (mt env dtmap t), loc)), loc) + readType (mt env dtmap t, loc) | L.CApp ((L.CApp ((L.CApp ((L.CFfi ("Basis", "xml"), _), _), _), _), _), _) => (L'.TFfi ("Basis", "string"), loc) @@ -498,22 +506,53 @@ val t = monoType env t val s = (L'.TFfi ("Basis", "string"), loc) in - ((L'.EAbs ("f", (L'.TFun (t, s), loc), (L'.TFun (t, s), loc), - (L'.ERel 0, loc)), loc), fm) + ((L'.EAbs ("f", readType (t, loc), readType' (t, loc), + (L'.EField ((L'.ERel 0, loc), "Read"), loc)), loc), fm) + end + | L.ECApp ((L.EFfi ("Basis", "readError"), _), t) => + let + val t = monoType env t + val s = (L'.TFfi ("Basis", "string"), loc) + in + ((L'.EAbs ("f", readType (t, loc), readErrType (t, loc), + (L'.EField ((L'.ERel 0, loc), "ReadError"), loc)), loc), fm) end | L.EFfi ("Basis", "read_int") => - ((L'.EFfi ("Basis", "stringToInt"), loc), fm) + let + val t = (L'.TFfi ("Basis", "int"), loc) + in + ((L'.ERecord [("Read", (L'.EFfi ("Basis", "stringToInt"), loc), readType' (t, loc)), + ("ReadError", (L'.EFfi ("Basis", "stringToInt_error"), loc), readErrType (t, loc))], + loc), + fm) + end | L.EFfi ("Basis", "read_float") => - ((L'.EFfi ("Basis", "stringToFloat"), loc), fm) + let + val t = (L'.TFfi ("Basis", "float"), loc) + in + ((L'.ERecord [("Read", (L'.EFfi ("Basis", "stringToFloat"), loc), readType' (t, loc)), + ("ReadError", (L'.EFfi ("Basis", "stringToFloat_error"), loc), readErrType (t, loc))], + loc), + fm) + end | L.EFfi ("Basis", "read_string") => let val s = (L'.TFfi ("Basis", "string"), loc) in - ((L'.EAbs ("s", s, (L'.TOption s, loc), - (L'.ESome (s, (L'.ERel 0, loc)), loc)), loc), fm) + ((L'.ERecord [("Read", (L'.EAbs ("s", s, (L'.TOption s, loc), + (L'.ESome (s, (L'.ERel 0, loc)), loc)), loc), readType' (s, loc)), + ("ReadError", (L'.EAbs ("s", s, s, (L'.ERel 0, loc)), loc), readErrType (s, loc))], loc), + fm) end | L.EFfi ("Basis", "read_bool") => - ((L'.EFfi ("Basis", "stringToBool"), loc), fm) + let + val t = (L'.TFfi ("Basis", "bool"), loc) + in + ((L'.ERecord [("Read", (L'.EFfi ("Basis", "stringToBool"), loc), readType' (t, loc)), + ("ReadError", (L'.EFfi ("Basis", "stringToBool_error"), loc), readErrType (t, loc))], + loc), + fm) + end | L.ECApp ((L.EFfi ("Basis", "return"), _), t) => let diff -r 550100a44cca -r 6e665c7c96f6 tests/fromStringErr.ur --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/fromStringErr.ur Sun Sep 07 12:15:46 2008 -0400 @@ -0,0 +1,3 @@ +fun main () : transaction page = return + 3 = {cdata (show _ (readError _ "3" : int))}
+ diff -r 550100a44cca -r 6e665c7c96f6 tests/fromStringErr.urp --- /dev/null Thu Jan 01 00:00:00 1970 +0000 +++ b/tests/fromStringErr.urp Sun Sep 07 12:15:46 2008 -0400 @@ -0,0 +1,5 @@ +debug +database dbname=test +exe /tmp/webapp + +fromStringErr