fboss/agent/switch_state.thrift (9 lines):
	- line 40: // TODO: replace with switch_config.MMUScalingFactor?
	- line 42: // TODO: replace with switch_config.QueueScheduling?
	- line 44: // TODO: replace with switch_config.StreamType?
	- line 63: // TODO: separate config and operational state
	- line 68: // TODO: use switch_config.PortState?
	- line 72: // TODO: use switch_config.PortSpeed?
	- line 74: // TODO: use switch_config.PortPause?
	- line 86: // TODO: this will deprecate port speed and port fec
	- line 92: // TODO: Current warmboot state doesn't have such field yet


fboss/agent/hw/sai/diag/DiagShell.cpp (8 lines):
	- line 208: // TODO: Customize
	- line 226: // TODO: block until repl loop is completed
	- line 317: // TODO: Log command output to Scuba
	- line 422: // TODO: Look into adding timeout for try_connect
	- line 423: // TODO: Look into adding context on which client is connected
	- line 425: // TODO: Look into how we can add timeout in future diffs
	- line 441: // TODO: Look into requesting results that take a long time
	- line 457: // TODO: Check platform specific termination criteria


fboss/agent/hw/sai/switch/SaiPortManager.cpp (6 lines):
	- line 46: // TODO fill these in when we have debug counter support in SAI
	- line 348: // TODO: do FDB entries associated with this port need to be removed
	- line 566: // TODO: support Preemphasis once it is also used
	- line 709: // TODO (srikrishnagopu): Use the read-only attribute
	- line 928: * TODO (srikrishnagopu) : Remove after rolling out AFE TRIM (S249471)
	- line 1122: * TODO: The ingress and egress mirror is cached in port handle


fboss/agent/IPv6Handler.cpp (6 lines):
	- line 101: // TODO: We could add an intfChanged() method to re-use the existing
	- line 171: // TODO:
	- line 241: // TODO: Add rate limiting so we don't generate too many requests for the
	- line 279: // TODO: Do we need to bother handling this yet?
	- line 461: // TODO: It might be nice to support duplicate address detection, and track
	- line 891: // TODO: add a constructor that doesn't require string processing


fboss/agent/hw/sai/switch/SaiSwitch.cpp (5 lines):
	- line 358: // FIXME : Stoo skipping for Tajo once T79717530 resolved
	- line 760: // TODO - compute used resource stats from internal data structures and
	- line 1476: // TODO: add counter to keep track of spurious rx packet
	- line 1520: // TODO: add counter to keep track of spurious rx packet
	- line 1611: * TODO: see if we can stop doing this at SwSwitch layre


fboss/py/fboss/cli/commands/port.py (5 lines):
	- line 61: # TODO: Make this more accurate.
	- line 78: # TODO: Make this function less complex
	- line 275: # TODO Try to call qsfp_service to handle xphy prbs logic first
	- line 294: # TODO Try to call qsfp_service to handle xphy prbs logic first
	- line 316: # TODO Try to call qsfp_service to handle xphy prbs logic first


fboss/agent/IPv4Handler.cpp (5 lines):
	- line 166: // TODO: assume vrf 0 now
	- line 190: // TODO: Also check to see if this is the broadcast address for one of the
	- line 218: // TODO: Also check to see if this is the broadcast address for one of the
	- line 226: // TODO: check the reason of punt, for now, assume it is for
	- line 236: // TODO: ideally, we need to store this packet until the ARP is done and


fboss/agent/hw/bcm/BcmPort.cpp (5 lines):
	- line 1093: // TODO: replace with pulling name from platform port
	- line 1205: // TODO: It would be nicer to use a monotonic clock, but unfortunately
	- line 1308: // TODO: Test if this is true when rx pause is enabled
	- line 1337: // TODO: outQueueLen_ only exports the average queue length over the last
	- line 1685: // TODO: we discovered a resource leak on this call when it


