[m-rev.] for review: move invoke_long_system_command into system_cmds

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Jan 5 20:29:37 AEDT 2024


On 2024-01-05 20:27 +11:00 AEDT, "Julien Fischer" <jfischer at opturion.com> wrote:
> While the only callers of invoke_long_system_command* are currently only
> in compile_target_code.m, it makes more sense to place them with other
> invoke_system_command predicates (it certainly makes them easier to
> find).

That's fine.

> Also, at 3000+ lines, compile_target_code.m is likely candidate
> for being broken up in the future.

Agreed.

Zoltan.


More information about the reviews mailing list