Skip to content

Conversation

@DimitriPapadopoulos
Copy link
Contributor

No description provided.

return not self == other

def may_interfer(self, dsts, src):
def may_interfere(self, dsts, src):
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think for this function you should provide a wrapper with the old name (marked as deprecated). This ensures any usages won’t straight up break…

Copy link
Contributor Author

@DimitriPapadopoulos DimitriPapadopoulos Mar 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'll move this change to a separate PR (#1476).

@W0ni
Copy link
Contributor

W0ni commented Mar 21, 2024

@serpilliere
I think it can be merged

@serpilliere serpilliere merged commit 1ba37c3 into cea-sec:master Mar 28, 2024
@serpilliere
Copy link
Contributor

Thank you for the fix @DimitriPapadopoulos !

@DimitriPapadopoulos DimitriPapadopoulos deleted the codespell branch March 28, 2024 16:35
DimitriPapadopoulos pushed a commit to DimitriPapadopoulos/miasm that referenced this pull request Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants