Hi Sjoerd, as far as I can remember we only use the DTD to resolve entities and id references. So if no DTD is available an error is triggered, but the document is still imported. Just my two cents... Jan On Feb 18, 2010, at 10:50, Sjoerd Mullender wrote:
Update of /cvsroot/monetdb/pathfinder/runtime In directory sfp-cvsdas-1.v30.ch3.sourceforge.com:/tmp/cvs-serv30011
Modified Files: Tag: Feb2010 shredder.mx Log Message: If there is no DTD, I guess there is not much point continuing. Found by Coverity.
Index: shredder.mx =================================================================== RCS file: /cvsroot/monetdb/pathfinder/runtime/shredder.mx,v retrieving revision 1.162 retrieving revision 1.162.2.1 diff -u -d -r1.162 -r1.162.2.1 --- shredder.mx 7 Jan 2010 15:24:29 -0000 1.162 +++ shredder.mx 18 Feb 2010 09:50:12 -0000 1.162.2.1 @@ -1362,8 +1362,10 @@ ctx->myDoc = xmlNewDoc(ctx->version); }
- if (!dtd) + if (!dtd) { GDKerror("shred_external_subset: problem with reading external DTD (\"%s\")\n", SystemID); + BAILOUT(shredCtx); + }
if (ctx->myDoc->extSubset) { stream_printf(GDKout, "!WARNING: double external subset (\"%s\") may cause problems.\n", SystemID);
------------------------------------------------------------------------------ Download Intel® Parallel Studio Eval Try the new software tools for yourself. Speed compiling, find bugs proactively, and fine-tune applications for parallel performance. See why Intel Parallel Studio got high marks during beta. http://p.sf.net/sfu/intel-sw-dev _______________________________________________ Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins
-- Jan Rittinger Lehrstuhl Datenbanken und Informationssysteme Wilhelm-Schickard-Institut für Informatik Eberhard-Karls-Universität Tübingen http://www-db.informatik.uni-tuebingen.de/team/rittinger