From efdd5ae602a6b303b1b796b5ef98945fb470ada4 Mon Sep 17 00:00:00 2001 From: Elias Naur Date: Tue, 3 Nov 2020 17:30:21 +0100 Subject: [PATCH] op/paint: make the shape argument last in FillShape FillShape is a generalization of Fill, so it seems more natural to have the extra argument last. Signed-off-by: Elias Naur --- op/paint/paint.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/op/paint/paint.go b/op/paint/paint.go index e5413566..1f305897 100644 --- a/op/paint/paint.go +++ b/op/paint/paint.go @@ -156,9 +156,8 @@ func (d PaintOp) Add(o *op.Ops) { bo.PutUint32(data[13:], math.Float32bits(d.Rect.Max.Y)) } -// FillShape fills the area described by the provided clip.Op with the -// provided color. -func FillShape(ops *op.Ops, shape clip.Op, c color.RGBA) { +// FillShape fills the clip shape with a color. +func FillShape(ops *op.Ops, c color.RGBA, shape clip.Op) { defer op.Push(ops).Pop() shape.Add(ops) Fill(ops, c)