[m-rev.] for review: merge integer token representations in the lexer

Julien Fischer jfischer at opturion.com
Sat Apr 22 19:35:55 AEST 2017



On Sat, 22 Apr 2017, Zoltan Somogyi wrote:

>
>> Ignore that, it's a dreadful idea -- those types are part of the
>> public interface of the term module.
>
> Given that these types are effectively describing an aspect
> of a set of soon-to-be-added builtin types, would anyone
> object moving them from term.m to builtin.m?

Yes, I object -- doing that makes sense for the compiler, but not for
any other programs, the huge majority of which are not going to want to
describe aspects of the new integer types (even if they make use of
them).

Perhaps, we should revisit the original issue here: is there actually
a problem with lexer.m importing term.m?  I can't see that doing 
so actually complicates the library structure that much.

Julien.


More information about the reviews mailing list