[m-rev.] for post-commit review: do more kinds of check for all foreign_procs

Zoltan Somogyi zoltan.somogyi at runbox.com
Fri Jan 30 05:19:04 AEDT 2026


For review by anyone. The diff is with -b.

While doing these changes, I saw that we use wildly different terminology
to refer to the same entities (foreign_procs) in different diagnostic messages.
The three main patterns are these:

   "Error: foreign clause for predicate xyz ..."

   "In ':- pragma foreign_proc' declaration for predicate xyz ..."

   "In the C code for predicate xyz: ..."

I propose that we change all these diagnostics to either something like this:

     "In the C language ':- pragma foreign_proc' declaration for predicate xyz ..."

or to something like this:

   "In the C foreign_proc for predicate xyz ..."

The former is more formal and complete, the second is shorter.
I slightly prefer the second. What do you guys think? Do you have
another alternative you prefer to these?
   
Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.fpmw
Type: application/octet-stream
Size: 1111 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20260130/570baf96/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.fpmw
Type: application/octet-stream
Size: 27102 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20260130/570baf96/attachment-0003.obj>


More information about the reviews mailing list