* Add getBreakpointsAddresses to list breakpoints offsets
* Add breakpoint highligthing and improve of disassembly UIs
* Improve dialogs and positions
* Add graph,offset option to preference
* Tiny text fix
* Updated radare2 sumodule
* GraphView: Add refactored graphview with animations etc.
* GraphView: Use new graphview and render background color.
* DisassemblerGraphView: t/f navigation, zoom with +/-, bugfixes.
moving a return value generally is unnecessary, as it prevents return value optimization.
The standard defines, that (named) return value optimization, i.e. copy elision
is only possible when returning a value from a non-volatile automatic object
or when it's a temporary that has not been bound to a reference.
(N)RVO is not possible when wrapping it with std::move, because the compiler
is not allowed to do copy elision from arbitrary function calls.
I think this was the intended purpose of this code. Also, it is cleaner.