resolved some const and reference issues.
avoids invoking copy constructor when passing objects as arguments.
save some time and easier to debug too.
eliminates a function that was called twice and basically did nothing
this was supposed to be part of the previous commit.
untangling this in order to eliminate SetMarker1 and SetMarker2. Should only have SetMarker and send in the right arguments. Will cut in half the amount of code that needs to be maintained/debugged.
This will reach into several classes as SetMarker? is a virtual function.
Eliminated functions that were not used anywhere.