[m-rev.] for review: replace the item list with structured ASTs

Zoltan Somogyi zoltan.somogyi at runbox.com
Tue Jul 21 02:05:02 AEST 2015



On Mon, 20 Jul 2015 09:59:53 +1000 (AEST), Julien Fischer <jfischer at opturion.com> wrote:
> >     we used to use to identify file types. Predicates that used to take strings
> >     representing suffixes as arguments now have to specify whether they can
> >     handle all these file types (source, information and optimization),
> 
> s/information/interface/

Actually, that is just a one-character typo in my vim macros, IF vs INF :-(

> > +% combinations of answers into an enum. For the details, see status_proposal
> > +% in compiler/notes.
> 
> There is no such file.

Git strikes again. I added it with "git add", not "git add -N", so obviously
"git diff" didn't show it :-(

> > +:- pred sym_name_get_module_name_det(sym_name::in, module_name::out) is det.
> > +
> 
> The usual convention for naming such predicates is to use "det_" as a prefix,
> rather than "_det" as a suffix.

I followed this suggestion, as well as all your other suggestions.

I am doing a final round of bootchecks, and will then commit the diff. I will
address anybody else's reviews after the commit.

Zoltan.




More information about the reviews mailing list