databendlabs / openraft / 20548630084

88%

Build Type

push

github

Committed by drmingdrmer

Commit Message

change: in 0.10.0: simplify `Extensions` API to single `get()` method

Simplify the `Extensions` type to require all stored values implement
`Default`. The `get()` method now auto-inserts `T::default()` if no
value exists.

Changes:
- Remove `new()` method, use `Default` impl instead
- Remove `insert()` method
- Remove `get_or_insert_with()` method
- Remove `get_or_default()` method
- Change `get()` to return `T` instead of `Option<T>`, auto-inserting default
- Require stored types to implement `Default`
- Add `Raft::extension::<T>()` convenience method wrapping `extensions().get::<T>()`

Upgrade tip:

Replace method calls:
- `Extensions::new()` → `Extensions::default()`
- `extensions.insert(val)` → (remove, values are auto-inserted via `get()`)
- `extensions.get::<T>().unwrap()` → `extensions.get::<T>()`
- `extensions.get_or_default::<T>()` → `extensions.get::<T>()`
- `extensions.get_or_insert_with(|| val)` → `extensions.get::<T>()`

All types stored in Extensions must now implement `Default`.

Run Details

28 of 28 new or added lines in 2 files covered. (100.0%)

9 existing lines in 4 files now uncovered.

15677 of 17717 relevant lines covered (88.49%)

159142.49 hits per line