[m-rev.] for review: warn about unneeded initial and final state vars

Zoltan Somogyi zoltan.somogyi at runbox.com
Sun May 18 07:25:24 AEST 2025



On Sun, 18 May 2025 02:32:17 +1000, Julien Fischer <jfischer at opturion.com> wrote:

> On Fri, 16 May 2025 at 03:46, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote: 
> > Add options to warn about unused state vars.
> > +            % with anoher clause that *does* use that same state variable.)
> 
> s/anoher/another/

Fixed.

> That looks fine otherwise.

Thanks for that. Attached is the rest of changes called for by the
new warnings, again for review by anyone. (There was a lot of unnecessary
passing of state vars, so it's big.)

I have bootchecked an earlier version of this diff with -O5 --intermod-opt
in hlc.gc, asm_fast.gc and in a debug grade, but will repeat all three
before commit.

This evening, I expect to post for review an entry for NEWS.md.

Zoltan.



-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.usw2
Type: application/octet-stream
Size: 2949 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20250518/4f10d537/attachment.obj>


More information about the reviews mailing list