Fix two compiler warnings

[jmcarcell]

BEGINRELEASENOTES

• Bump the SIO version to v00-02
• Fix two compiler warnings with clang

ENDRELEASENOTES

These are the warnings:

/LCIO/sio/src/api.cc:497:17: warning: moving a temporary object prevents copy elision [-Wpessimizing-move]
      rec_buf = std::move( device.take_buffer() ) ;
                ^
/LCIO/sio/src/api.cc:497:17: note: remove std::move call here
      rec_buf = std::move( device.take_buffer() ) ;
                ^~~~~~~~~~~                     ~~

/LCIO/src/cpp/include/IMPL/AccessChecked.h:25:13: warning: definition of implicit copy constructor for 'AccessChecked' is deprecated because it has a user-declared destructor [-Wdeprecated-copy-with-dtor]
    virtual ~AccessChecked() = default;
            ^
/LCIO/src/cpp/include/IMPL/TrackStateImpl.h:31:5: note: in implicit copy constructor for 'IMPL::AccessChecked' first required here
    TrackStateImpl(TrackStateImpl const&) = default ;
    ^
/LCIO/src/cpp/src/TESTS/test_trackstate.cc:85:24: note: in defaulted copy constructor for 'IMPL::TrackStateImpl' first required here
        TrackStateImpl c(b);

[tmadlener]

This should be fixed in upstream sio as well I think.

[tmadlener]

The warnings in the SIO sources have been fixed upstream and it should be enough to simply bump the SIO version to v00-02 here

LCIO/CMakeLists.txt

Lines 172 to 173 in 3687a34

	GIT_REPOSITORY https://github.com/iLCSoft/SIO.git
	GIT_TAG v00-01

[jmcarcell]

The warnings in the SIO sources have been fixed upstream and it should be enough to simply bump the SIO version to v00-02 here

LCIO/CMakeLists.txt

Lines 172 to 173 in 3687a34

	GIT_REPOSITORY https://github.com/iLCSoft/SIO.git
	GIT_TAG v00-01

Done

[tmadlener]

Are all warnings gone now? I.e. could we add -Werror to CI? We already have -Werror at least in some CI workflows. Guess we just don't have a workflow with clang.

[jmcarcell]

Yep, I get no warnings with both gcc and clang