Remove BaseHandle / BaseObject TypeTraits is trivialgit 47/309647/1
authorEunki, Hong <eunkiki.hong@samsung.com>
Mon, 15 Apr 2024 04:23:21 +0000 (13:23 +0900)
committerEunki, Hong <eunkiki.hong@samsung.com>
Mon, 15 Apr 2024 04:23:57 +0000 (13:23 +0900)
Since Dali::Vector doesn't call destructor, BaseHandle / BaseObject
shoudl 'not' be used as Dali::Vector.

To avoid some memory leak mistake, let we remove some mis implementations

Change-Id: I001993d10812bddc0beb888d41e6bc316301484b
Signed-off-by: Eunki, Hong <eunkiki.hong@samsung.com>
dali-toolkit/devel-api/text/range.h
dali-toolkit/devel-api/text/spans/base-span.h
dali-toolkit/internal/text/spannable/span-ranges-container-impl.h
dali-toolkit/internal/text/spannable/spannable-impl.h
dali-toolkit/internal/text/spannable/spannable-string-impl.h
dali-toolkit/internal/text/spannable/spanned-impl.h
dali-toolkit/internal/text/spannable/spans/base-span-impl.h
dali-toolkit/internal/text/string-text/character-sequence-impl.h
dali-toolkit/internal/text/string-text/range-impl.h

index 14538fd..2a43766 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_TEXT_RANGE_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -144,16 +144,6 @@ public: //Methods
 
 } // namespace Toolkit
 
-// Allow Range to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Range> : public BasicTypes<Dali::Toolkit::Text::Range>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_TEXT_RANGE_H
index e3f7f29..5769965 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_TEXT_BASE_SPAN_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -122,16 +122,6 @@ public: // Not intended for application developers
 
 } // namespace Toolkit
 
-// Allow BaseSpan to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::BaseSpan> : public BasicTypes<Dali::Toolkit::Text::BaseSpan>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_TEXT_BASE_SPAN_H
index f89846e..ffbef1e 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_SPANS_CONTAINER_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -121,16 +121,6 @@ private:
 
 } // namespace Toolkit
 
-// Allow SpanRangesContainer to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::SpanRangesContainer> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::SpanRangesContainer>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_SPANS_CONTAINER_IMPL_H
index 0b56043..40b388a 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_SPANNABLE_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -102,16 +102,6 @@ inline const Internal::Spannable& GetImplementation(const Dali::Toolkit::Text::S
 
 } // namespace Toolkit
 
-// Allow Spannable to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::Spannable> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::Spannable>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_SPANNABLE_IMPL_H
index 5ecb823..5dccab4 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_SPANNABLE_STRING_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -144,16 +144,6 @@ inline const Internal::SpannableString& GetImplementation(const Dali::Toolkit::T
 
 } // namespace Toolkit
 
-// Allow SpannableString to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::SpannableString> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::SpannableString>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_SPANNABLE_STRING_IMPL_H
index 4e5a8c7..e6c18cd 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_SPANNED_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -103,16 +103,6 @@ inline const Internal::Spanned& GetImplementation(const Dali::Toolkit::Text::Spa
 
 } // namespace Toolkit
 
-// Allow Spanned to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::Spanned> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::Spanned>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_SPANNED_IMPL_H
index 5f01524..cd2f850 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_BASE_SPAN_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -116,16 +116,6 @@ inline const Internal::BaseSpan& GetImplementation(const Dali::Toolkit::Text::Ba
 
 } // namespace Toolkit
 
-// Allow BaseSpan to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::BaseSpan> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::BaseSpan>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_BASE_SPAN_IMPL_H
index d04883b..27312b7 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_CHARACTER_SEQUENCE_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -109,16 +109,6 @@ inline const Internal::CharacterSequence& GetImplementation(const Dali::Toolkit:
 
 } // namespace Toolkit
 
-// Allow AbstractStyleSpan to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::CharacterSequence> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::CharacterSequence>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_CHARACTER_SEQUENCE_IMPL_H
index 8b0f0d2..b4e6c1a 100644 (file)
@@ -2,7 +2,7 @@
 #define DALI_TOOLKIT_INTERNAL_TEXT_RANGE_IMPL_H
 
 /*
- * Copyright (c) 2022 Samsung Electronics Co., Ltd.
+ * Copyright (c) 2024 Samsung Electronics Co., Ltd.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -126,16 +126,6 @@ inline const Internal::Range& GetImplementation(const Dali::Toolkit::Text::Range
 
 } // namespace Toolkit
 
-// Allow Range to be treated as a POD type
-template<>
-struct TypeTraits<Dali::Toolkit::Text::Internal::Range> : public Dali::BasicTypes<Dali::Toolkit::Text::Internal::Range>
-{
-  enum
-  {
-    IS_TRIVIAL_TYPE = true
-  };
-};
-
 } // namespace Dali
 
 #endif // DALI_TOOLKIT_INTERNAL_TEXT_RANGE_IMPL_H