[Comment] Invalid URL #20967

Closed
opened 2026-01-31 07:29:14 +00:00 by claunia · 2 comments
Owner

Originally created by @JeremieBergeron on GitHub (Dec 7, 2023).

The link at the line 591 of src/renderer/gdi/state.cpp redirect to an error page.

It would be appreciated if it could be updated to the right link.

20dad62471/src/renderer/gdi/state.cpp (L588-L593)

Originally created by @JeremieBergeron on GitHub (Dec 7, 2023). The [link at the line 591 of src/renderer/gdi/state.cpp](https://github.com/microsoft/terminal/blob/20dad624710249f3a4ee77c47a58d4ca367a820b/src/renderer/gdi/state.cpp#L591 ) redirect to an error page. It would be appreciated if it could be updated to the right link. https://github.com/microsoft/terminal/blob/20dad624710249f3a4ee77c47a58d4ca367a820b/src/renderer/gdi/state.cpp#L588-L593
claunia added the In-PRIssue-DocsProduct-MetaArea-CodeHealth labels 2026-01-31 07:29:14 +00:00
Author
Owner

@zadjii-msft commented on GitHub (Dec 11, 2023):

As a note, that LOC goes back to the original open-source release of the Terminal. That link is probably super old - I sure have no memory of what article that may have pointed to in the first place

@zadjii-msft commented on GitHub (Dec 11, 2023): As a note, that LOC goes back to the original open-source release of the Terminal. That link is probably super old - I sure have no memory of what article that may have pointed to in the first place
Author
Owner

@AvishekDas5 commented on GitHub (Sep 23, 2025):

Hi! I noticed this issue and fixed the broken MSDN link.
The original link redirected to a removed article which I found here and now republished here.

I’ve opened a PR #19372 that updates the link to the correct Microsoft Learn page.

@AvishekDas5 commented on GitHub (Sep 23, 2025): Hi! I noticed this issue and fixed the broken MSDN link. The original link redirected to a removed article which I found [here](https://mskb.pkisolutions.com/kb/94646) and now republished [here](https://learn.microsoft.com/en-us/windows/win32/gdi/character-widths). I’ve opened a PR #19372 that updates the link to the correct Microsoft Learn page.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#20967