[m-rev.] for review: --warn-can-fail-functions
Julien Fischer
jfischer at opturion.com
Wed Aug 7 16:45:54 AEST 2024
On Wed, 7 Aug 2024 at 06:53, Zoltan Somogyi <zoltan.somogyi at runbox.com> wrote:
>
> For review by anyone. I am seeking feedback mainly about
> the name and documentation of the new option.
The name seems fine; you may want to extend the entry in the NEWS file
to explain
why this is useful .
> The third attached file lists all the can-fail functions in the standard library.
> It would be nice to replace these, probably with predicate versions
> (which some of them probably already have), though they would need
> a longer process: (a) deprecate by marking as obsolete, (b) wait for a release,
> and then (c) delete.
We can certainly go ahead and add missing predicates versions and obsolete the
function versions now.
The diff looks fine.
(BTW, did you receive my last response regarding the change to where .prof files
are placed?)
Julien.
More information about the reviews
mailing list