That's odd. It validated for me... maybe those tests aren't run. Anyway, there 
was a missing paren, now fixed. Apologies

Simon

| -----Original Message-----
| From: Edward Z. Yang [mailto:ezy...@mit.edu]
| Sent: 18 January 2013 21:29
| To: Simon Peyton-Jones
| Cc: ghc-devs@haskell.org
| Subject: Syntax error in perf/should_run/all.T
| 
| Looks like this hunk is not quite correct:
| 
| commit 52234dfedb9ad07585271f33513fb2c9e5012e1a
| Author: Simon Peyton Jones <simo...@microsoft.com>
| Date:   Thu Jan 17 13:52:35 2013 +0000
| 
|     Adjust performance bounds slightly
| 
| diff --git a/tests/perf/should_run/all.T b/tests/perf/should_run/all.T
| index 08ff230..cdb8852 100644
| --- a/tests/perf/should_run/all.T
| +++ b/tests/perf/should_run/all.T
| @@ -227,9 +227,10 @@ test('T5536',
|                                               1250000000)),
|                             # expected value: 1246287228 (i386/Linux)
|        if_wordsize(64,
| -          stats_num_field('bytes allocated', 2480000000,
| -                                             2510000000)),
| -                           # expected value: 2492589480 (amd64/Linux)
| +          stats_range_field('bytes allocated', 892399040, 5),
| +                           # expected value: 2,492,589,480
| (amd64/Linux)
| +                           # 17/1/13:          892,399,040
| (x86_64/Linux)
| +                           #                   (new demand analyser)
| 
| The file now syntax errors for me.
| 
| Edward
_______________________________________________
ghc-devs mailing list
ghc-devs@haskell.org
http://www.haskell.org/mailman/listinfo/ghc-devs

Reply via email to