[m-rev.] for review: break up simplify.m
Julien Fischer
jfischer at opturion.com
Thu Jul 24 10:47:00 AEST 2014
Hi Zoltan,
On Wed, 23 Jul 2014, Zoltan Somogyi wrote:
> This is a cleanup of simplify.m before work on fixing bug 311.
>
> For review by anyone. One point for the reviewer to consider: should we make format_call.m
> a part of the new simplify package? It does its task as part of simplification, but
> the fact that format_call has something to work on is noticed by determinism analysis.
In relation to the format_call module I suggest separating out the
predicate is_format_call/3 and putting it in one of the common utility
modules, hlds_code_util for example. It should then be possible to make
format_call a private submodule of the simplify package, since
determinism analysis only makes uses of is_format_call/3.
The rest of the diff looks fine.
Cheers,
Julien.
More information about the reviews
mailing list