fboss/agent/hw/bcm/BcmSwitch.cpp (5 lines):
	- line 844: // TODO: We may want to trap ARP on a per-port or per-VLAN basis.
	- line 861: // TODO: We may want to trap NDP on a per-port or per-VLAN basis.
	- line 905: // TODO: Eventually the spanning tree state should be part of the Port
	- line 909: // TODO: This should eventually be done as part of applyConfig()
	- line 1149: // TODO: This function contains high-level logic for how to apply the


fboss/agent/packet/SflowStructs.h (5 lines):
	- line 63: // TODO (sgwang)
	- line 88: // TODO (sgwang)
	- line 93: // TODO (sgwang)
	- line 99: // TODO (sgwang)
	- line 104: // TODO (sgwang)


fboss/qsfp_service/module/sff/SffModule.cpp (4 lines):
	- line 30: // TODO: Since this is an extended experiment, we resue the aoi_override flag
	- line 96: // TODO: add support for turning off these fields via a command-line flag
	- line 268: // TODO: migrate all cable types
	- line 657: // TODO: does this really mean the cable is too long?


fboss/agent/hw/sai/tracer/AclApiTracer.cpp (4 lines):
	- line 157: // TODO: To be implemented if the function is used by fboss
	- line 163: // TODO: To be implemented if the function is used by fboss
	- line 170: // TODO: To be implemented if the function is used by fboss
	- line 179: // TODO: To be implemented if the function is used by fboss


fboss/agent/ApplyThriftConfig.cpp (3 lines):
	- line 1689: /* TODO t7153326: Following code is added for backward compatibility
	- line 1721: /* TODO t7153326: Following code is added for backward compatibility
	- line 3005: // TODO: For now we are allowing v4 LLs to be programmed because they are


fboss/agent/if/ctrl.thrift (3 lines):
	- line 403: 5: i64 speedMbps; // TODO: i32 (someone is optimistic about port speeds)
	- line 752: * TODO (allwync): get rid of getIpRoute after agent code with thrift
	- line 769: * TODO (allwync): get rid of getRouteTable after agent code with thrift


fboss/agent/state/SwitchState.cpp (3 lines):
	- line 65: // TODO: it might be worth splitting up limits for ecmp/ucmp
	- line 108: // TODO: add rest of fields as we convert them to thrifty
	- line 219: // TODO verify that created state here is internally consistent t4155406


build/fbcode_builder/CMake/FBPythonBinary.cmake (3 lines):
	- line 50: # TODO: On Windows we require the Python libraries as well.
	- line 187: # TODO: generate a main executable that will invoke Python3
	- line 420: # TODO: In the future it would be nice to support pre-compiling the source


fboss/lib/usb/GalaxyI2CBus.cpp (3 lines):
	- line 58: return false; // TODO: Return unknown instead
	- line 70: return false; // TODO: Return unknown instead
	- line 86: return false; // TODO: Return unknown instead


fboss/agent/platforms/wedge/WedgePort.cpp (3 lines):
	- line 35: * TODO: Not much code here yet.
	- line 139: // TODO: make it possible to generate a PortStatus struct solely
	- line 153: // TODO (aeckert): it would be nicer if the BcmPortGroup wrote its


fboss/platform/fw_util/fw_util.cpp (3 lines):
	- line 18: // TODO: will check platform information (maybe fboss whoami) to
	- line 24: // TODO: add else logic for other platforms
	- line 32: // TODO: Add file lock to prevent multiple instace of fw-util from running


fboss/agent/state/AclEntry.cpp (2 lines):
	- line 193: // TODO: remove all migration along with old ser/des after next disruptive push
	- line 624: // TODO: add qualifier in AclTableQualifier enum


fboss/agent/hw/switch_asics/Tomahawk4Asic.h (2 lines):
	- line 48: /* TODO: Mimicking TH3 size here, figure out the defaults for TH4*/
	- line 54: /* TODO: Mimicking TH3 size here, figure out the defaults for TH4*/


fboss/agent/state/Thrifty.h (2 lines):
	- line 218: // TODO: fill out extra fields as needed
	- line 265: // TODO: in future, FieldsT and ThrifT should be one type


