Skip to content

Add EventAttributes.None - #676

Merged
Washi1337 merged 2 commits into
Washi1337:developmentfrom
Sergio0694:user/sergiopedri/add-eventattributes-none
Sep 22, 2025
Merged

Add EventAttributes.None#676
Washi1337 merged 2 commits into
Washi1337:developmentfrom
Sergio0694:user/sergiopedri/add-eventattributes-none

Conversation

@Sergio0694

Copy link
Copy Markdown
Contributor

For consistency with PropertyAttributes. Also added a couple tests.

Introduced the None value to the EventAttributes enum to explicitly represent events with no attributes. This improves clarity and completeness of the enum definition.
Added unit tests to verify EventDefinition correctly handles None, SpecialName, and RtSpecialName attributes, including IsSpecialName and IsRuntimeSpecialName properties.
@Sergio0694 Sergio0694 changed the title Add 'EventAttributes.None' Add EventAttributes.None Sep 19, 2025
@Washi1337 Washi1337 added enhancement dotnet Issues related to AsmResolver.DotNet labels Sep 22, 2025
@Washi1337 Washi1337 added this to the 6.0.0 milestone Sep 22, 2025
@Washi1337
Washi1337 merged commit 708f492 into Washi1337:development Sep 22, 2025
7 checks passed
@Washi1337

Copy link
Copy Markdown
Owner

Thanks!

@Sergio0694
Sergio0694 deleted the user/sergiopedri/add-eventattributes-none branch September 22, 2025 17:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dotnet Issues related to AsmResolver.DotNet enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants