Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

build error with __builtin_addressof #370

Closed
hwulfmeyer opened this issue Jul 24, 2024 · 1 comment
Closed

build error with __builtin_addressof #370

hwulfmeyer opened this issue Jul 24, 2024 · 1 comment

Comments

@hwulfmeyer
Copy link

hwulfmeyer commented Jul 24, 2024 •
edited
Loading

We are running debian 9 with gcc/g++ 6.3.0 and encountered compile errors with version 3.5.0 (first time using sick_scan_xd).

cmake -DROS_VERSION=0 -DLDMRS=0 -DSCANSEGMENT_XD=0 -G "Unix Makefiles" ../

error: ‘__builtin_addressof’ was not declared in this scope

usage in:

sick_scan_xd/roswrap/src/include/boost_wrap/sp_counted_impl.hpp

Line 176 in 6bdc5a3

return ti == BOOST_SP_TYPEID(D)? boost::detail::get_local_deleter( __builtin_addressof( del ) ): 0;

sick_scan_xd/roswrap/src/include/boost_wrap/sp_counted_impl.hpp

Line 269 in 6bdc5a3

return ti == BOOST_SP_TYPEID(D)? boost::detail::get_local_deleter( __builtin_addressof( d_ ) ): 0;

Looking around for a while I found that replacing __builtin_addressof with std::addressof() solves the errors.
@rostest
Copy link
Collaborator

rostest commented Jul 26, 2024

Thanks for your feedback and contribution. We will replace __builtin_addressof expressions in the next release.

@rostest rostest closed this as completed Aug 15, 2024
weinmalSICKAG added a commit that referenced this issue Oct 23, 2024
@weinmalSICKAG
Merge tag '3.6.0' into develop
8e560a8 
* add: Launchfiles and configuration for picoScan120
* add: Optional AngleRangeFilter and IntervalFilter for picoScan
* fix: Obsolete topic "/sick_multiscan/scan" removed
* add: IMU automatically deactivated after receiving an error code for IMU activation from picoScan w/o addons
* fix: Customization of hash values for authorization #366
* fix: Replaced builtin_addressof expressions #370
* add: Different UDP timeouts for state initial and running, improved UDP timeout handling
* fix: Picoscan range_min value in laserScan message #382
* add: Support for RMS2xxx LIDoutputstate telegrams
* fix: sick_generic_caller debug assertion #385
* add: Check of udp receiver ip at startup
* add: cmake-option to overwrite optimization level
* change: Documentation restructured
* add: Improved field evaluation TiM7xx, Tim7xxS (publish LIDinputstate messages, configuration and services for options FieldSetSelectionMethod and ActiveFieldSet)
* fix: PicoScan parameter add_transform_xyz_rpy #399
* fix: LMS4000 encoder settings #403
* fix: CMake-flag for target sick_scan_xd_api_dockertest #404
* change: Merge PR #405 (typo) and PR #406 (sick_scan_xd_api_test)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@hwulfmeyer @rostest