installer/centos-7-x86_64/package-fboss.py (2 lines):
	- line 119: # TODO Future work planned
	- line 194: # TODO use rpmdev-newspec $FBOSS_BINS to create spec and edit it.


fboss/agent/hw/sai/tracer/SaiTracer.cpp (2 lines):
	- line 329: // TODO: For other APIs, create new API wrappers and invoke wrappedApi()
	- line 1266: // TODO: For other APIs, create new API wrappers and invoke


build/fbcode_builder/getdeps/platform.py (2 lines):
	- line 56: # TODO: Ideally, this function would inspect the current cgroup for any
	- line 88: # TODO: There may be some way to approximate an availability


fboss/agent/hw/bcm/BcmQcmManager.cpp (2 lines):
	- line 187: // TODO: This routine goes away once we use common ACL infra to create
	- line 527: // TODO reconfigure the QCM parameters in hardware


build/fbcode_builder/CMake/FBCMakeParseArgs.cmake (2 lines):
	- line 49: # TODO: We aren't extremely strict about error checking for one-value
	- line 125: # TODO: Our caller still won't be able to distinguish between an empty


fboss/agent/hw/bcm/BcmAPI.cpp (2 lines):
	- line 75: * TODO (skhare)
	- line 166: // TODO (skhare)


fboss/agent/SwSwitch.cpp (2 lines):
	- line 758: // TODO: Figure out the best way to handle errors here.
	- line 1773: // TODO - figure out a way to send full agent config


fboss/agent/state/Port.cpp (2 lines):
	- line 171: // TODO: store admin state as enum, not string?
	- line 181: // TODO: store speed as enum, not string?


fboss/cli/fboss2/commands/show/transceiver/CmdShowTransceiver.h (2 lines):
	- line 49: // TODO: explore performance improvement if we make all this parallel.
	- line 181: // TODO: sort here?


fboss/agent/MPLSHandler.cpp (2 lines):
	- line 54: // TODO: add ods counter for such dropped mpls packet
	- line 64: // TODO: handle labels punted to cpu


fboss/agent/hw/bcm/BcmTxPacket.cpp (2 lines):
	- line 125: // TODO: use VLAN_ETH_ZLEN when it is available in /linux/if_vlan.h
	- line 280: // TODO (xiangzhu) Will further optimizing the performance, possibly


fboss/agent/DHCPv4Handler.cpp (2 lines):
	- line 322: // TODO we should add router id information to the packet
	- line 370: // FIXME We should really forward this along unchanged.


fboss/cli/fboss2/CmdSubcommands.cpp (2 lines):
	- line 84: // TODO explore moving this check to a compile time check
	- line 103: // TODO explore moving this check to a compile time check


fboss/agent/hw/sai/store/SaiObject.h (2 lines):
	- line 212: // XXX TODO: side-effect mode does NOT work with optionals
	- line 230: // XXX TODO: side-effect mode does NOT work with optionals


fboss/platform/rackmon/Rackmon.cpp (2 lines):
	- line 42: // its 256 bytes wasted. but worth it. TODO use a
	- line 204: // TODO We probably need a timer to ensure we


fboss/lib/usb/BaseWedgeI2CBus.cpp (2 lines):
	- line 154: // TODO: remove this after we ensure exclusive access to cp2112 chip
	- line 172: // TODO: remove this after we ensure exclusive access to cp2112 chip


fboss/lib/platforms/PlatformProductInfo.cpp (2 lines):
	- line 108: // TODO remove LC once fruid.json is fixed on Galaxy Linecards
	- line 113: // TODO remove FAB once fruid.json is fixed on Galaxy fabric cards


fboss/agent/state/VlanMap.h (2 lines):
	- line 62: * TODO: Maintain a second index by name. Typically we have only a
	- line 82: * TODO: Maintain a second index by name. Typically we have only a


fboss/agent/hw/sai/hw_test/HwTestMirrorUtils.cpp (2 lines):
	- line 91: // TODO: add truncate check
	- line 145: // TODO: add truncate check


