fix alerts reported by lgtm

[jgrewe]

not done

edit: should be enough for now :)

[lgtm-com[bot]]

This pull request fixes 7 alerts when merging e426fb5da9cde746f3ab8474d60a2d63129ba2a4 into 2085ebb07d0e212dee8fb72e83d1ab960502c22b - view on LGTM.com

fixed alerts:

• 3 for Declaration hides parameter
• 1 for Missing header guard
• 1 for Throwing pointers
• 1 for Constant return type on member
• 1 for Use of potentially dangerous function

[lgtm-com[bot]]

This pull request fixes 8 alerts when merging 120782f2930b094fe26fdc902180bc4b05407503 into 2085ebb07d0e212dee8fb72e83d1ab960502c22b - view on LGTM.com

fixed alerts:

• 3 for Declaration hides parameter
• 1 for Inconsistent definition of copy constructor and assignment ('Rule of Two')
• 1 for Missing header guard
• 1 for Throwing pointers
• 1 for Constant return type on member
• 1 for Use of potentially dangerous function

[lgtm-com[bot]]

This pull request fixes 18 alerts when merging 16b57f33208173ebda64050c3769c62af2782eb5 into 2085ebb07d0e212dee8fb72e83d1ab960502c22b - view on LGTM.com

fixed alerts:

• 11 for Inconsistent definition of copy constructor and assignment ('Rule of Two')
• 3 for Declaration hides parameter
• 1 for Missing header guard
• 1 for Throwing pointers
• 1 for Constant return type on member
• 1 for Use of potentially dangerous function

[lgtm-com[bot]]

This pull request fixes 26 alerts when merging 741d18bf1875666f7d74359238c1a9fb683daffb into 2085ebb07d0e212dee8fb72e83d1ab960502c22b - view on LGTM.com

fixed alerts:

• 17 for Inconsistent definition of copy constructor and assignment ('Rule of Two')
• 3 for Declaration hides parameter
• 2 for FIXME comment
• 1 for Missing header guard
• 1 for Throwing pointers
• 1 for Constant return type on member
• 1 for Use of potentially dangerous function