[m-rev.] for review: rewrite rest of options_file.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Mon Jun 8 05:49:29 AEST 2020


For review by anyone.

Before commit, I intend to reorder the contents of options_file.m,
putting related predicates next to each other, but this diff leaves
predicates in their current places to make review easier.

Does anyone know whether the functionality of options_file.m has any
test cases for it, apart from Mercury.config and Mercury.options files?
I couldn't find any. I do think there should be some, but I am not sure
whether these should go in e.g. tests/hard_coded, or in a new subdirectory
of tests dedicated to tests of option file handling. (In those tests,
the code in the module(s) being compiled almost doesn't matter,
which is a pretty clear point of difference from the contents of
all the other test directories). Any preferences? And does anyone
have options files in their own projects that go beyond the usual
Mercury.{config,options} files in their complexity that I could use
as (parts of) test cases?

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.of3
Type: application/octet-stream
Size: 1732 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20200608/6ff529da/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.of3
Type: application/octet-stream
Size: 86402 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20200608/6ff529da/attachment-0003.obj>


More information about the reviews mailing list