fboss/agent/hw/bcm/BcmPortQueueManager.cpp (2 lines):
	- line 278: * TODO: Scheduling params require using portGport instead of queueGport due
	- line 420: * TODO: Scheduling params require using portGport instead of queueGport due


build/fbcode_builder/getdeps.py (2 lines):
	- line 834: # TODO: we could just go ahead and build it here, but I
	- line 904: # TODO: Break up complex function


fboss/cli/fboss2/commands/show/interface/CmdShowInterface.h (2 lines):
	- line 46: // TODO: move this to a utility since it
	- line 75: // TODO: implement output


fboss/qsfp_service/QsfpServiceHandler.cpp (1 line):
	- line 322: // TODO:(shankaran) - when healthcheck fails remove the sak from set.


fboss/agent/packet/IPv6Hdr.h (1 line):
	- line 34: * TODO: Add support for IPv6 extension headers.


cmake/Agent.cmake (1 line):
	- line 24: # TODO (rsunkad) re-enable this library for use with linking with libmain


fboss/agent/platforms/common/utils/Wedge100LedUtils.cpp (1 line):
	- line 143: // TODO: adding retries adds tolerance in case the i2c bus is


fboss/agent/FsdbSyncer.h (1 line):
	- line 28: // TODO - change to AgentConfig once SwSwitch can pass us that


fboss/qsfp_service/lib/QsfpCache.h (1 line):
	- line 92: * TODO: could change api to take Port SwitchState objects down the


fboss/agent/TunIntf.cpp (1 line):
	- line 123: // TODO: if needed, we can adjust send buffer size, TUNSETSNDBUF


fboss/agent/hw/bcm/BcmSflowExporter.cpp (1 line):
	- line 266: // TODO: prob need to handle ret code?


fboss/agent/platforms/sai/SaiBcmYampPlatform.cpp (1 line):
	- line 35: // TODO skhare


fboss/cli/fboss2/CmdHandler.cpp (1 line):
	- line 130: // TODO: explore if the parsing library provides a better way to implement


fboss/agent/platforms/wedge/oss/WedgePlatform.cpp (1 line):
	- line 21: // TODO: Initialize the LEDs.  The LED handling code isn't open source yet,


fboss/agent/LookupClassUpdater.cpp (1 line):
	- line 298: // TODO - ideally we shouldn't care about whether


cmake/AgentHwSaiSwitch.cmake (1 line):
	- line 46: # TODO Support SAI_PHY_IMPL(T86789543)


fboss/agent/state/RouteNextHop.cpp (1 line):
	- line 62: //   at the SwitchState level seamlessly. TODO: test what happens to


common/logging/logging.h (1 line):
	- line 12: // TODO: actually implement this


fboss/agent/hw/sai/api/HwWriteBehavior.h (1 line):
	- line 16: * TODO - make HwWriteBehavior granularity be per swtich


fboss/qsfp_service/module/QsfpModule.cpp (1 line):
	- line 351: // TODO: there may be a better way of determining this rather than


fboss/fsdb/if/fsdb_oper.thrift (1 line):
	- line 17: // TODO: path extensions here like regex support


fboss/agent/RxPacket.h (1 line):
	- line 60: // TODO: only vrf 0 now


fboss/platform/fw_util/darwinFwUtil/UpgradeBinaryDarwin.cpp (1 line):
	- line 299: // TODO: Upgrade the fan_cpld through I2C


fboss/agent/hw/sai/switch/SaiHostifManager.cpp (1 line):
	- line 329: // TODO: Can we have reason code to a queue mapping that does not have


fboss/platform/rackmon/Register.cpp (1 line):
	- line 47: // TODO We currently do not need more than 32bit values as per


fboss/agent/hw/sai/switch/phy/SaiPortManager.cpp (1 line):
	- line 162: // TODO: Support programming dynamic tx_settings


fboss/agent/platform_config.thrift (1 line):
	- line 127: // TODO: Will deprecate the optional fields in PlatformConfig and start using


