[m-rev.] for review: naming magling, step 1
Mark Brown
dougl at cs.mu.OZ.AU
Mon Jan 7 17:18:53 AEDT 2002
On 04-Jan-2002, Zoltan Somogyi <zs at cs.mu.OZ.AU> wrote:
> For review by Fergus.
>
> Zoltan.
>
> The first step towards centralizing name construction and mangling issues
> in the browser directory.
>
> This change does not update either util/mdemangle.c or profiler/demangle.m,
> even though it should. Both those files should be replaced by code that simply
> calls code in the browser directory (in the modules now added and in modules to
> be added later) to perform their job. In both cases, replacing them piecemeal
> is extra work of little value, so I intend to replace them only when all the
> issues related to names and mangling has been centralized in the browser
> directory.
>
> browser/names.m:
> A new module for holding information about the structure of names.
>
> browser/mangle.m:
> A new module for handling name mangling, i.e. the conversion of names
> to and from a form acceptable inside C identifiers.
>
> It implements a new scheme for name manging, which (unlike the previous
> scheme) does not obscure a whole name just because one part contains
> characters that cannot occur in identifiers.
>
> browser/dl.m:
> browser/name_mangle.m:
> Make these modules stand on their own, not as submodules of module mdb.
> They are needed in other places as well.
The reason we introduced the super-module mdb was to minimise a problem
with browser modules polluting the user's namespace. Making these
modules standalone will cause that problem to re-occur, unless there is
something I missed. Moving the modules out of mdb is fine, but you
should put them all in another super-module (e.g. mercury_util).
See http://www.cs.mu.oz.au/research/mercury/mailing-lists/mercury-developers/mercury-developers.0002/0026.html
for the original change.
(Note that I haven't reviewed this change.)
Cheers,
Mark.
--------------------------------------------------------------------------
mercury-reviews mailing list
post: mercury-reviews at cs.mu.oz.au
administrative address: owner-mercury-reviews at cs.mu.oz.au
unsubscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: unsubscribe
subscribe: Address: mercury-reviews-request at cs.mu.oz.au Message: subscribe
--------------------------------------------------------------------------
More information about the reviews
mailing list