Joe-Abraham commented on issue #322: URL: https://github.com/apache/iceberg-cpp/issues/322#issuecomment-4019318667
I've created a PR to resolve the build warnings reported here. The fixes have been split into logical commits to make the review process easier: 1. **Fixed unused parameter warnings**: Added `[[maybe_unused]]` where appropriate. 2. **Fixed uninitialized member warnings**: Added default initializers to struct and class members. 3. **Fixed integer sign comparison warnings**: Used `<utility>` functions like `std::cmp_equal`, `std::cmp_less`. 4. **Fixed various other build warnings**: Addressed specific operator precedence warnings and redundant `std::move` calls. 5. **Cleaned up shadowing warnings/unused functions**: Renamed shadowed arguments and removed unused local test helpers. The PR should be linked soon! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
