[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.

Julien.



More information about the reviews mailing list