OpenCOLLADA icon indicating copy to clipboard operation
OpenCOLLADA copied to clipboard

possibly dangling reference to a temporary

Open oneforall2 opened this issue 2 years ago • 4 comments

/tmp/OpenCOLLADA-1.6.68/DAEValidator/library/src/Dae.cpp: In member function ‘virtual void opencollada::Dae::readFile(const std::string&)’: /tmp/OpenCOLLADA-1.6.68/DAEValidator/library/src/Dae.cpp:79:30: error: possibly dangling reference to a temporary [-Werror=dangling-reference] 79 | const auto & instances = root().selectNodes( | ^~~~~~~~~ /tmp/OpenCOLLADA-1.6.68/DAEValidator/library/src/Dae.cpp:79:60: note: the temporary was destroyed at the end of the full expression ‘((opencollada::Dae*)this)->opencollada::Dae::.opencollada::XmlDoc::root().opencollada::XmlNode::selectNodes(std::operator+(__cxx11::basic_string<_CharT, _Traits, _Alloc>&&, const __cxx11::basic_string<_CharT, _Traits, _Alloc>&) with _CharT = char; _Traits = char_traits; _Alloc = allocator)’ 79 | const auto & instances = root().selectNodes( | ~~~~~~~~~~~~~~~~~~^ 80 | xpath_all + Strings::instance_animation + gcc 13.1.0

oneforall2 avatar May 29 '23 06:05 oneforall2

I've been seeing this on Fedora as well. Unfortunately there has been no activity upstream for some time. I'm concerned I'll have to remove OpenCOLLADA from Fedora.

hobbes1069 avatar May 29 '23 12:05 hobbes1069

I got some tips from the Fedora devel list and at least got it compiling. It just ignores the errors but that code hasn't change in ages and I don't have the knowledge to fix it properly.

https://src.fedoraproject.org/rpms/openCOLLADA/blob/rawhide/f/openCOLLADA-pragma.patch

hobbes1069 avatar May 29 '23 13:05 hobbes1069

I used -Wno-error=dangling-reference for now. But would be nice to see it fixed :)

oneforall2 avatar May 29 '23 15:05 oneforall2

Already had the missing includes

oneforall2 avatar May 29 '23 15:05 oneforall2