fix(core): clean code, remove useless includes

src/object/clonable-object.h

@@ -19,10 +19,6 @@
 #ifndef _CLONABLE_OBJECT_H_
 #define _CLONABLE_OBJECT_H_
 
-#include <string>
-
-#include "linphone/utils/general.h"
-
 #include "property-container.h"
 
 // =============================================================================

src/object/object-p.h

@@ -21,8 +21,6 @@
 
 #include <unordered_map>
 
-#include "linphone/utils/general.h"
-
 #include "variant/variant.h"
 
 // =============================================================================

src/object/object.h

@@ -21,16 +21,15 @@
 
 #include <memory>
 
-#include "linphone/utils/general.h"
-
 #include "property-container.h"
 
 // =============================================================================
 
 LINPHONE_BEGIN_NAMESPACE
 
-class LINPHONE_PUBLIC Object : public std::enable_shared_from_this<Object>, public PropertyContainer {
-
+class LINPHONE_PUBLIC Object :
+	public std::enable_shared_from_this<Object>,
+	public PropertyContainer {
 public:
 	virtual ~Object ();
 

src/object/property-container.h

@@ -19,10 +19,6 @@
 #ifndef _PROPERTY_CONTAINER_H_
 #define _PROPERTY_CONTAINER_H_
 
-#include <string>
-
-#include "linphone/utils/general.h"
-
 #include "variant/variant.h"
 
 // =============================================================================

src/object/singleton.h

@@ -26,7 +26,7 @@
 LINPHONE_BEGIN_NAMESPACE
 
 template<class T>
-class Singleton : public Object {
+class LINPHONE_PUBLIC Singleton : public Object {
 public:
 	virtual ~Singleton () = default;
 

src/utils/utils.cpp

@@ -93,10 +93,10 @@ string Utils::toString (const void *val) {
 // -----------------------------------------------------------------------------
 
 #define STRING_TO_NUMBER_IMPL(TYPE, SUFFIX) \
-	TYPE Utils::sto ## SUFFIX (const string &str, size_t *idx, int base) { \
+	TYPE Utils::sto ## SUFFIX(const string &str, size_t * idx, int base) { \
 		return sto ## SUFFIX(str.c_str(), idx, base); \
 	} \
-	TYPE Utils::sto ## SUFFIX (const char *str, size_t *idx, int base) { \
+	TYPE Utils::sto ## SUFFIX(const char *str, size_t * idx, int base) { \
 		char *p; \
 		TYPE v = strto ## SUFFIX(str, &p, base); \
 		if (idx) \
@@ -105,10 +105,10 @@ string Utils::toString (const void *val) {
 	} \
 
 #define STRING_TO_NUMBER_IMPL_BASE_LESS(TYPE, SUFFIX) \
-	TYPE Utils::sto ## SUFFIX (const string &str, size_t *idx) { \
+	TYPE Utils::sto ## SUFFIX(const string &str, size_t * idx) { \
 		return sto ## SUFFIX(str.c_str(), idx); \
 	} \
-	TYPE Utils::sto ## SUFFIX (const char *str, size_t *idx) { \
+	TYPE Utils::sto ## SUFFIX(const char *str, size_t * idx) { \
 		char *p; \
 		TYPE v = strto ## SUFFIX(str, &p); \
 		if (idx) \

src/variant/variant.h

@@ -81,7 +81,7 @@ public:
 	Variant (const std::string &value);
 
 	// void* constructor. Must be explicitly called.
-	template<typename T, typename = typename std::enable_if<std::is_same<T, void *>::value> >
+	template<typename T, typename = typename std::enable_if<std::is_same<T, void *>::value>>
 	Variant (T value) : Variant (Variant::createGeneric(value)) {}
 
 	~Variant ();
@@ -98,7 +98,7 @@ public:
 	template<typename T>
 	void setValue (const T &value) {
 		// Yeah, I'm crazy but it's useful to avoid code duplication.
-		new (this) Variant(value);
+		new(this) Variant(value);
 	}
 
 	template<typename T>