Sergey "Shnatsel" Davidoff

Document the status of resolver v2 support

3

Cargo has [made it possible to depend on the same version of a given crate with different feature sets](https://doc.rust-lang.org/cargo/reference/resolver.html#feature-resolver-version-2), provided that one version is a runtime dependency and another is...

Opt-in `std::simd` support

1

The [`std::simd`](https://doc.rust-lang.org/stable/std/simd/index.html) module provides a safe, portable API for SIMD. Together with the [`multiversion`](https://crates.io/crates/multiversion) crate it allows for SIMD that works from a single source file on every platform, and...

Use fuzzer to verify interoperability with LZ4 reference implementation

2

This PR depends on #3, please review that first. This verifies that: 1. Data encoded by the reference LZ4 implementation can be decoded by this code 2. Data encoded by...

Bug: some duplicates not detected

6

Some bit-identical files are not detected by Czkawka when searching for duplicates. Steps to reproduce: 1. Unpack this archive: [zune_png_divergences_99d956e64.tar.gz](https://github.com/qarmin/czkawka/files/10951552/zune_png_divergences_99d956e64.tar.gz) 2. Scan the folder for duplicates with default settings, observe...

Add a function to apply Exif rotation

4

This is required to display JPEG photos correctly. Doesn't wire it up to the JPEG decoding process yet, because that would be a semver-breaking change. But at least provides all...

Increased generation loss in JPEG encoder

1

In image v0.25.2, decoding and re-encoding JPEG 100 times makes the whole image go mostly green. A showcase and reproduction code can be found here: https://github.com/OetkenPurveyorOfCode/generation_loss

Implement a `rows()` iterator on `SubImage`

6

Right now the [`ImageBuffer`](https://docs.rs/image/0.25.2/image/struct.ImageBuffer.html) type has a `rows()` function to iterate over the rows of the image, but ~~[`GenericImageView`](https://docs.rs/image/0.25.2/image/trait.GenericImageView.html)~~ [`SubImage`](https://docs.rs/image/latest/image/struct.SubImage.html) does not. Working with rows rather than individual pixels is...

Cropping API doesn't check bounds, easy to misuse

7

In `image` v0.25.2, the `crop()` and `crop_imm()` functions return an image view or an image unconditionally, even if the area to be cropped is completely out of bounds. This makes...

`DynamicImage::crop()` is slow because `to_image()` is slow

1

In v0.25.2, the crop operation is implemented as: https://github.com/image-rs/image/blob/e176cd414ac6cc73909ff162e5d0b677f9d4fb08/src/dynimage.rs#L422 which obtains a view into an image, and then calls `to_image()` on it to turn it into a new image buffer....

The documentation on `crop()` is confusing

1

See https://docs.rs/image/0.25.2/image/enum.DynamicImage.html#method.crop In `image` v0.25.2 the documentation for `crop()` states: > Note: this method does _not_ modify the object, and its signature will be replaced with `crop_imm()`’s in the 0.24...