[m-dev.] for review: fix MR_LOWLEVEL_DEBUG
Thomas Charles CONWAY
conway at cs.mu.OZ.AU
Tue Jul 7 12:14:50 AEST 1998
Zoltan Somogyi, you write:
>
> +#define MR_CONTEXT(x) MR_ENGINE(context).x
> +
A comment may be helpful here.
> printf("\nnondstack dump\n");
> - for (fr = maxfr; fr > nondetstack_zone->min; fr = bt_prevfr(fr)) {
> + for (fr = maxfr; fr > MR_CONTEXT(nondetstack_zone)->min;
> + fr = bt_prevfr(fr)) {
> dumpframe(fr);
> }
> return;
Should that be MR_maxfr rather than maxfr?
Apart from those to trivial things the change looks fine, so I think
it would be fine to commit it.
Thomas
--
Thomas Conway <conway at cs.mu.oz.au>
Nail here [] for new monitor. )O+
More information about the developers
mailing list