[m-rev.] diff: simplify code constructing pred names

Zoltan Somogyi zoltan.somogyi at runbox.com
Mon Feb 7 12:39:53 AEDT 2022


The diff is not really worth reviewing, but it does raise
a question. At the moment, there are several modules
that construct new pred names (this diff affects two of them).
There is no overall pattern, and what partial pattern does exist
is, in my opinion, flawed, for reasons documented in the XXXs
added by this diff.

Would anyone object if I moved all the code that constructs
new pred names to a new module, pred_name.m? This seems to me
to be a necessary first step to imposing an overall naming scheme.
(Whose implementation would of course require updating all
the modules, both C and Mercury, that want *parse* pred names.)

Zoltan.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: Log.pn
Type: application/octet-stream
Size: 167 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20220207/efe4042b/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: DIFF.pn
Type: application/octet-stream
Size: 6406 bytes
Desc: not available
URL: <http://lists.mercurylang.org/archives/reviews/attachments/20220207/efe4042b/attachment-0001.obj>


More information about the reviews mailing list