<div dir="ltr"><div dir="ltr"><br></div><div class="gmail_quote gmail_quote_container"><div dir="ltr" class="gmail_attr">On Tue, 16 Dec 2025 at 18:06, 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">For review by anyone. The diff does not include the automatically-updated<br>
getopt.m.<br>
<br>
I would like feedback on<br>
<br>
- whether we should change the argument order of record_arguments<br>
to match the new record_all_arguments (I think we should, since it was<br>
added after the last release), and<br></blockquote><div><br></div><div>Yes. <br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
- whether we should use the new functionality to let mmc continue<br>
doing its job in the presence of bad options.<br></blockquote><div><br></div><div>What do you mean by "continue doing its job"? Presumably, we still</div><div>stop at some point and complain about the bad options?<br></div><div><br></div><div>> Add a mechanism for recording *all* command line args.<br><br>...<br><br>> diff --git a/NEWS.md b/NEWS.md<br>> index 777c9bbac..41349f35c 100644<br>> --- a/NEWS.md<br>> +++ b/NEWS.md<br>> @@ -1918,7 +1918,7 @@ Changes that may break compatibility<br>> * We have renamed the `lexer` and `parser` modules of the Mercury standard<br>> library to `mercury_term_lexer` and `mercury_term_parser` respectively.<br>> <br>> -* We have made slight changes to the names and/or the functionality<br>> +* We have made slight changes to the names and/or to the functionality<br>> of several predicates in the `getopt` and `getopt_io` modules.<br>> <br>> * We have removed the legacy support for the Alpha architecture.<br>> @@ -2027,10 +2027,11 @@ Changes to the Mercury standard library<br>> <br>> * The following new predicates have been added:<br>> <br>> + - pred `record_all_arguments/7`<br>> - pred `record_arguments/8`<br>> - pred `expand_file_specials/8`<br>> <br>> -* The following variants of the existing process_options predicate<br>> +* The following variants of the existing `process_options` predicate<br>> have been added:<br>> <br>> - pred `process_options_io/6`<br>> @@ -2051,8 +2052,8 @@ Changes to the Mercury standard library<br>> - pred `process_options/7`<br>> - pred `process_options_track/7`<br>> <br>> - All these predicates used to return error indications in the form of a<br>> - simple string. They now return error indications using the existing<br>> + All these predicates used to return error indications in the form of<br>> + a simple string. They now return error indications using the existing<br>> structured type `option_error`, which can be converted into a string<br>> on demand.<br><br>You have made these changes to the wrong section of the NEWS file.<br>This is the news for 22.01; your changes should be listed in the<br>news since 22.01 section.<br><br>(Perhaps we should move the 22.01 news into the HISTORY file now, rather<br>than waiting until just before the creation of the next release branch?)<br><br>That's fine otherwise.<br><br>Julien.<br></div></div></div>