[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