fix!: explicitly mark _KGDTENTRY64 and _KIDTENTRY64 as opaque types in bindgen

[wmmc88]

This pull request includes several changes to the wdk-build crate to improve its functionality and maintainability. The most important changes include explicitly marking _KGDTENTRY64 and _KIDTENTRY64 as opaque types and adding new tests for the bindgen_header_contents method.

The new types are generated as:

#[repr(C)]
#[derive(Copy, Clone)]
pub union _KGDTENTRY64 {
    pub _address: u8,
}
pub union _KIDTENTRY64 {
    pub _address: u8,
}

and replace the historic structs that were manually added to the bindgen input header. These historic structs are no longer present in modern WDK headers, and these types should be treated as opaque.

Cfg Updates:

• crates/wdk-build/build.rs: Updated the cfg name from nightly_feature to nightly_toolchain to reflect its purpose of enabling when the toolchain is detected as nightly.

Blocklist Additions:

• crates/wdk-build/src/bindgen.rs: Changed _KGDTENTRY64 and _KIDTENTRY64 to be explicitly marked as opaque as they have no definitions in WDK.

Typo Fix:

• crates/wdk-build/src/lib.rs: Corrected a typo in a comment from "processs" to "process".

Code Refactoring:

• crates/wdk-build/src/lib.rs: Refactored the bindgen_header_contents method to use flat_map and collect for improved readability and performance. Removed the hardcoded definitions for KGDTENTRY64 and KIDTENTRY64.

Test Additions:

• crates/wdk-build/src/lib.rs: Added new tests for the bindgen_header_contents method to ensure it generates the correct headers for different driver configurations (Wdm, Kmdf, and Umdf).
   
   
  BREAKING CHANGE: definitions for _KGDTENTRY64 and _KIDTENTRY64 have been removed

[Copilot]

Copilot reviewed 24 out of 39 changed files in this pull request and generated 2 comments.

Files not reviewed (15)

• .vscode/settings.json: Language not supported
• crates/wdk-sys/src/input.h: Language not supported
• crates/wdk-sys/src/wdf.c: Language not supported
• .github/workflows/code-formatting-check.yaml: Evaluated as low risk
• .github/workflows/codeql.yml: Evaluated as low risk
• .github/workflows/build.yaml: Evaluated as low risk
• Cargo.toml: Evaluated as low risk
• crates/wdk-sys/src/ntddk.rs: Evaluated as low risk
• crates/wdk-build/src/bindgen.rs: Evaluated as low risk
• CONTRIBUTING.md: Evaluated as low risk
• .github/workflows/docs.yaml: Evaluated as low risk
• crates/wdk-sys/Cargo.toml: Evaluated as low risk
• crates/wdk-sys/src/lib.rs: Evaluated as low risk
• .github/workflows/test.yaml: Evaluated as low risk
• .github/workflows/lint.yaml: Evaluated as low risk

Comments suppressed due to low confidence (3)

crates/wdk-sys/src/hid.rs:13

• The reason parameter is not valid for the #[allow] attribute. It should be removed.

#[allow(missing_docs, reason = "most items in the WDK headers have no inline documentation, so bindgen is unable to generate documentation for their bindings")]

crates/wdk-sys/src/hid.rs:19

• The reason parameter is not valid for the #[allow] attribute. It should be removed.

#[allow(clippy::wildcard_imports, reason = "the underlying c code relies on all type definitions being in scope, which results in the bindgen generated code relying on the generated types being in scope as well")]

crates/wdk-sys/build.rs:47

• The usage of unreachable! macro might not be the best way to handle unexpected configurations. Consider using a more descriptive error message or handle the case more gracefully.

unreachable!("generate_wdf_function_table is only called with WDF driver configurations")