[Stephen Griffin (stephenegriffin)]

Summary

Fixes #854.

• Normalize fetched and caller-supplied property values together before selecting an editor, preventing a returned PT_ERROR value from being interpreted as a GUID pointer.
• Treat MAPI_E_NOT_FOUND for a requested non-error type as missing input so the requested property can be created; otherwise select the actual returned property type.
• Retain fetched-property ownership until the dialog workflow completes and preserve caller ownership of supplied values.
• Include the latest main. The diff against main is limited to property input normalization and its clang-format 17 formatting.

Validation

• Existing Debug x64 and x86 build tasks pass with no warnings or errors.
• Complete existing native suites pass: 261/261 on x64 and 261/261 on x86.
• All 433 CI-checked C/C++ files pass clang-format 17 using the workflow's options.
• The existing unit-test project does not exercise this editor-selection path. Follow-up Expand native test coverage to property editors and multivalue row editing #1022 now explicitly includes Access Violation when Editing a PT_CLSID Property #854 regression cases for input normalization, editor selection, supplied/fetched values, and ownership on acceptance/cancellation. Earlier local source-extracted policy checks are not claimed as persistent native regression coverage.

[github-actions]

Test Results

    8 files  ±0      8 suites  ±0   54s ⏱️ ±0s
  261 tests ±0    261 ✅ ±0  0 💤 ±0  0 ❌ ±0 
2 088 runs  ±0  2 088 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit d3b2953. ± Comparison against base commit 7f1fdb8.

♻️ This comment has been updated with latest results.