* Replace scroll algorithm with just using offset
* WIP
* Text is now shown properly on each node
* WIP
* Scaling well WIP
* Fix positioning for Graph
* Overview properly displays Rect WIP
* Rect in overview moves along with the mouse WIP
* Overview rect properly moves WIP
* mouse properly moves WIP
* in middle of fixing resizing WIP
* scaling issue WIP
* Properly zooms
* almost done
* Edge width fixed for the overview
* Fixed a small bug
* Clean up
* Adding a header to the DisassemblyWidget to show the function prototype
* Moving the afcf headerLabel functionality to the GraphWidget
* In middle of implementation
* In middle
* syntax highlighter added
* Implementation done.
* Fixed the changes that were not necessary.
* 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.