[m-rev.] for review: mantis bug 401, and changing use_module

Julien Fischer jfischer at opturion.com
Mon Jan 11 16:47:54 AEDT 2016

On Sun, 10 Jan 2016, Zoltan Somogyi wrote:

> The part of the log message discussing the change to the
> meaning of use_module declarations should be read by everyone.
> The rest is for review by anyone.


> Handle "use_module in interface, import_module in implementation".


> tests/invalid/bad_instance.m:
> tests/submodules/class.m:
> tests/submodules/nested.m:
> tests/submodules/nested3.m:
> tests/submodules/parent.m:
>     Make names fully module qualified where the language rules now require it.
> tests/submodules/deeply_nested.m:
>     Comment a part of the test that tested the partially qualified names,

    Comment out a ...

The change looks fine.


More information about the reviews mailing list