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

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Mar 4 13:56:29 AEDT 2022


2022-03-04 13:48 GMT+11:00 "Peter Wang" <novalazy at gmail.com>:
> On Fri, 04 Mar 2022 05:33:48 +1100 "Zoltan Somogyi" <zoltan.somogyi at runbox.com> wrote:
>> 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?
>> I think the answer to that should be "yes".
> 
> Does tests/general/interpreter.m serve any purpose any more?

In effect, it tests one the samples. Ideally, all the samples should be
tested, but testing one is better than nothing. That being said,
the test case tests only unifications, not clause resolution, which is
a pretty big hole :-(

> If not, I suggest to delete it.

I am ambivalent, but wouldn't object. Julien?

Zoltan.



More information about the reviews mailing list