Coding guidelines are not followed

[shakthi-prashanth-m]

• Inconsistency in naming Classes, methods, members, etc.
• Use of raw pointers throughout the source code. Need to replace raw pointers by smart pointers and references.
• C++ style is dragged but C-style coding is dominant.
• C++11 is not strictly adapted.
• PImpl style is NOT adapted.
• Selectors are not suffixed with const. Opened pull request to address this: Suffix const for selector methods and parameters Dronecode/camera-manager#152
• Members are not differentiated uniformly using either mMember or _member or member_, etc.
• Unnecessary use of operator new.
• Usage of reference and const reference wherever applicable instead of returning objects by value.
• New Types must be part of a class if their usage is part of the class. They must not be defined in File scope.
• Filenames are inconsistent. We need to follow either CamelCase or snake_case. The repository has mixture of both: https://github.com/intel/camera-streaming-daemon/tree/master/src