Conversation
The original text
::{CLSID for Control Panel}\::{CLSID for printers folder}
doesn't match the previous sentence. Furthermore, double-GUID Control Panel monikers are a legacy pattern.
I replaced the original text with the Control Panel GUID = {5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0}
|
iPhilip : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
John Kennedy (GrantMeStrength)
left a comment
There was a problem hiding this comment.
The replacement CLSID {5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0} needs verification. The original text showed a two-level virtual folder path (::{Control Panel CLSID}\::{Printers CLSID}) as an example, but the replacement is a single-level CLSID. The CLSID 5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0 appears to be for the Hardware and Sound control panel page, not the Printers folder. For a complete, accurate example, please provide the full virtual folder path with both CLSIDs explicitly, e.g.: ::{21EC2020-3AEA-1069-A2DD-08002B30309D}\::{2227A280-3AEA-1069-A2DE-08002B30309D} (Control Panel → Printers), and cite a reference.
|
Copilot /review |
John Kennedy (GrantMeStrength)
left a comment
There was a problem hiding this comment.
Review: Needs Verification
This PR replaces the generic CLSID placeholder with {5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0}.
The well-known Control Panel CLSIDs are:
- Classic:
{21EC2020-3AEA-1069-A2DD-08002B30309D} - New (Vista+):
{26EE0668-A00A-44D7-9371-BEB064C98683}
Please verify that {5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0} is a valid/appropriate CLSID for the scenario documented here.
|
iPhilip : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. Jim Walker (@jwmsft) |
The original text
doesn't match the previous sentence.
Furthermore, double-GUID Control Panel monikers are a legacy pattern. Thus, I replaced the original text with the Control Panel GUID:
::{5399E694-6CE5-4D6C-8FCE-1D8870FDCBA0}