[m-rev.] for review: moose fixes

Julien Fischer juliensf at students.cs.mu.OZ.AU
Wed Jul 23 07:15:34 AEST 2003



On Wed, 23 Jul 2003, Fergus Henderson wrote:

> On 22-Jul-2003, Julien Fischer <juliensf at students.cs.mu.OZ.AU> wrote:
> >
> > Allow moose to parse conditional goals written using the if - then - else
> > notation.  This makes the code generated by moose less ugly when the `.moo'
> > file contains these type of goals.
>
> The changes that you posted look fine, but I think there are a few
> things that you missed:
>
> 	- Don't you need to also modify term_to_ite
> 	  (in extras/moose/mercury_syntax.m) to handle the
> 	  new kind of it-then-else goals?
>
On closer inspection term_to_ite and a number of other predicates
aren't actually used.  Moose compiles and runs quite happily without
them.  Are they just dead code or was there some (unimplemented) purpose
to them?

I wasn't actually intending to preserve the difference between
the two notations for if-then-else goals in the generated code
(although if anyone yells loudly enough ...)

> 	- The new syntax should be documented
> 	  (in the extras/moose/README file).
>
> 	  [In fact, the old syntax for if-then-else also needs
> 	  documenting, I think.]
>
Why?  It's just normal Mercury syntax - no other syntax (other than moose
specific declarations) are mentioned in the README file.


--------------------------------------------------------------------------
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