search

osmcode / osmium-tool

Command line tool for working with OpenStreetMap data based on the Osmium library.
https://osmcode.org/osmium-tool/
GNU General Public License v3.0
509 stars 107 forks source link

Compiler warning: unreferenced local variable #203

Closed sphoto closed 3 years ago

sphoto commented 3 years ago

While building osmium-tool (using VS2015 on Win10) some warnings occur, due to libosmium commit efd7e07 (timer.hpp)

D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1110): warning C4101: 'timer_simpl
e_case': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1028): warning C4101: 'timer_sort'
: unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1111): warning C4101: 'timer_compl
ex_case': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1079): warning C4101: 'timer_split
': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1035): warning C4101: 'timer_dupl'
: unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1059): warning C4101: 'timer_inter
section': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1071): warning C4101: 'timer_locat
ions_list': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
D:\projects\osmium\20201206\libosmium\include\osmium/area/detail/basic_assembler.hpp(1148): warning C4101: 'timer_roles
': unreferenced local variable [d:\projects\osmium\20201206\osmium-tool\build\src\osmium.vcxproj]
joto commented 3 years ago

Thanks for the report. I believe this has been fixed in libosmium 5e72f050 .

sphoto commented 3 years ago

Warnings are gone now, thanks!