Lightning Viewer Example: Avoid Using Heap Memory When Is Not Necessary

When a class/struct supports move semantics or they are value type, then no need to keep a pointer of them as a member in parent class.

Also it's better to use std::unique_ptr instead of QScopedPointer.

In the Lightning Viewer Example, some of these rules are not followed.