[m-rev.] For review: Add `set' command to declarative debugger

Julien Fischer juliensf at cs.mu.OZ.AU
Tue Oct 26 23:45:19 AEST 2004


On Tue, 26 Oct 2004, Ian MacLarty wrote:

> For review by anyone.
>
> Estimated hours taken: 2
> Branches: main
>
> Make `set' mdb command work from within the declarative debugger.
>
> browser/browse.m
> 	Make set_browse_param call a new predicate set_param/5 in
> 	browser_info.m.
>
> browser/browser_info.m
> 	Add set_param/5 to set browser parameters from an option table.
>
> browser/declarative_user.m
> 	Add and handle `set' command.
> 	Add help text.
>
> browser/parse.m
> 	Make parse/2 public so it can be called from the declarative debugger.
>
> tests/debugger/declarative/browse_arg.exp
> tests/debugger/declarative/browse_arg.inp
> 	Add test case.
>
Do you need to update tests/debugger/Mmakefile as well?
Otherwise that looks fine.

Cheers,
Julien.
--------------------------------------------------------------------------
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