Skip to content

Conversation

@vitkuz573
Copy link
Contributor

@vitkuz573 vitkuz573 commented Nov 3, 2024

Implement LOWORD and HIWORD Macros for 32-Bit Value Extraction

Summary of Changes

  1. Macro Implementation

    • Added LOWORD and HIWORD macros to extract 16-bit segments from 32-bit values.
  2. Unit Tests

    • Introduced tests for LOWORD and HIWORD in MacroTests.cs to verify functionality with various test cases.
  3. Visibility Testing

    • Updated MacrosTests.cs to validate visibility settings for LOWORD and HIWORD.
  4. Native Methods Update

    • Included LOWORD and HIWORD in NativeMethods.txt for project reference.

Files Changed

  • src/Microsoft.Windows.CsWin32/templates/PInvokeClassMacros.cs
  • test/GenerationSandbox.Tests/MacroTests.cs
  • test/GenerationSandbox.Tests/NativeMethods.txt
  • test/Microsoft.Windows.CsWin32.Tests/MacrosTests.cs

Copy link
Member

@AArnott AArnott left a comment

Choose a reason for hiding this comment

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

Thank you!

@AArnott AArnott enabled auto-merge January 22, 2025 17:40
@AArnott AArnott disabled auto-merge January 22, 2025 20:51
@AArnott
Copy link
Member

AArnott commented Feb 7, 2025

/azp run

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

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.

2 participants