Rewrite handling EXIF orientation — add tests, make it plugin-independent

[skalee]

What's Changing and Why

Handling EXIF orientation has been rewritten from scratch. In a result:

• it no longer relies on plugins (flip, rotate, and others);
• solves issues coming from plugins like black border (Rotation adds black border #721) in scope of EXIF rotation;
• everything is performed in a single pass.

Pictures are now processed with bitmap transformation functions, which map pixels from source bitmap to correctly rotated one. Computational complexity is proportional to number of pixels in given picture.

Finally, tests for EXIF rotation are added in this pull request. They were missing so far.

What else might be affected

It somewhat addresses #873, but list of dependencies of rotate plugin is still unclear.

I suppose nothing breaks. Public API is unaffected.

Things to consider

Perhaps transformBitmap() should be promoted to public low-level API. Perhaps some plugins like flip can leverage it. For example, a whole flip vertically operation could be written as:

transformBitmap(
  img,
  img.getWidth(),
  img.getHeight(),
  (x,y) => [img.getWidth() - x - 1, y],
);

Tasks

• Add tests
• Update Documentation — not applicable IMO
• Update jimp.d.ts — I am not sure if it's needed here, no exported method has been changed
• Add SemVer Label — I don't know what should be done here, please advise

📦 Published PR as canary version: 0.10.2-canary.875.842.0

✨ Test out this PR locally via:

npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install [email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
npm install @jimp/[email protected]
# or 
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add [email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]
yarn add @jimp/[email protected]

[hipstersmoothie]

🚀 PR was released in v0.10.2 🚀