Changeset: 4549df47da9e for MonetDB
URL: http://dev.monetdb.org/hg/MonetDB?cmd=changeset;node=4549df47da9e
Modified Files:
        MonetDB5/src/mal/Tests/All
        MonetDB5/src/mal/Tests/tst054.mal
        MonetDB5/src/mal/Tests/tst054.stable.err
        MonetDB5/src/mal/Tests/tst054.stable.out
        MonetDB5/src/mal/mal_parser.mx
Branch: default
Log Message:

Handle parsing error
Don't store an incomplete instruction in the MAL block.


diffs (146 lines):

diff -r ea244b45d593 -r 4549df47da9e MonetDB5/src/mal/Tests/All
--- a/MonetDB5/src/mal/Tests/All        Fri May 21 10:48:32 2010 +0200
+++ b/MonetDB5/src/mal/Tests/All        Fri May 21 11:13:41 2010 +0200
@@ -49,6 +49,7 @@
 tst051
 tst052
 tst053
+tst054
 tst055
 tst056
 tst057
diff -r ea244b45d593 -r 4549df47da9e MonetDB5/src/mal/Tests/tst054.mal
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/MonetDB5/src/mal/Tests/tst054.mal Fri May 21 11:13:41 2010 +0200
@@ -0,0 +1,8 @@
+function z()(
+:int,:int,:int,:int,:int,:int,:int,:int,:int,:int,
+:int,:int,:int,:int,:int,:int,:int,:int,:int,:int,
+:int,:int,:int,:int,:int,:int,:int,:int,:int,:int);
+io.print("hello");
+return (1,2,3,4,5,6,7,8,9,0,1,2,3,4,5,6,7,8,9,0,1,2,3,4,5,6,7,8,9,0);
+end z;
+z();
diff -r ea244b45d593 -r 4549df47da9e MonetDB5/src/mal/Tests/tst054.stable.err
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/MonetDB5/src/mal/Tests/tst054.stable.err  Fri May 21 11:13:41 2010 +0200
@@ -0,0 +1,72 @@
+stderr of test 'tst054` in directory 'src/mal` itself:
+
+
+# 11:11:08 >  
+# 11:11:08 >   mserver5 "--config=/ufs/mk/current//Linux/etc/monetdb5.conf" 
--debug=10 --set gdk_nr_threads=0 --set 
"monet_mod_path=/ufs/mk/current//Linux/lib/MonetDB5:/ufs/mk/current//Linux/lib/MonetDB5/lib:/ufs/mk/current//Linux/lib/MonetDB5/bin"
 --set "gdk_dbfarm=/ufs/mk/current//Linux/var/MonetDB5/dbfarm"  --set 
mapi_open=true --set xrpc_open=true --set mapi_port=31658 --set xrpc_port=44186 
--set monet_prompt= --set mal_listing=2 --trace  --dbname=mTests_src_mal  
tst054.mal
+# 11:11:08 >  
+
+# builtin opt  gdk_arch = 64bitx86_64-unknown-linux-gnu
+# builtin opt  gdk_version = 1.39.0
+# builtin opt  prefix = /ufs/mk/current//Linux
+# builtin opt  exec_prefix = ${prefix}
+# builtin opt  gdk_dbname = demo
+# builtin opt  gdk_dbfarm = ${prefix}/var/MonetDB/dbfarm
+# builtin opt  gdk_debug = 0
+# builtin opt  gdk_alloc_map = no
+# builtin opt  gdk_vmtrim = yes
+# builtin opt  monet_admin = adm
+# builtin opt  monet_prompt = >
+# builtin opt  monet_welcome = yes
+# builtin opt  monet_mod_path = ${exec_prefix}/lib/MonetDB
+# builtin opt  monet_daemon = no
+# builtin opt  host = localhost
+# builtin opt  mapi_port = 50000
+# builtin opt  mapi_clients = 2
+# builtin opt  mapi_open = false
+# builtin opt  mapi_autosense = false
+# builtin opt  sql_debug = 0
+# builtin opt  standoff_ns = 
+# builtin opt  standoff_start = start
+# builtin opt  standoff_end = end
+# config opt   prefix = /ufs/mk/current//Linux
+# config opt   config = ${prefix}/etc/monetdb5.conf
+# config opt   prefix = /ufs/mk/current//Linux
+# config opt   exec_prefix = ${prefix}
+# config opt   gdk_dbfarm = ${prefix}/var/MonetDB5/dbfarm
+# config opt   monet_mod_path = 
${exec_prefix}/lib/MonetDB5:${exec_prefix}/lib/MonetDB5/lib:${exec_prefix}/lib/MonetDB5/bin
+# config opt   mero_pidfile = ${prefix}/var/run/MonetDB/merovingian.pid
+# config opt   mero_controlport = 50001
+# config opt   sql_optimizer = default_pipe
+# config opt   minimal_pipe = inline,remap,deadcode,multiplex,garbageCollector
+# config opt   default_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mitosis,mergetable,deadcode,commonTerms,joinPath,reorder,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   nov2009_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   replication_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,history,replication,multiplex,garbageCollector
+# config opt   accumulator_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,accumulators,dataflow,history,multiplex,garbageCollector
+# config opt   recycler_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,deadcode,constants,commonTerms,joinPath,deadcode,recycle,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   cracker_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,selcrack,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   sidcrack_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,sidcrack,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   datacell_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,deadcode,constants,commonTerms,joinPath,datacell,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   octopus_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mitosis,mergetable,deadcode,constants,commonTerms,joinPath,octopus,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   mapreduce_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mapreduce,mergetable,deadcode,commonTerms,joinPath,reorder,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   datacyclotron_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,datacyclotron,mergetable,deadcode,constants,commonTerms,joinPath,reorder,deadcode,reduce,dataflow,history,replication,multiplex,garbageCollector
+# config opt   derive_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mitosis,mergetable,deadcode,commonTerms,derivePath,joinPath,reorder,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   dictionary_pipe = 
inline,remap,dictionary,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,history,multiplex,garbageCollector
+# config opt   compression_pipe = 
inline,remap,evaluate,costModel,coercions,emptySet,aliases,mergetable,deadcode,constants,commonTerms,joinPath,deadcode,reduce,dataflow,compression,dataflow,history,multiplex,garbageCollector
+# cmdline opt  config = /ufs/mk/current//Linux/etc/monetdb5.conf
+# cmdline opt  gdk_nr_threads = 0
+# cmdline opt  monet_mod_path = 
/ufs/mk/current//Linux/lib/MonetDB5:/ufs/mk/current//Linux/lib/MonetDB5/lib:/ufs/mk/current//Linux/lib/MonetDB5/bin
+# cmdline opt  gdk_dbfarm = /ufs/mk/current//Linux/var/MonetDB5/dbfarm
+# cmdline opt  mapi_open = true
+# cmdline opt  xrpc_open = true
+# cmdline opt  mapi_port = 31658
+# cmdline opt  xrpc_port = 44186
+# cmdline opt  monet_prompt = 
+# cmdline opt  mal_listing = 2
+# cmdline opt  gdk_dbname = mTests_src_mal
+#warning: please don't forget to set your vault key!
+#(see /ufs/mk/current//Linux/etc/monetdb5.conf)
+
+# 11:11:08 >  
+# 11:11:08 >  Done.
+# 11:11:08 >  
+
diff -r ea244b45d593 -r 4549df47da9e MonetDB5/src/mal/Tests/tst054.stable.out
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000
+++ b/MonetDB5/src/mal/Tests/tst054.stable.out  Fri May 21 11:13:41 2010 +0200
@@ -0,0 +1,31 @@
+stdout of test 'tst054` in directory 'src/mal` itself:
+
+
+# 11:11:08 >  
+# 11:11:08 >   mserver5 "--config=/ufs/mk/current//Linux/etc/monetdb5.conf" 
--debug=10 --set gdk_nr_threads=0 --set 
"monet_mod_path=/ufs/mk/current//Linux/lib/MonetDB5:/ufs/mk/current//Linux/lib/MonetDB5/lib:/ufs/mk/current//Linux/lib/MonetDB5/bin"
 --set "gdk_dbfarm=/ufs/mk/current//Linux/var/MonetDB5/dbfarm"  --set 
