From 27d2000103d64c47b3c07e92f1d0bd16eadaeac2 Mon Sep 17 00:00:00 2001 From: qiulaidongfeng <2645477756@qq.com> Date: Sun, 1 Oct 2023 17:08:52 +0800 Subject: [PATCH] go/printer: use max/min func Change-Id: I2f708bca0c1e26fb63083731927d5d6a51d41690 --- src/go/printer/nodes.go | 17 ++++------------- src/go/printer/printer.go | 5 +---- 2 files changed, 5 insertions(+), 17 deletions(-) diff --git a/src/go/printer/nodes.go b/src/go/printer/nodes.go index e41ffc19583..97c2cab0f84 100644 --- a/src/go/printer/nodes.go +++ b/src/go/printer/nodes.go @@ -44,10 +44,7 @@ import ( // linebreaks. At the moment there is no easy way to know about // future (not yet interspersed) comments in this function. func (p *printer) linebreak(line, min int, ws whiteSpace, newSection bool) (nbreaks int) { - n := nlimit(line - p.pos.Line) - if n < min { - n = min - } + n := max(nlimit(line-p.pos.Line), min) if n > 0 { p.print(ws) if newSection { @@ -670,9 +667,7 @@ func walkBinary(e *ast.BinaryExpr) (has4, has5 bool, maxProblem int) { h4, h5, mp := walkBinary(l) has4 = has4 || h4 has5 = has5 || h5 - if maxProblem < mp { - maxProblem = mp - } + maxProblem = max(maxProblem, mp) } switch r := e.Y.(type) { @@ -685,9 +680,7 @@ func walkBinary(e *ast.BinaryExpr) (has4, has5 bool, maxProblem int) { h4, h5, mp := walkBinary(r) has4 = has4 || h4 has5 = has5 || h5 - if maxProblem < mp { - maxProblem = mp - } + maxProblem = max(maxProblem, mp) case *ast.StarExpr: if e.Op == token.QUO { // `*/` @@ -699,9 +692,7 @@ func walkBinary(e *ast.BinaryExpr) (has4, has5 bool, maxProblem int) { case "/*", "&&", "&^": maxProblem = 5 case "++", "--": - if maxProblem < 4 { - maxProblem = 4 - } + maxProblem = max(maxProblem, 4) } } return diff --git a/src/go/printer/printer.go b/src/go/printer/printer.go index 5cf4e4bb5fc..ff36d581402 100644 --- a/src/go/printer/printer.go +++ b/src/go/printer/printer.go @@ -861,10 +861,7 @@ func (p *printer) writeWhitespace(n int) { // nlimit limits n to maxNewlines. func nlimit(n int) int { - if n > maxNewlines { - n = maxNewlines - } - return n + return min(n, maxNewlines) } func mayCombine(prev token.Token, next byte) (b bool) {