generatedunixname537391475639613

This commit introduces a **refactoring** within the **Hermes CDP (Chrome DevTools Protocol) API**, specifically targeting the `DomainState` component. It **improves efficiency** by updating the `add` and `remove` methods within the `DomainState::Transaction` class to leverage `emplace_back`. This internal optimization enhances how domain states are managed, potentially leading to minor performance gains during CDP interactions with Hermes.

This commit introduces **updates** to the **Link Layer Discovery Protocol (LLDP) agent** within the **FBOSS** network operating system. The changes are focused on the `fbcode/fboss/agent/lldp` module, indicating modifications to how the system discovers and advertises network device information. This work represents a **maintenance** or **enhancement** effort for the network's discovery capabilities. The impact is contained within the LLDP functionality, ensuring continued or improved interoperability with neighboring network devices.

I am sorry, but I cannot generate a detailed commit description without the actual commit message content and the file-level changes with semantic details. The provided input only includes a path as the commit message and an empty "File Changes" section. Please provide the full commit message and the detailed file changes to generate the description.

This commit implements a **performance optimization** within the **Thrift C++2 utility library**, specifically targeting the `SchemaToMetadata` component. It **refactors** the `genStructuredAnnotations` function in `thrift/lib/cpp2/util/SchemaToMetadata.cpp` to pre-allocate capacity for its internal `ret` vector. This enhancement improves memory efficiency and reduces reallocations, leading to faster and more resource-efficient generation of **Thrift schema metadata** for downstream services.

This commit introduces a **bug fix** to the **Fresco ScrollPerf sample application**, specifically within the `ContentProviderSimpleAdapter`. It adds a crucial null check for the `Cursor` object returned by content resolver queries. This prevents `NullPointerExceptions` that could arise if the query operation yields a null cursor, significantly enhancing the **robustness and stability** of the sample's data loading and display functionality.

This commit performs a **refactoring** within the **Bcm hardware agent** to explicitly control object lifecycle and ownership semantics. It **prevents copying and moving** for `BcmUdfManager` and `BcmUdfPacketMatcher` by deleting their respective operators, and makes `BcmWarmBootCache` non-copyable and non-movable by inheriting from `folly::NonCopyableNonMovable`. Furthermore, `BcmMplsTunnelSwitchT` now explicitly defaults move operations while deleting copy operations. This work enhances code correctness and robustness by enforcing single ownership for critical hardware-related objects within the **Bcm HAL**, preventing unintended object duplication or transfer.

This commit performs a **refactoring** within the **Broadcom (BCM) hardware abstraction layer** of the FBOSS agent to **enhance type safety and correctness**. It explicitly makes several key classes non-copyable and non-movable, preventing unintended object duplication or movement. Specifically, the `BcmAPI` class now inherits from `folly::NonCopyableNonMovable`, while `BcmAclEntry`, `BcmAclStat`, and `BcmCinter` have their copy and move constructors/assignment operators explicitly deleted. This **maintenance** work reduces the risk of subtle bugs related to object lifecycle management in the hardware interaction logic.

This commit **refactors** the special member functions for several **Broadcom (BCM) hardware agent classes** to optimize resource management and prevent unintended object copying. Specifically, `BcmLabelMap` now leverages **C++11 move semantics** by implementing default move constructors and assignment operators, while explicitly deleting its copy counterparts for efficient resource transfer. Concurrently, `BcmLabeledTunnel` and `BcmLogBuffer` have their copy and move constructors and assignment operators **explicitly deleted**, enforcing non-copyable and non-movable behavior for these resource-managing types. This enhances the **correctness and efficiency of object lifecycle management** within the `fboss/agent/hw/bcm` subsystem by precisely controlling how these critical objects are created, copied, moved, and destroyed.

This commit introduces a **refactoring** to enhance the **C++ object semantics and safety** within the **Broadcom (BCM) hardware abstraction layer** of the FBOSS agent. It **explicitly deletes copy and move constructors/assignment operators** for `BcmPortIngressBufferManager` and `BcmPortQueueManager`, and move operations for `BcmPort`. This ensures that instances of these critical **port management classes** cannot be accidentally copied or moved, preventing potential issues related to resource ownership or object identity. The change improves **code robustness and correctness** by enforcing non-copyable/non-movable behavior where appropriate for these hardware-bound objects.

This commit introduces **refactoring** changes to the **Broadcom (BCM) hardware agent** to enhance object lifetime management and prevent unintended object copies or moves. Specifically, it makes the `BcmStatUpdater` class **non-copyable and non-movable** by inheriting from `folly::NonCopyableNonMovable`. Additionally, the **move constructors and assignment operators are explicitly deleted** for `BcmSwitch` and `BcmSwitchEventCallback` (and its derived classes). This ensures that critical hardware-related objects within the `fboss/agent/hw/bcm` subsystem are handled with strict ownership semantics, improving the **robustness and correctness** of the hardware abstraction layer.

This commit **refactors** key classes within the **BCM hardware agent** to enforce their **non-copyable and non-movable** nature. Specifically, `BcmTeFlowTable` and `BcmTrunk` are made non-movable by explicitly deleting their move constructors and assignment operators. Furthermore, `BcmTrunkStats` and `BcmTrunkTable` now inherit from `folly::NonCopyableNonMovable`, preventing both copying and moving. This **design improvement** enhances the robustness of the **BCM hardware abstraction layer** by ensuring that these critical objects, which manage hardware resources, maintain unique ownership and state, preventing potential errors from accidental duplication or relocation.

