MsGraphicsPkg/Library/SimpleUIToolKit/ListBox.c (7 lines):
	- line 178: // TODO - need to pass in font color.
	- line 184: // TODO - need to pass in font color.
	- line 319: // TODO - Hover.
	- line 332: // TODO - need to pass in font color.
	- line 338: // TODO - need to pass in font color.
	- line 388: UINT32  CheckBoxHeight          = (CheckBoxHitAreaHeight / 3);  // TODO - 1/3 the height of a listbox cell?
	- line 425: // TODO - for checkbox type listbox cells, how to handle text indent - indent checkbox as well?  For now


MsGraphicsPkg/DisplayEngineDxe/FormDisplay.c (6 lines):
	- line 1399: OrigY += (ControlRect.Bottom - ControlRect.Top) + 40;       // TODO - appropriate buffer between listbox label and listbox?
	- line 1478: // TODO - allocated buffer needs to be cleaned up when we exit but not before the listbox is done with it...
	- line 1564: ((MsUiGetLargeFontHeight () == FontInfo.FontSize) ? &gMsColorTable.LabelTextLargeColor : &gMsColorTable.LabelTextNormalColor),    // TODO - Foreground (text) color.
	- line 1716: TextColor,    // TODO - Foreground (text) color.
	- line 1783: (MsUiGetLargeFontHeight () == FontInfo.FontSize ? &gMsColorTable.LabelTextLargeColor : &gMsColorTable.LabelTextNormalColor),    // TODO - Foreground (text) color.
	- line 2137: // TODO:  Remove this filter when implementing hover.


MsGraphicsPkg/Library/SwmDialogsLib/PasswordDialog.c (5 lines):
	- line 46: // TODO: These should move to a UNI file so they can later be localized.
	- line 58: // TODO - better way to handle sharing these between the local functions?
	- line 480: (VOID *)SWM_MB_IDOK          // TODO - not the best way to do this.
	- line 515: (VOID *)SWM_MB_IDCANCEL          // TODO - not the best way to do this.
	- line 938: // TODO - avoid having to cast a constant value from a pointer.


MsGraphicsPkg/OnScreenKeyboardDxe/OnScreenKeyboardDriver.c (5 lines):
	- line 10: // TODO list:
	- line 1811: // TODO - Maintain rotation angle and origin when the scale is changed
	- line 2472: // TODO - need to optimize this routine and/or point set...
	- line 3304: // TODO - Close Protocol on Absolute Pointer if an error occurs?
	- line 3336: // TODO - Needs to be implemented


MsGraphicsPkg/Library/SwmDialogsLib/SemmUserAuthDialog.c (5 lines):
	- line 46: // TODO: These should move to a UNI file so they can later be localized.
	- line 57: // TODO - better way to handle sharing these between the local functions?
	- line 575: (VOID *)SWM_MB_IDOK          // TODO - not the best way to do this.
	- line 610: (VOID *)SWM_MB_IDCANCEL          // TODO - not the best way to do this.
	- line 1031: // TODO - avoid having to cast a constant value from a pointer.


MsGraphicsPkg/Library/SwmDialogsLib/MessageBox.c (4 lines):
	- line 294: //TODO: Change the MessageBox Colors to Themes just like PasswordDialog and replace this.
	- line 323: (VOID *)(UINTN)Button2Code          // TODO - not the best way to do this.
	- line 359: (VOID *)(UINTN)Button1Code          // TODO - not the best way to do this.
	- line 720: // TODO - avoid having to cast a constant value from a pointer.


XmlSupportPkg/Library/XmlTreeLib/fasterxml/xml_fasterxml.c (3 lines):
	- line 1391: // TODO: Naive implementation gathers an identifier and compares it
	- line 1392: // TODO: to the string.  Smarter implementations gather a character
	- line 1393: // TODO: at a time and compare as they go along.


MfciPkg/MfciDxe/MfciDxe.c (3 lines):
	- line 785: // TODO: Telemetry here
	- line 911: // TODO uncomment the below after debugging is complete
	- line 913: // TODO: Log telemetry for any errors that occur.


