fix(mod-morph): Only mask/clear mods when mod-morph is configured to do so - #3488
Open
urob wants to merge 2 commits into
Open
fix(mod-morph): Only mask/clear mods when mod-morph is configured to do so#3488urob wants to merge 2 commits into
urob wants to merge 2 commits into
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
This fixes a small inconsistency in mod-morph which currently always applies and clears the mod mask, even when configured not to mask anything. This matters in interaction with other mod-morph instances or other behaviors that modify the mod mask (example: #3161 (comment)). The fix adds a small conditional so that the mod mask is only set when
cfg->masked_modsis.