From dd768bf1c623521b27d1b16ab50a7b80cbcf8b37 Mon Sep 17 00:00:00 2001
From: Dmitry Vyukov <dvyukov@google.com>
Date: Sun, 18 Feb 2018 09:31:12 +0100
Subject: prog: reorder Minimize arguments

Make the predicate the last argument.
It's more common and convenient (arguments are not separated by multiple lines).
---
 pkg/repro/repro.go | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

(limited to 'pkg')

diff --git a/pkg/repro/repro.go b/pkg/repro/repro.go
index bbe0e2d7c..fc79f5a8a 100644
--- a/pkg/repro/repro.go
+++ b/pkg/repro/repro.go
@@ -428,14 +428,15 @@ func (ctx *context) minimizeProg(res *Result) (*Result, error) {
 	if res.Opts.Fault {
 		call = res.Opts.FaultCall
 	}
-	res.Prog, res.Opts.FaultCall = prog.Minimize(res.Prog, call, func(p1 *prog.Prog, callIndex int) bool {
-		crashed, err := ctx.testProg(p1, res.Duration, res.Opts)
-		if err != nil {
-			ctx.reproLog(0, "minimization failed with %v", err)
-			return false
-		}
-		return crashed
-	}, true)
+	res.Prog, res.Opts.FaultCall = prog.Minimize(res.Prog, call, true,
+		func(p1 *prog.Prog, callIndex int) bool {
+			crashed, err := ctx.testProg(p1, res.Duration, res.Opts)
+			if err != nil {
+				ctx.reproLog(0, "minimization failed with %v", err)
+				return false
+			}
+			return crashed
+		})
 
 	return res, nil
 }
-- 
cgit mrf-deployment