[m-rev.] for review: fix for split_c_files test

Simon Taylor stayl at cs.mu.OZ.AU
Sat Mar 15 18:17:06 AEDT 2003


On 15-Mar-2003, Fergus Henderson <fjh at cs.mu.OZ.AU> wrote:
> For review by stayl.  In particular, Simon, what was the rationale for
> the comment "The modules in SPLIT_PROGS should also be in ORDINARY_PROGS
> above."? 

Looking at it now, I can't see any good reason for it.

> I don't think that is correct; I think split_c_files should only
> be in SPLIT_PROGS, not in ORDINARY_PROGS.  So this diff deletes that comment,
> and removes split_c_files from SPLIT_PROGS.
 
> Branches: main
> Estimated hours taken: 0.5
> 
> tests/hard_coded/Mmakefile:
> 	Don't run the `split_c_files' test without --split-c-files.
> 	In particular, don't run it in hlc grades, since it gets
> 	compiled with `--trace deep', which reports an error in hlc grades.
> 	
> tests/hard_coded/split_c_files.exp:
> 	Delete this file, since it is no longer used;
> 	the expected output when compiled with --split-c-files
> 	is in split_c_files.split.exp.

OK.

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