fboss/agent/state/NeighborEntry.h (1 line):
	- line 23: // TODO: remove this in favor of state::NeighborState in switch_state.thrift


fboss/agent/hw/bcm/BcmMultiPathNextHop.cpp (1 line):
	- line 25: // TODO:


fboss/agent/platforms/wedge/WedgePort.h (1 line):
	- line 59: // TODO: deprecate this


fboss/agent/SwitchStats.h (1 line):
	- line 235: // TODO: add support for addRepeatedValue() in FSDB client-side library


fboss/agent/hw/bcm/BcmQosPolicy.cpp (1 line):
	- line 174: #ifdef IS_OSS /* TODO: remove once OSS support added */


fboss/qsfp_service/platforms/wedge/Wedge100Manager.cpp (1 line):
	- line 14: // TODO: Will fully migrate I2CBusApi into TransceiverPlatformApi. Then we will


fboss/cli/fboss2/tabulate/table_internal.hpp (1 line):
	- line 190: // TODO: Maybe this can be configured?


fboss/agent/packet/TCPHeader.h (1 line):
	- line 83: // TODO support options


fboss/agent/LookupClassRouteUpdater.cpp (1 line):
	- line 24: * TODO get rid of this flag once configs have been updated to


fboss/agent/hw/sai/hw_test/HwTestNeighborUtils.cpp (1 line):
	- line 53: // TODO Lookup interface route and confirm that it points to CPU nhop


fboss/agent/packet/LlcHdr.h (1 line):
	- line 82: * TODO: Add support for Type 2 and Type 3 PDUs.


fboss/agent/ArpHandler.cpp (1 line):
	- line 184: // TODO: We need a more robust mechanism for setting up the ethernet


fboss/agent/ndp/IPv6RouteAdvertiser.cpp (1 line):
	- line 149: // TODO: In the future it would be nice to support allocating buf_ in a DMA


fboss/qsfp_service/lib/QsfpCache.cpp (1 line):
	- line 71: // TODO: explore using atomic exchanges on shared_ptr to get rid of lock.


fboss/agent/hw/sai/diag/DiagShellClient.cpp (1 line):
	- line 131: // TODO: fill in ClientInformation, or get rid of it in the API


fboss/agent/hw/switch_asics/GaronneAsic.h (1 line):
	- line 40: // TODO - verify this


fboss/agent/thrift_packet_stream/BidirectionalPacketStream.cpp (1 line):
	- line 269: // TODO:(shankaran) - Try to reconnect when there is a failure.


fboss/agent/hw/sai/hw_test/SaiSwitchEnsemble.cpp (1 line):
	- line 103: // TODO once hw shell access is supported


fboss/agent/platforms/sai/SaiBcmMinipackPlatform.cpp (1 line):
	- line 24: // TODO skhare


fboss/agent/packet/IPv4Hdr.cpp (1 line):
	- line 56: // TODO: check the checksum


build/fbcode_builder/getdeps/buildopts.py (1 line):
	- line 521: # TODO: `mkscratch` doesn't currently know how best to place things on


fboss/mdio/Mdio.h (1 line):
	- line 221: // TODO (ccpowers): remove this as soon as we've stopped using the accton xphy


fboss/qsfp_service/platforms/wedge/GalaxyManager.cpp (1 line):
	- line 23: // TODO: Will fully migrate I2CBusApi into TransceiverPlatformApi. Then we will


fboss/agent/hw/sai/hw_test/dataplane_tests/SaiAclTableGroupTrafficTests.cpp (1 line):
	- line 55: // TODO (skhare) Use ResourceGenerator to create this map, where the number


fboss/lib/phy/NullPortStats.h (1 line):
	- line 24: // TODO (ccpowers): remove this once all platforms have their own PortStats type


fboss/agent/packet/SflowStructs.cpp (1 line):
	- line 76: // TODO infer the flow records size instead and remove the input param.


fboss/agent/hw/bcm/BcmRxPacket.h (1 line):
	- line 54: // TODO (skhare) Simplify/fold into BcmRxPacket