MsGraphicsPkg/Library/SwmDialogsLib/SingleSelectDialog.c (3 lines):
	- line 12: // TODO: Figure out how many options can realistically fit in the window
	- line 52: // TODO: These should move to a UNI file so they can later be localized.
	- line 703: // TODO - avoid having to cast a constant value from a pointer.


MsGraphicsPkg/Library/SimpleUIToolKit/ToggleSwitch.c (2 lines):
	- line 466: // TODO - bounds should apply to the toggle switch and it's text, combined.
	- line 763: this->m_pToggleSwitch->pToggleSwitchOnText  = pToggleSwitchOnText;      // TODO - need to allocate buffer and save?


MsWheaPkg/Include/Library/MuTelemetryHelperLib.h (2 lines):
	- line 13: // TODO: This is against common Mu practice.
	- line 16: // TODO: Make this a MACRO so it can be disabled.


DfciPkg/IdentityAndAuthManager/AuthManager.c (2 lines):
	- line 188: // TODO: add anti-hammering attack mitigation
	- line 205: // TODO: Save state for anti-hammering attack mitigation


MsCorePkg/MuVarPolicyFoundationDxe/MuVarPolicyFoundationDxe.c (2 lines):
	- line 72: // TODO VARPOL: Telemetry.
	- line 291: // TODO VARPOL: Telemetry.


MsGraphicsPkg/Library/SimpleUIToolKit/Utilities.c (2 lines):
	- line 13: #define MS_DEFAULT_FONT_SIZE       MsUiGetStandardFontHeight ()   // Default font size is 32px high.  (TODO - merge with MsDisplayEngine.h copy).
	- line 156: // TODO - Need a better way to determine this.  Currently hard-coded based on knowledge of the custom registered fonts in the Simple Window Manager driver.


MsGraphicsPkg/Library/SimpleUIToolKit/Label.c (1 line):
	- line 250: // TODO - this needn't be a separate structure from the label context structure.


UefiTestingPkg/AuditTests/UefiVarLockAudit/UEFI/LockTestXml.c (1 line):
	- line 110: //TODO:  this code should change to do string compare on attribute name.


DfciPkg/Application/DfciMenu/DfciMenu.c (1 line):
	- line 746: //  TODO: Verify requirement for on premises network


PcBdsPkg/PcBdsPkg.ci.yaml (1 line):
	- line 28: "ShellPkg/ShellPkg.dec"  # this needs to be reviewed.  TODO: bug??


MsGraphicsPkg/Library/SimpleUIToolKit/Canvas.c (1 line):
	- line 583: // TODO: Need to check whether there's already a default control?


MsCorePkg/Library/DxeQueueUefiVariableLib/DxeQueueUefiVariableLib.c (1 line):
	- line 22: // TODO: should this be a PCD? So that it can be modified?


MsCorePkg/Library/DxeIsCapsuleSupportedLib/CapsuleEsrtTableLayer.c (1 line):
	- line 33: // TODO: query the ESRT cache to determine if the FWClass matches an entry


DfciPkg/Application/DfciMenu/DfciSARecovery.c (1 line):
	- line 539: //  TODO: Verify requirement for on prem network


MsWheaPkg/Library/MuTelemetryHelperLib/MuTelemetryHelperLib.c (1 line):
	- line 178: // TODO: Consider doing this with a stack buffer and max size for maximum


UefiTestingPkg/FunctionalSystemTests/HeapGuardTest/App/Arm/UefiHardwareNxProtection.c (1 line):
	- line 19: //  TODO: Does ARM have an equivelent "EFER BIT" to check if NX protections are on?


MsGraphicsPkg/Library/SimpleUIToolKit/Button.c (1 line):
	- line 441: // TODO - function should take a flag to indicate default selection.


XmlSupportPkg/Library/UnitTestResultReportJUnitFormatLib/UnitTestResultReportLib.c (1 line):
	- line 282: //TODO:  need to handle timing.  Right now its hard coded to 1 second.


MsGraphicsPkg/SimpleWindowManagerDxe/WindowManager.c (1 line):
	- line 1368: // TODO - Configure Absolute Pointer protocol's mode structure.


DfciPkg/IdentityAndAuthManager/IdentityManager.c (1 line):
	- line 41: //TODO: add any security filtering here.  For example if we add a Auth Timeout or max access count