Commits on Feb 18, 2024

1. Remove unused variables in fbpcf/mpc_std_lib/unified_data_process/ser… …

   …ialization/RowStructureDefinition.h
   
   Summary:
   LLVM-15 has a warning `-Wunused-but-set-variable` which we treat as an error because it's so often diagnostic of a code issue. Unused variables can compromise readability or, worse, performance.
   
   This diff either (a) removes an unused variable and, possibly, it's associated code, or (b) qualifies the variable with `[[maybe_unused]]`, mostly in cases where the variable _is_ used, but, eg, in an `assert` statement that isn't present in production code.
   
    - If you approve of this diff, please use the "Accept & Ship" button :-)
   
   Reviewed By: palmje
   
   Differential Revision: D53779568

   

   r-barnes authored and facebook-github-bot committed Feb 18, 2024
   Configuration menu

   • View commit details
   • Copy full SHA for 90bae56
   • Browse repository at this point

   Copy the full SHA

   90bae56 View commit details

   Browse the repository at this point in the history