fboss/agent/state/MatchAction.cpp (1 line):
	- line 92: // TODO: remove all migration along with old ser/des after next disruptive push


fboss/agent/hw/sai/api/SaiApi.h (1 line):
	- line 222: // TODO: assert on All<IsSaiAttribute>


build/fbcode_builder/docker_builder.py (1 line):
	- line 37: # system packages are installed.  TODO: For users not defined in the


fboss/mdio/FbFpgaMdio.cpp (1 line):
	- line 34: // TODO: read fpga version register and set up version_


fboss/agent/hw/HwSwitchStats.h (1 line):
	- line 61: // TODO: FSDB needs to support count() method on stats


fboss/cli/fboss2/utils/clients/BmcClient.cpp (1 line):
	- line 34: // TODO: Once BMC devices are provisioned with a non self-signed cert


fboss/agent/IPHeaderV4.cpp (1 line):
	- line 37: // TODO: other sanity checks (i.e. packet length, checksum...)


fboss/agent/lldp/LinkNeighbor.cpp (1 line):
	- line 287: // TODO: We don't actually verify the checksum right now.


fboss/qsfp_service/module/cmis/CmisModule.cpp (1 line):
	- line 1810: std::nullopt, // Part Number : TODO: Read and cache tcvrPartNumber


build/fbcode_builder/getdeps/dyndeps.py (1 line):
	- line 117: # TODO: audit that both source paths have the same inode number


fboss/agent/hw/bcm/BcmFieldProcessorFBConvertors.cpp (1 line):
	- line 121: // TODO: Stop doing this so we can use ECN


fboss/agent/hw/sai/switch/SaiBufferManager.cpp (1 line):
	- line 189: // TODO throw error if shared bytes is set. We don't handle that in SAI


fboss/agent/hw/bcm/BcmCosQueueManager.cpp (1 line):
	- line 28: // TODO (skhare) refactor and replace with right OpenNSA API


fboss/agent/hw/bcm/BcmWarmBootCache.h (1 line):
	- line 116: * TODO - Convert mask to mask len for efficient storage/lookup


fboss/agent/hw/sai/api/MirrorApi.h (1 line):
	- line 226: // TODO: Add SaiSflowMirrorTraits here.


fboss/agent/platforms/sai/SaiBcmPlatform.h (1 line):
	- line 25: // TODO: implement this


fboss/agent/hw/sai/diag/PythonRepl.cpp (1 line):
	- line 56: // TODO: what to put in argc/argv?!?


fboss/agent/state/QosPolicy.cpp (1 line):
	- line 242: // TODO: remove this check after another push


fboss/agent/state/Vlan.h (1 line):
	- line 233: * TODO - replace getNeighborEntryTable as getNeighborTable


fboss/agent/hw/bcm/BcmIntf.h (1 line):
	- line 96: // TODO: we now generate one station entry per interface, even if all


fboss/agent/rib/RoutingInformationBase.h (1 line):
	- line 196: * TODO: Consider breaking down update into add, del, syncClient


fboss/agent/state/NodeMapDelta.h (1 line):
	- line 126: // TODO: We should probably change this to store raw pointers.


fboss/agent/packet/ICMPHdr.cpp (1 line):
	- line 99: // TODO: clean up the EthHdr code and use EthHdr here


fboss/agent/hw/bcm/BcmPortUtils.cpp (1 line):
	- line 90: // TODO: add more mappings + better error message


fboss/qsfp_service/module/QsfpFieldInfo.h (1 line):
	- line 16: int dataAddress; // TODO: rename to page


fboss/agent/hw/sai/fake/FakeSaiPort.cpp (1 line):
	- line 207: // TODO: Use number of queues by querying SAI_SWITCH_ATTR_NUMBER_OF_QUEUES


fboss/agent/platforms/common/utils/Wedge400LedUtils.cpp (1 line):
	- line 15: // TODO: T70244223. Clarifying the right LED color for uplinks


