[m-rev.] diff: updating programming style in samples
Julien Fischer
jfischer at opturion.com
Wed Jul 7 14:59:45 AEST 2021
Hi Zoltan,
On Wed, 7 Jul 2021, Zoltan Somogyi wrote:
> When I searched the samples for examples of parsing to recommend
> in m-users, I found myself looking at code I would not want to recommend
> to anyone. This diff goes some way towards fixing that.
>
> BTW, it would have been nice if the samples had test cases that were
> exercised during bootcheck.
The samples directory has a Mmakefile that builds most of it; plugging
that into the bootcheck script should be straightforward.
There's also a tests directory, but that looks to be suffering some
bitrot.
> The biggest sample program, muz, has
> a simple test, but it cannot have ever been passed, because muz is
> missing a module (ironically named repository.m), which, looking at the
> backup snapshot I took of the CVS repository when I left UoM, seems
> to have been never added to CVS.
repository.m is only imported by the unused module zlogic.m; muz should
(and does) compile and run fine, e.g.
$ mmake muz.depend
$ mmake muz
$ ./muz -t toolkit.tex example.tex
should work.
> Can we add a policy that any *new* programs added to samples
> have to be tested by bootcheck?
No objection from me, although we would need to set things up so
that is possible first.
> Similarly, should we change bootcheck so that -e (checking the extras)
> is the default?
Does it still work? IIRC, -e was never that much of a check and I would
imagine that it has bitrotted.
Julien.
More information about the reviews
mailing list