[m-rev.] for review: fix debugger/solver_test failing test case

Ian MacLarty maclarty at cs.mu.OZ.AU
Thu Jul 28 14:10:14 AEST 2005


>
> On Wed, 27 Jul 2005, Ian MacLarty wrote:
>
> > For review by anyone.
> >
> > Estimated hours taken: 1
> > Branches: main and 0.12
> >
> > Only show procedures *declared* in a the module given to the `procedures' mdb
> > command.
> >

Fix a bug in this patch: compare the proc's module name with the *full* module
name.  This showed up because it broke the debugger/breakpoints test.  Sorry, I
know I should have run *all* tests before posting for review. I hadn't
commited the mistake.

This is the interdiff:

diff -u trace/mercury_trace_tables.c trace/mercury_trace_tables.c
--- trace/mercury_trace_tables.c	26 Jul 2005 15:39:54 -0000
+++ trace/mercury_trace_tables.c	27 Jul 2005 07:40:03 -0000
@@ -365,7 +365,7 @@
         /*
         ** Only show procs which are declared in the module.
         */
-        if (MR_streq(decl_module, name)) {
+        if (MR_streq(decl_module, module->MR_ml_name)) {
             MR_print_proc_id_and_nl(fp, module->MR_ml_procs[i]);
         }
     }

Ian.

--------------------------------------------------------------------------
mercury-reviews mailing list
post:  mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe:   Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------



More information about the reviews mailing list