bloomberg / rmqcpp

A batteries included C++ RabbitMQ Client Library/API.
Apache License 2.0
68 stars 16 forks source link

Refactor framer test to fix unused variable warning #20

Closed ahsanabbas123 closed 11 months ago

ahsanabbas123 commented 11 months ago

Problem statement

Building with -Wall -Werror on linux was giving a -Werror=ununsed-variable

Proposed changes

Changed to directly check value in assert

This passes the build for consuming from vcpkg to make the port in https://github.com/microsoft/vcpkg/pull/34797 work.