Re: [Monetdb-developers] [Monetdb-pf-checkins] pathfinder/compiler/algebra intro_rec_border.c, XQuery_0-20, 1.5, 1.5.4.1
On Mon, Oct 29, 2007 at 10:00:19AM +0000, Jan Rittinger wrote:
Update of /cvsroot/monetdb/pathfinder/compiler/algebra In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv15130/algebra
Modified Files: Tag: XQuery_0-20 intro_rec_border.c Log Message: -- Fix a bug for recursive queries detected by Maurice.
which one?
and should it (the bug, resp. its fix) be tested?
Stefan
Index: intro_rec_border.c
RCS file: /cvsroot/monetdb/pathfinder/compiler/algebra/intro_rec_border.c,v retrieving revision 1.5 retrieving revision 1.5.4.1 diff -u -d -r1.5 -r1.5.4.1 --- intro_rec_border.c 27 Jun 2007 13:46:26 -0000 1.5 +++ intro_rec_border.c 29 Oct 2007 10:00:16 -0000 1.5.4.1 @@ -95,6 +95,13 @@ } break;
case pa_fcns:
/* this also skips the introduction of a rec_border
operator for the content of an empty elements:
elem (fcns (nil, nil)). */
if (R(n)->kind == pa_nil)
break;
/* else fall through */ default: /* follow the children until a base or a leaf is reached */ for (unsigned int i = 0; i < PFPA_OP_MAXCHILD && n->child[i]; i++)
This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a browser. Download your FREE copy of Splunk now >> http://get.splunk.com/ _______________________________________________ Monetdb-pf-checkins mailing list Monetdb-pf-checkins@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/monetdb-pf-checkins
On 10/30/2007 09:32 AM, Stefan Manegold wrote with possible deletions:
On Mon, Oct 29, 2007 at 10:00:19AM +0000, Jan Rittinger wrote:
Update of /cvsroot/monetdb/pathfinder/compiler/algebra In directory sc8-pr-cvs16.sourceforge.net:/tmp/cvs-serv15130/algebra
Modified Files: Tag: XQuery_0-20 intro_rec_border.c Log Message: -- Fix a bug for recursive queries detected by Maurice.
which one?
and should it (the bug, resp. its fix) be tested?
It was a thinking error introduced when I changed the constructor implementation to twig constructors. This bug will not occur a second time thus no test is needed (at least in my eyes).
participants (2)
-
Jan Rittinger
-
Stefan Manegold