[m-rev.] for review: carve two new modules out of write_deps_file.m

Zoltan Somogyi zoltan.somogyi at runbox.com
Thu Aug 1 23:35:47 AEST 2024


Since this diff has no algorithmic changes and just moves code around,
the only thing worth reviewing are the names of the new modules.

Note that the best name for the new generate_mmakefile_fragments module
is (sort-of) already taken. generate_dep_d_files.m does NOT actually generate
.dep and .d files; it generates most of the information from which the new
generate_mmakefile_fragments.m generates the contents of those files.
The rest of that info comes from the code that now remains in write_deps_file.m.
I will look into whether that code should be moved into generate_dep_d_files.m
after this diff is committed.

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.wdfx
Type: application/octet-stream
Size: 764 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240801/9d9c3f1c/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.wdfx
Type: application/octet-stream
Size: 216271 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20240801/9d9c3f1c/attachment-0003.obj>


More information about the reviews mailing list