[m-rev.] for post-commit review: semantic help pieces for grade options
Julien Fischer
jfischer at opturion.com
Mon Jul 14 14:25:12 AEST 2025
On Wed, 2 Jul 2025 at 20:22, Zoltan Somogyi <zoltan.somogyi at runbox.com>
wrote:
>
>
> On Wed, 2 Jul 2025 16:54:40 +1000, Julien Fischer <jfischer at opturion.com>
> wrote:
> > The rest looks fine.
>
> Thanks for all of that. The attached diff addresses all your comments.
>
I have one somewhat belated comment on this.
> diff --git a/tests/warnings/help_text.err_exp
b/tests/warnings/help_text.err_exp
> index 56cb25734..2d33dfd0e 100644
> --- a/tests/warnings/help_text.err_exp
> +++ b/tests/warnings/help_text.err_exp
...
> @@ -385,7 +397,11 @@ Grade options
>
> --memory-profiling (grade modifier: `.memprof')
> Prepare the generated code for profiling of memory usage and
retention
> - by mprof. This option is supported only when targeting C.
> + by mprof. Please see the
> + "Using mprof for profiling memory retention" section in the
Mercury
> + User's Guide for details.
I don't think memory rentention profiling is the main thing that memprof
grades
are used for, so I would either say:
Please see the "Using mprof for profiling memory allocation" and
"User mprof for profiling memory retention" sections in the Mercury
User's Guide for details.
or, if you have to list one only section, point to the allocation one.
Julien.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20250714/232021e6/attachment.html>
More information about the reviews
mailing list