[GH-ISSUE #1445] Replace all min & max constructs with clamp #675

New issue

Closed

opened 2026-03-15 23:46:55 +03:00 by kerem · 1 comment

kerem commented

2026-03-15 23:46:55 +03:00

Owner

Originally created by @bluejekyll on GitHub (Apr 9, 2021).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/1445

This is a code cleanup. There are a lot of places in the code where min and max are applied to values to ensure they are between to valid min and max values. In Rust 1.50, clamp was introduced, this would be an improvement: https://doc.rust-lang.org/std/cmp/trait.Ord.html#method.clamp

Originally created by @bluejekyll on GitHub (Apr 9, 2021). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/1445 This is a code cleanup. There are a lot of places in the code where min and max are applied to values to ensure they are between to valid min and max values. In Rust 1.50, clamp was introduced, this would be an improvement: https://doc.rust-lang.org/std/cmp/trait.Ord.html#method.clamp

kerem

2026-03-15 23:46:55 +03:00

closed this issue
added the
crate:all

cleanup

easy
labels

kerem commented

2026-03-15 23:47:11 +03:00

Author

Owner

@chrismiceli commented on GitHub (Apr 30, 2021):

My attempt to resolve this issue is available for review https://github.com/bluejekyll/trust-dns/pull/1473

@chrismiceli commented on GitHub (Apr 30, 2021): My attempt to resolve this issue is available for review https://github.com/bluejekyll/trust-dns/pull/1473

kerem referenced this issue

2026-03-16 02:14:53 +03:00

[PR #675] [MERGED] give trust-dns a logo #1593