fboss/agent/hw/bcm/BcmFieldProcessorUtils.cpp (1 line):
	- line 25: // TODO (skhare) Find OpenNSA method for this


fboss/agent/packet/PktFactory.h (1 line):
	- line 127: // TODO: support TCP segment


fboss/agent/hw/bcm/BcmWarmBootCache.cpp (1 line):
	- line 463: // TODO: Investigate why port_bitmap contains


fboss/platform/rackmon/RackmonThriftHandler.cpp (1 line):
	- line 23: // TODO interpret string source.deviceType to support ORv3 devices.


fboss/agent/hw/sai/fake/FakeSai.cpp (1 line):
	- line 251: // FIXME: implement this


fboss/agent/hw/switch_asics/EbroAsic.h (1 line):
	- line 40: // TODO - verify this


fboss/cli/fboss2/commands/show/port/CmdShowPort.h (1 line):
	- line 134: // TODO: should be case insensitive


fboss/agent/state/NeighborTable.h (1 line):
	- line 57: * TODO: We should switch from NodeMap to PrefixMap when the new PrefixMap


fboss/platform/sensor_service/hw_test/SensorsTest.cpp (1 line):
	- line 87: // TODO assert for non empty response once this thrift API is implemented


fboss/agent/MirrorManagerImpl.cpp (1 line):
	- line 52: // TODO: support configuring LAG egress for mirror


fboss/agent/packet/NDPRouterAdvertisement.h (1 line):
	- line 161: // TODO: add support for options


fboss/agent/hw/bcm/BcmCinter.h (1 line):
	- line 41: * TODO: bcm_rx_*


fboss/qsfp_service/platforms/wedge/Wedge40Manager.cpp (1 line):
	- line 13: // TODO: Will fully migrate I2CBusApi into TransceiverPlatformApi. Then we will


fboss/agent/hw/bcm/BcmEgress.cpp (1 line):
	- line 317: // TODO figure out why the following check fails


fboss/cli/fboss2/utils/Table.cpp (1 line):
	- line 88: // TODO: figure out why we can't use bottom_border on the header instead


fboss/lib/LogThriftCall.h (1 line):
	- line 126: // TODO: add version for Coro once coro based thrift server is standardized


build/fbcode_builder/getdeps/builder.py (1 line):
	- line 122: # TODO: Introduce some sort of defaults config that can select by


fboss/agent/hw/sai/api/LoggingUtil.h (1 line):
	- line 302: // TODO: implement this


build/fbcode_builder/getdeps/copytree.py (1 line):
	- line 75: TODO: I'd like to either extend this or add a variant that


fboss/agent/platforms/sai/SaiFakePlatform.h (1 line):
	- line 48: // TODO: add support in fake


fboss/mdio/FbFpgaMdio.h (1 line):
	- line 26: // TODO: use a static factory method when we add more versions


cmake/AgentHwTest.cmake (1 line):
	- line 214: # TODO: Compile takes too long. Commenting until addressing it


fboss/platform/fan_service/ServiceConfig.h (1 line):
	- line 299: // TODO - just use thrift enum names


fboss/agent/hw/switch_asics/TajoAsic.h (1 line):
	- line 18: // TODO - verify this


fboss/agent/hw/sai/api/SaiAttribute.h (1 line):
	- line 179: // TODO:


fboss/cli/fboss2/commands/show/interface/flaps/CmdShowInterfaceFlaps.h (1 line):
	- line 53: // TODO - figure out why getRegexCounters fails for OSS


fboss/agent/platforms/sai/SaiBcmWedge400Platform.cpp (1 line):
	- line 35: // TODO skhare


fboss/agent/hw/bcm/BcmQcmManager.h (1 line):
	- line 16: // TODO: rohitpuri


fboss/agent/hw/sai/api/NeighborApi.h (1 line):
	- line 66: // TODO MAYBE: parameterize (this? whole API?) by IP?


build/fbcode_builder/getdeps/py_wheel_builder.py (1 line):
	- line 158: # TODO: process the subdirectories of data_dir_name