Mathis/sharp
1
0
Fork 0
mirror of https://github.com/lovell/sharp.git synced 2025-07-09 18:40:16 +02:00
Code Issues Packages Projects Releases Wiki Activity

Ensure delays of the origin image is preserved (#3062)

Browse Source 
By not initializing the delay vector, since we assume it's empty by default.

See: #3061.
This commit is contained in:
Kleis Auke Wolthuizen 2022-01-28 12:50:31 +01:00 committed by GitHub
parent 68823a5edb
commit 7aa340232e
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 9 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/pipeline.h
View File

@ -327,7 +327,6 @@ struct PipelineBaton {
ensureAlpha(-1.0), ensureAlpha(-1.0),
colourspaceInput(VIPS_INTERPRETATION_LAST), colourspaceInput(VIPS_INTERPRETATION_LAST),
colourspace(VIPS_INTERPRETATION_LAST), colourspace(VIPS_INTERPRETATION_LAST),
delay{-1},
loop(-1), loop(-1),
tileSize(256), tileSize(256),
tileOverlap(0), tileOverlap(0),

9
test/unit/webp.js
View File

@ -183,6 +183,15 @@ describe('WebP', function () {
assert.deepStrictEqual(updated.delay, expectedDelay); assert.deepStrictEqual(updated.delay, expectedDelay);
}); });
it('should preserve delay between frames', async () => {
const updated = await sharp(fixtures.inputWebPAnimated, { pages: -1 })
.webp()
.toBuffer()
.then(data => sharp(data, { pages: -1 }).metadata());
assert.deepStrictEqual(updated.delay, [120, 120, 90, 120, 120, 90, 120, 90, 30]);
});
it('should work with streams when only animated is set', function (done) { it('should work with streams when only animated is set', function (done) {
fs.createReadStream(fixtures.inputWebPAnimated) fs.createReadStream(fixtures.inputWebPAnimated)
.pipe(sharp({ animated: true })) .pipe(sharp({ animated: true }))