- Start Date: 2023-02-07
- RFC PR: amaranth-lang/rfcs#5
- Amaranth Issue: amaranth-lang/amaranth#754
Remove Const.normalize
Summary
Remove Const.normalize(value, shape)
.
Motivation
From the name it is not clear what it is supposed to achieve (it's truncation and inversion according to the shape) and it does not check types of arguments.
We already have Const(value, shape).value
and most developers should be aware of it. Having Const.normalize(value, shape)
as well provides no benefit over the former. It's also longer.
Explanation
The Const.normalize
method is deprecated (with the suggestion to use Const().value
) and removed.
Drawbacks
- Churn.
Rationale and alternatives
We could keep it. Removing it reduces the API surface and makes the language a bit more elegant.
Prior art
None.
Unresolved questions
None.
Future possibilities
None.