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

Julien Fischer jfischer at opturion.com
Fri Mar 4 14:47:11 AEDT 2022


On Fri, 4 Mar 2022, Zoltan Somogyi wrote:

> 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?

No objections from me.

Julien.


More information about the reviews mailing list