<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 1 Aug 2024 at 23:35, Zoltan Somogyi <<a href="mailto:zoltan.somogyi@runbox.com">zoltan.somogyi@runbox.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Since this diff has no algorithmic changes and just moves code around,<br>
the only thing worth reviewing are the names of the new modules.<br>
<br>
Note that the best name for the new generate_mmakefile_fragments module<br>
is (sort-of) already taken. generate_dep_d_files.m does NOT actually generate<br>
.dep and .d files; it generates most of the information from which the new<br>
generate_mmakefile_fragments.m generates the contents of those files.<br>
The rest of that info comes from the code that now remains in write_deps_file.m.<br>
I will look into whether that code should be moved into generate_dep_d_files.m<br>
after this diff is committed.<br></blockquote><div><br></div><div>The new module names (and diff) look fine.</div><div><br></div><div>Julien.</div></div></div>