You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In the current codebase we have EditLabel StateID String, which suggests that the Machine type provides an interface for entering an edit state when a StateID is selected. However, the way its used in Building.elm disobeys this assumption, as the ID can be both a StateID and TransitionID:
In fact, it just ignores the ID and instead checks the state of the machine. In npda-v3, I refactored this into EditStateLabel and EditTransitionLabel, which then has a one-to-one relationship to EditingStateLabel and EditingTransitionLabel respectively.
The text was updated successfully, but these errors were encountered:
In the current codebase we have
EditLabel StateID String
, which suggests that the Machine type provides an interface for entering an edit state when a StateID is selected. However, the way its used in Building.elm disobeys this assumption, as the ID can be both a StateID and TransitionID:finsm/src/Building.elm
Lines 347 to 360 in 398f02c
In fact, it just ignores the ID and instead checks the state of the machine. In npda-v3, I refactored this into
EditStateLabel
andEditTransitionLabel
, which then has a one-to-one relationship toEditingStateLabel
andEditingTransitionLabel
respectively.The text was updated successfully, but these errors were encountered: