Skip to content

Commit 615250b

Browse files
author
Oron Port
committed
fix ToggleIO default value initialization issue
1 parent 06e32c9 commit 615250b

File tree

2 files changed

+4
-3
lines changed

2 files changed

+4
-3
lines changed

lib/src/main/scala/dfhdl/platforms/resources/Button.scala

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,5 @@ import dfhdl.compiler.ir.PhysicalNumber.Ops.Hz
66
enum Button extends Encoded.Toggle:
77
case Released, Pressed
88

9-
object Button extends ToggleIOComp[Button](Button.Pressed, maxFreqMinPeriod = 10.Hz)
9+
//TODO: No need for `None` when https://github.com/scala/scala3/issues/24201 is fixed
10+
object Button extends ToggleIOComp[Button](Button.Pressed, None, maxFreqMinPeriod = 10.Hz)

lib/src/main/scala/dfhdl/platforms/resources/ToggleIO.scala

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,11 +27,11 @@ object ToggleIO:
2727
end ToggleIO
2828

2929
abstract class ToggleIOComp[T <: Toggle](
30-
defaultActiveState: T,
30+
defaultActiveState: => T,
3131
standard: ConfigN[constraints.IO.Standard] = None,
3232
maxFreqMinPeriod: ConfigN[RateNumber] = None
3333
):
34-
class Resource private[ToggleIOComp] (val activeState: T = defaultActiveState)
34+
class Resource private[ToggleIOComp] (val activeState: T)
3535
extends ToggleIO[T]:
3636
injectConstraint(constraints.IO(standard = standard))
3737
if (maxFreqMinPeriod != None)

0 commit comments

Comments
 (0)