encode dimIndex ranges

[burrbull]

cc @adamgreig
Partially related to stm32-rs/stm32-rs#701 (comment)

[burrbull]

cc @Emilgardis

[bors]

✌️ burrbull can now approve this pull request. To approve and merge a pull request, simply reply with bors r+. More detailed instructions are available here.

[adamgreig]

LGTM. Maybe worth fixing the clippy lint about the return Some(min..=max);?

I also think the max.wrapping_sub(min) could just be (max - min) since it can't ever underflow, but it doesn't really matter here.

bors d+

[burrbull]

Fixed.
bors r+

[bors]

👎 Rejected by too few approved reviews

[adamgreig]

hah, what use is bors delegating if as soon as you push a commit it revokes approval?

bors r+

[bors]

Build succeeded:

• build (1.46.0)
• build (stable)
• clippy_check
• test
• test-strict

[therealprof]

hah, what use is bors delegating if as soon as you push a commit it revokes approval?

bors r+

Yeah, need to change the configuration to not dismiss a review after a new commit.

[Emilgardis]

hah, what use is bors delegating if as soon as you push a commit it revokes approval?
bors r+

Yeah, need to change the configuration to not dismiss a review after a new commit.

it's quite easy to change branch protection rules, should we?

[therealprof]

I'm all for it, especially for projects with little risk of abuse (like sneaking in changes after approval of unmergeable PRs). However we'd have to be more permissive in our rules: https://github.com/rust-embedded/wg/blob/master/ops/post-transfer.md#repository-settings