diff options
author | 2011-05-30 09:31:50 -0400 | |
---|---|---|
committer | 2011-05-30 09:31:50 -0400 | |
commit | ce09df0fd2af2465c1df0fbeacf0cb6a07cd2add (patch) | |
tree | fbcc9d6f1fc4ec715af7af96f489195487cc4fc0 | |
parent | 8fa4027917a95498a0b354e614aaad7e0210e2d7 (diff) |
Fix bug with string literals in unAs
-rw-r--r-- | src/c/urweb.c | 22 | ||||
-rw-r--r-- | src/mono_opt.sml | 2 | ||||
-rw-r--r-- | tests/t_t.ur | 5 |
3 files changed, 12 insertions, 17 deletions
diff --git a/src/c/urweb.c b/src/c/urweb.c index 1edf47e0..865ce024 100644 --- a/src/c/urweb.c +++ b/src/c/urweb.c @@ -3562,7 +3562,7 @@ __attribute__((noreturn)) void uw_redirect(uw_context ctx, uw_Basis_string url) } uw_Basis_string uw_Basis_unAs(uw_context ctx, uw_Basis_string s) { - uw_Basis_string r = uw_malloc(ctx, strlen(s) + 1); + uw_Basis_string ret = uw_malloc(ctx, strlen(s) + 1), r = ret; for (; *s; ++s) { if (s[0] == '\'') { @@ -3572,27 +3572,21 @@ uw_Basis_string uw_Basis_unAs(uw_context ctx, uw_Basis_string s) { *r++ = '\''; break; } else if (s[0] == '\\') { - if (s[1] == '\\') { - *r++ = '\\'; - *r++ = '\\'; - ++s; - } else if (s[1] == '\'') { - *r++ = '\\'; - *r++ = '\''; - ++s; - } else - *r++ = '\''; + *r++ = '\\'; + *r++ = s[1]; + ++s; } else *r++ = s[0]; } if (*s == 0) break; - } else if (s[0] == 'T' && s[1] == '.') - ++s; + } else if (s[0] == 'T' && s[1] == '_' && s[2] == 'T' && s[3] == '.') + s += 3; else *r++ = s[0]; } - return r; + *r = 0; + return ret; } uw_Basis_string uw_Basis_mstrcat(uw_context ctx, ...) { diff --git a/src/mono_opt.sml b/src/mono_opt.sml index ffadeec4..5abbf900 100644 --- a/src/mono_opt.sml +++ b/src/mono_opt.sml @@ -102,7 +102,7 @@ fun unAs s = fun doChars (cs, acc) = case cs of #"T" :: #"_" :: #"T" :: #"." :: cs => doChars (cs, acc) - | #"'" :: cs => doString (cs, acc) + | #"'" :: cs => doString (cs, #"'" :: acc) | ch :: cs => doChars (cs, ch :: acc) | [] => String.implode (rev acc) diff --git a/tests/t_t.ur b/tests/t_t.ur index 9e38a30d..a20e45f1 100644 --- a/tests/t_t.ur +++ b/tests/t_t.ur @@ -1,3 +1,4 @@ -table t : {A : int} +table t : {A : int, B : string} -task initialize = fn () => dml (UPDATE t SET A = A + 1 WHERE TRUE) +task initialize = fn () => dml (UPDATE t SET A = A + 1 WHERE TRUE); + dml (UPDATE t SET B = 'q' WHERE TRUE) |