[m-rev.] for review: allow comments and blank lines in mdb scripts

Peter Ross pro at missioncriticalit.com
Fri Jun 15 17:25:31 AEST 2007


On 6/15/07, Ian MacLarty <maclarty at csse.unimelb.edu.au> wrote:
> Ignore empty lines, or lines that start with `#', in sourced mdb scripts.
> This is useful for formatting and documenting mdb scripts.
>
> Create a local copy of the mdbrc file so that we can test new mdb scripts
> in the test suite.
>
> NEWS:
>         Mention the changes.
>
> configure.in:
>         Create a copy of mdbrc.in in the tests directory and create an mdbrc
>         from this file using different substitutions than those used for the
>         mdbrc that will be installed.  This is necessary so that we can use
>         the local mdbrc file when running the tests.  Previously if the mdbrc
>         file was changed (for example a new alias was added), then we had to
>         install it before testing.
>
> doc/user_guide.texi:
>         Document the new behaviour of the source command.
>
> scripts/mdb_grep:
> scripts/mdb_open:
> scripts/mdb_track:
>         Add comments to these mdb scripts.
>
> scripts/mdbrc.in:
>         Use a different substitution variable for the location of mdb_doc,
>         since for the test version of mdbrc mdb_doc will be in a different
>         location to the other scripts.
>
> tests/Mmake.common:
>         Get mdb to use the local mdbrc when invoked for the tests.
>
>         Clean mdbrc and mdbrc.in.
>
> tests/debugger/Mercury.options:
> tests/debugger/Mmakefile:
> tests/debugger/scripts.exp:
> tests/debugger/scripts.inp:
> tests/debugger/scripts.m:
>         Add a test case for the three existing mdb scripts.
>
> trace/mercury_trace_readline.c:
> trace/mercury_trace_readline.h:
>         Rename MR_trace_readline_expand_args to MR_trace_readline_from_script
>         and get it to ignore comment lines and blank lines.
>
> trace/mercury_trace_internal.c:
>         Adapt to changes in mercury_trace_readline.h.
>
Looks fine.
--------------------------------------------------------------------------
mercury-reviews mailing list
Post messages to:       mercury-reviews at csse.unimelb.edu.au
Administrative Queries: owner-mercury-reviews at csse.unimelb.edu.au
Subscriptions:          mercury-reviews-request at csse.unimelb.edu.au
--------------------------------------------------------------------------



More information about the reviews mailing list