[m-rev.] for post-commit review: break up mercury_to_mercury.m

Julien Fischer jfischer at opturion.com
Fri Apr 7 14:35:11 AEST 2023


On Thu, 6 Apr 2023, Zoltan Somogyi wrote:

> Break up mercury_to_mercury.m.
> 
> compiler/mercury_to_mercury.m:
>     Delete this module, and replace it with ...
> 
> compiler/parse_tree_out_cons_id.m:
> compiler/parse_tree_out_sym_name.m:
> compiler/parse_tree_out_type.m:
> compiler/parse_tree_out_misc.m:
>     ... these four modules. The first three write out the entities
>     in their names: cons_ids, sym_names, and types. The fourth contains
>     the rest of the old mercury_to_mercury.m, plus a few predicates
>     moved there from prog_out.m that deal with indentation.
> 
> compiler/parse_tree.m:
>     Include the four new modules, and stop including the deleted module.
> 
> compiler/notes/compiler_design.html:
>     Document the new modules.
> 
> compiler/prog_out.m:
>     Delete the code moved to parse_tree_out_misc.m.
> 
> compiler/*.m:
>     Adjust the imports as needed. Most modules need only one, maybe two
>     of mercury_to_mercury's four successor modules.

That's fine.

Julien.




More information about the reviews mailing list