[m-rev.] for review: module_add_type_defn

Julien Fischer jfischer at opturion.com
Tue Jun 27 23:05:18 AEST 2017


Hi Zoltan,

On Tue, 27 Jun 2017, Zoltan Somogyi wrote:

> For review by anyone.
>

> Foreign MergedBody

?

> Make the code adding new types to the HLDS readable.
> 
> The motivation for this diff was that I wanted the compiler to generate
> a warning if a module declared the same type twice. (During the cleanup
> of unify_proc.m I did recently, I found and fixed such a duplicate
> declaration.)

The diff looks fine.

Julien.


More information about the reviews mailing list