[m-rev.] for review: delete read_term* from term_io.m

Julien Fischer jfischer at opturion.com
Fri Mar 4 12:10:00 AEDT 2022


Hi Zoltan,

On Fri, 4 Mar 2022, Zoltan Somogyi wrote:

> For review by Julien, since he can answer the next question.
>
> The deletion should be mentioned in NEWS, but NEWS says
> that it is news *for* 22.01, and this belongs in news *since* 22.01,
> so where should I put it? I presume you don't want this diff
> in 22.01.

You should add a new heading to the top of the NEWS file on the
trunk:

      NEWS since Mercury 22.01
      =========================

and place any post-22.01 NEWS under it.

> Another question is: should I make tests/general/interpreter.m
> read-only, dependent on tests/debugger/interpreter.m, and copied
> over from that file (after a temporaryt chmod) if that file ever changes?

There's a relationship between those two and samples/interpreter.m as
well.

> I think the answer to that should be "yes".

Agreed.

The diff looks fine.

Julien.


More information about the reviews mailing list