feat(vips): filter max_frames(n) (#195)

* feat(vips): filter max_frames(n) * test: update golden files
cshum · Oct 6, 2022 · 5b3e578 · 5b3e578
1 parent f5efd67
commit 5b3e578
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 1 deletion.
diff --git a/testdata/golden/filters%3Amax_frames%283%29/dancing-banana.gif b/testdata/golden/filters%3Amax_frames%283%29/dancing-banana.gif
diff --git a/vips/filter.go b/vips/filter.go
@@ -146,7 +146,7 @@ func (v *Processor) watermark(ctx context.Context, img *Image, load imagor.LoadF
 	return
 }
 
-func frames(ctx context.Context, img *Image, _ imagor.LoadFunc, args ...string) (err error) {
+func frames(_ context.Context, img *Image, _ imagor.LoadFunc, args ...string) (err error) {
 	ln := len(args)
 	if ln == 0 {
 		return

diff --git a/vips/process.go b/vips/process.go
@@ -69,6 +69,11 @@ func (v *Processor) Process(
 				}
 			}
 			break
+		case "max_frames":
+			if n, _ := strconv.Atoi(p.Args); n > 0 {
+				maxN = n
+			}
+			break
 		case "stretch":
 			stretch = true
 			break

diff --git a/vips/processor_test.go b/vips/processor_test.go
@@ -125,6 +125,7 @@ func TestProcessor(t *testing.T) {
 			{name: "original no animate", path: "filters:fill(white):format(jpeg)/dancing-banana.gif"},
 			{name: "original animated", path: "dancing-banana.gif"},
 			{name: "original animated quality", path: "filters:quality(60)/dancing-banana.gif"},
+			{name: "original animated max_frames", path: "filters:max_frames(3)/dancing-banana.gif"},
 			{name: "rotate animated", path: "fit-in/100x150/filters:rotate(90):fill(yellow)/dancing-banana.gif"},
 			{name: "crop animated", path: "30x20:100x150/dancing-banana.gif"},
 			{name: "crop-percent animated", path: "0.1x0.2:0.89x0.72/dancing-banana.gif"},