chriskohlhoff / asio

Asio C++ Library
http://think-async.com/Asio
4.85k stars 1.2k forks source link

Declaration of ‘result’ shadows a previous local #1249

Open tbeger opened 1 year ago

tbeger commented 1 year ago

asio/asio/include/asio/detail/impl/descriptor_ops.ipp: In function ‘bool asio::detail::descriptor_ops::set_user_non_blocking(int, asio::detail::descriptor_ops::state_type&, bool, asio::error_code&)’:

asio/asio/include/asio/detail/impl/descriptor_ops.ipp:120:9: error: declaration of ‘result’ shadows a previous local [-Werror=shadow]

120 | int result = ::fcntl(d, F_GETFL, 0);

  |         ^~~~~~

asio/asio/include/asio/detail/impl/descriptor_ops.ipp:115:7: note: shadowed declaration is here

115 | int result = ::ioctl(d, FIONBIO, &arg);

  |       ^~~~~~

asio/asio/include/asio/detail/impl/descriptor_ops.ipp: In function ‘bool asio::detail::descriptor_ops::set_internal_non_blocking(int, asio::detail::descriptor_ops::state_type&, bool, asio::error_code&)’:

asio/asio/include/asio/detail/impl/descriptor_ops.ipp:183:9: error: declaration of ‘result’ shadows a previous local [-Werror=shadow]

183 | int result = ::fcntl(d, F_GETFL, 0);

  |         ^~~~~~

asio/asio/include/asio/detail/impl/descriptor_ops.ipp:178:7: note: shadowed declaration is here

178 | int result = ::ioctl(d, FIONBIO, &arg);

  |       ^~~~~~
vinipsmaker commented 1 year ago

Fix landed in master: https://github.com/chriskohlhoff/asio/commit/2cf5d1a57527c8a98da161aded4850ab557e4eb0

Please close the issue.

SweatingEgg commented 1 year ago

OP said 2 lines: line#115, line#178. But, only line#115 fixed.