Fix TO() and DF() calling layer_state_set_[kb,user] twice #6003
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When using
TO()
andDF()
,layer_state_set_user()
was observed to be called twice. This was not a keydown/keyup thing - the code first does anAND
, then anOR
, which both calllayer_state_set()
, which in turn callslayer_state_set_kb()
... Instead of doing that, we can directly set the layer state here.Types of Changes
Issues Fixed or Closed by This PR
Checklist