mapi_open=true --set xrpc_open=true --set mapi_port=31658 --set xrpc_port=44186 
--set monet_prompt= --set mal_listing=2 --trace  --dbname=mTests_src_mal  
tst054.mal
+# 11:11:08 >  
+
+# MonetDB server v5.21.0, based on kernel v1.39.0
+# Serving database 'mTests_src_mal', using 4 threads
+# Compiled for x86_64-unknown-linux-gnu/64bit with 64bit OIDs dynamically 
linked
+# Found 7.751 GiB available main-memory.
+# Copyright (c) 1993-July 2008 CWI.
+# Copyright (c) August 2008-2010 MonetDB B.V., all rights reserved
+# Visit http://monetdb.cwi.nl/ for further information
+# Listening for connection requests on mapi:monetdb://eir.ins.cwi.nl:31658/
+# MonetDB/SQL module v2.39.0 loaded
+# MonetDB/GIS module v0.19.0 loaded
+!SyntaxException:parseError:return 
(1,2,3,4,5,6,7,8,9,0,1,2,3,4,5,6,7,8,9,0,1,2,3,4,5,6,7,8,9,0);
+!SyntaxException:parseError:       ^<identifier> expected
+function user.z() 
(_1:int,_2:int,_3:int,_4:int,_5:int,_6:int,_7:int,_8:int,_9:int,_10:int,_11:int,_12:int,_13:int,_14:int,_15:int,_16:int,_17:int,_18:int,_19:int,_20:int,_21:int,_22:int,_23:int,_24:int,_25:int,_26:int,_27:int,_28:int,_29:int,_30:int);
+    io.print("hello");
+end z;
+!TypeException:user.main[1]:'user.z' undefined in: _1:any := user.z()
+function user.main():void;
+    user.z();
+end main;
+
+# 11:11:08 >  
+# 11:11:08 >  Done.
+# 11:11:08 >  
+
diff -r ea244b45d593 -r 4549df47da9e MonetDB5/src/mal/mal_parser.mx
--- a/MonetDB5/src/mal/mal_parser.mx    Fri May 21 10:48:32 2010 +0200
+++ b/MonetDB5/src/mal/mal_parser.mx    Fri May 21 11:13:41 2010 +0200
@@ -1484,7 +1484,7 @@
                        if( l==0 || i) {
                                 parseError(cntxt,"<identifier> expected\n"); 
                                 skipToEnd(cntxt);
-                                pushInstruction(curBlk,curInstr);
+                                freeInstruction(curInstr);
                                 return;
                        }
                        @:GETvariable@
_______________________________________________
Checkin-list mailing list
[email protected]
http://mail.monetdb.org/mailman/listinfo/checkin-list

Reply via email to