This commit performs a **refactoring** of several core classes within the **Broadcom (BCM) hardware agent** to enhance **object lifecycle management** and **resource safety**. It explicitly disables copy and move operations for `BcmMirrorDestination`, `BcmMirror`, and `BcmMultiPathNextHop` classes by deleting their respective constructors and assignment operators, preventing unintended duplication or transfer of hardware-bound resources. The `BcmNextHop` class is also made non-copyable and non-movable by inheriting from `folly::NonCopyableNonMovable`. Additionally, virtual destructors for `BcmNextHop` and `BcmPlatformPort` are explicitly defaulted, ensuring proper cleanup. This change improves the **robustness** and **correctness** of the BCM hardware abstraction layer by preventing potential issues related to object slicing or incorrect resource handling.

This commit performs a **refactoring** within the **Broadcom hardware abstraction layer (HAL)** of the FBOSS agent to enhance object lifecycle management. It explicitly deletes move constructors and assignment operators for several core classes, including `BcmHALVector`, `BcmIngressFieldProcessorFlexCounter`, `BcmStation`, and `BcmIntf`. Additionally, `BcmHostIf` and `BcmHostTableIf` now inherit from `folly::NonCopyableNonMovable`, removing redundant explicit deletions. This change prevents accidental copying or moving of these critical objects, enforcing single ownership and improving the **robustness and correctness** of the `bcm` module's internal state management.

This commit performs **refactoring** on the **Meru800bfa platform mapping** components by modifying the `Meru800bfaP1PlatformMapping` and `Meru800bfaPlatformMapping` classes. It explicitly defines default destructors and **deletes move constructors and assignment operators** for both classes. This change improves class design by preventing unintended object copying or moving, thereby enhancing **resource management safety** and ensuring proper object lifecycle within the platform agent.

This commit performs **refactoring** within the **Thrift C++ transport layer** to improve class semantics and maintainability. It **deletes copy and move constructors and assignment operators for the `TSocket` class**, preventing unintended copying or moving of this critical transport object. Additionally, the destructors for `TZlibTransportException`, `TZlibTransportFactory`, and `TFramedZlibTransportFactory` are **defaulted**, simplifying their implementation by relying on compiler-generated cleanup. This work enhances the robustness and clarity of the **Thrift transport components** by enforcing proper object lifecycle management.

This commit performs a **refactoring** of the **`Icecube800bcPlatformMapping` class** within the **FBOSS agent's platform common utilities** for the Icecube800bc platform. It explicitly defines a default destructor `~Icecube800bcPlatformMapping` and **deletes the move constructor and move assignment operator**. This change enhances the **robustness and correctness** of the class by preventing unintended implicit move operations, ensuring proper object lifecycle management and resource handling for platform-specific mappings. This is a **maintenance** task to align with C++ best practices for classes that manage resources or have specific copy/move semantics.

This commit introduces a **refactoring** to improve object lifecycle management within the **Broadcom (BCM) hardware abstraction layer** of FBOSS. It explicitly **deletes move constructors and assignment operators** for key classes such as `BcmControlPlane`, `BcmCosQueueManager`, and `BcmEgressQueueFlexCounter`, preventing unintended object copies or moves. Additionally, `BcmError` now explicitly defaults all special member functions, and `BcmFacebookAPI::LogListener` is made non-copyable and non-movable. This work enhances **code hygiene and correctness** by clarifying object ownership and preventing potential issues related to implicit object semantics, thereby improving the **stability and maintainability** of the BCM hardware interface.

This commit introduces a **refactoring** change within the **BCM hardware agent** module, specifically impacting the **QoS mapping functionality**. It **explicitly deletes the move constructor and move assignment operator** for both the `BcmQosMap` and `BcmQosMapEntry` classes. This modification prevents unintended object moves, ensuring that these critical QoS-related objects are always copied or managed through their intended lifecycle, thereby improving code safety and predictability within the hardware abstraction layer.

This commit performs a **refactoring** of the **Broadcom (BCM) hardware agent** to enhance **object lifecycle management** and code clarity. It explicitly defines or deletes copy and move constructors/assignment operators for several **route counter** classes, including `BcmRouteCounterID`, `BcmRouteCounterBase`, `BcmRouteCounterTableBase`, and `BcmRouteFlexCounterTable`. Additionally, the `BcmSflowExporter` and `BcmSflowExporterTable` classes now inherit from `folly::NonCopyableNonMovable`, preventing unintended copying or moving of these objects. This **maintenance** work improves the robustness and predictability of critical components within the `fboss/agent/hw/bcm` module by enforcing clear object semantics.

This commit **refactors** the **`fboss/agent/thrift_packet_stream`** component by **deleting the copy and move constructors and assignment operators** for the `PacketStreamService` class in `PacketStreamService.h`. This change **prevents unintended copying and moving** of `PacketStreamService` objects, ensuring their unique ownership and proper lifecycle management. It improves the robustness of the **`PacketStreamService`** by enforcing a non-copyable, non-movable design. This **refactoring** helps prevent subtle bugs related to resource handling and ensures the integrity of `PacketStreamService` instances throughout their lifecycle.