-
Notifications
You must be signed in to change notification settings - Fork 4.9k
Fix Console Title test when running on RS2 #24016
Conversation
CC @danmosemsft |
|
||
if (newTitle.Length > 513 && PlatformDetection.IsWindowsRedStone2) | ||
{ | ||
// RS2 has a bug when getting the windnow title when the title length is longer than 513 character |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
typo: windnow window
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we file an issue to delete this workaround at some point? I guess we will stop caring about this when everyone has stopped using the problematic builds.
Windows team decided not to fix this for RS2. if we are going to enusre in the future we'll not run on RS2, then yes I can track that with issue. But I don't think this will be the case. we'll live with that for good. |
Thanks @EricaMo for catching the typo :-) |
* Fix Console Title test when running on RS2 * Fix the typo Commit migrated from dotnet/corefx@4be1dea
No description provided.