Revert "[NUI] Fix build warning[CA1000]"
authorhuiyu <huiyu.eun@samsung.com>
Tue, 15 Dec 2020 02:10:34 +0000 (11:10 +0900)
committerdongsug-song <35130733+dongsug-song@users.noreply.github.com>
Tue, 15 Dec 2020 06:33:59 +0000 (15:33 +0900)
This reverts commit 4cef848feaaefe4abac0b5f7890b8402e37fbcf6.

12 files changed:
src/Tizen.NUI.Components/Controls/Extension/SlidingSwitchExtension.cs
src/Tizen.NUI/src/internal/DisposeQueue.cs
src/Tizen.NUI/src/public/Accessibility/Accessibility.cs
src/Tizen.NUI/src/public/BaseComponents/Style/ViewStyle.cs
src/Tizen.NUI/src/public/KeyValue.cs
src/Tizen.NUI/src/public/Layouting/LayoutItem.cs
src/Tizen.NUI/src/public/Layouting/LayoutTransition.cs
src/Tizen.NUI/src/public/Layouting/LinearLayout.cs
src/Tizen.NUI/src/public/TransitionAnimations/TransitionAnimations.cs
src/Tizen.NUI/src/public/ViewProperty/BackgroundExtraData.cs
src/Tizen.NUI/src/public/ViewProperty/TextShadow.cs
src/Tizen.NUI/src/public/VisualMaps.cs

index b37cb5e..1086ceb 100755 (executable)
@@ -27,7 +27,7 @@ namespace Tizen.NUI.Components.Extension
     /// </remark>
     /// </summary>
     [EditorBrowsable(EditorBrowsableState.Never)]
-    internal class SlidingSwitchExtension : SwitchExtension, IDisposable
+    internal class SlidingSwitchExtension : SwitchExtension
     {
         private Animation slidingAnimation;
 
@@ -68,7 +68,6 @@ namespace Tizen.NUI.Components.Extension
             }
         }
 
-        [EditorBrowsable(EditorBrowsableState.Never)]
         public override void OnDispose(Button button)
         {
             if (slidingAnimation != null)
@@ -81,22 +80,5 @@ namespace Tizen.NUI.Components.Extension
                 slidingAnimation = null;
             }
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                slidingAnimation?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 }
index 5fc50b5..917ee28 100755 (executable)
@@ -12,7 +12,7 @@ using System.Collections.Generic;
 
 namespace Tizen.NUI
 {
-    internal class DisposeQueue : IDisposable
+    internal class DisposeQueue
     {
         private static readonly DisposeQueue _disposableQueue = new DisposeQueue();
         private List<IDisposable> _disposables = new List<IDisposable>();
@@ -68,22 +68,5 @@ namespace Tizen.NUI
                 _disposables.Clear();
             }
         }
-
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                _disposableQueue?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
-
     }
 }
index 4164a79..2421356 100755 (executable)
@@ -32,7 +32,7 @@ namespace Tizen.NUI.Accessibility
     // This will be public opened after ACR done. (Before ACR, need to be hidden as Inhouse API)
     [SuppressMessage("Microsoft.Design", "CA1724: Type names should not match namespaces")]
     [EditorBrowsable(EditorBrowsableState.Never)]
-    public class Accessibility : IDisposable
+    public class Accessibility
     {
         #region Constructor, Distructor, Dispose
         private Accessibility()
@@ -223,21 +223,5 @@ namespace Tizen.NUI.Accessibility
 
         private static string tag = "NUITEST";
         #endregion Private
-
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                dummy?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 }
index 8543257..782e050 100755 (executable)
@@ -26,7 +26,7 @@ namespace Tizen.NUI.BaseComponents
     /// </summary>
     /// This will be public opened in tizen_6.0 after ACR done. Before ACR, need to be hidden as inhouse API.
     [EditorBrowsable(EditorBrowsableState.Never)]
-    public partial class ViewStyle : BindableObject, IDisposable
+    public partial class ViewStyle : BindableObject
     {
         private string styleName;
         private View.States? state;
@@ -768,21 +768,5 @@ namespace Tizen.NUI.BaseComponents
         {
             Margin = new Extents(start, end, top, bottom);
         }
-
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                padding?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 }
index 033e210..87c5445 100755 (executable)
@@ -16,7 +16,6 @@
  */
 
 using System;
-using System.ComponentModel;
 using Tizen.NUI.Binding;
 
 namespace Tizen.NUI
@@ -24,7 +23,7 @@ namespace Tizen.NUI
     /// <summary>
     /// KeyValue class.
     /// </summary>
-    public class KeyValue : IDisposable
+    public class KeyValue
     {
         /// <summary>
         /// Int key.
@@ -353,23 +352,5 @@ namespace Tizen.NUI
                 KeyString = Key;
             }
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                TrueValue?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 }
index e330949..3133f44 100755 (executable)
@@ -37,7 +37,7 @@ namespace Tizen.NUI
     /// [Draft] Base class for layouts. It is used to layout a View
     /// It can be laid out by a LayoutGroup.
     /// </summary>
-    public class LayoutItem : IDisposable
+    public class LayoutItem
     {
         static bool LayoutDebugFrameData = false; // Debug flag
         private MeasureSpecification OldWidthMeasureSpec; // Store measure specification to compare against later
@@ -624,24 +624,5 @@ namespace Tizen.NUI
 
             return changed;
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                _padding?.Dispose();
-                _margin?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 }
index 0b851cb..3611f15 100755 (executable)
@@ -17,7 +17,6 @@
 
 using System.Collections.Generic;
 using System;
-using System.ComponentModel;
 
 namespace Tizen.NUI
 {
@@ -101,7 +100,7 @@ namespace Tizen.NUI
     /// Parts of the transition that can be configured to provide a custom effect.
     /// </summary>
     /// <since_tizen> 6 </since_tizen>
-    public class TransitionComponents : IDisposable
+    public class TransitionComponents
     {
         /// <summary>
         /// TransitionComponents default constructor.
@@ -144,24 +143,6 @@ namespace Tizen.NUI
         /// </summary>
         /// <since_tizen> 6 </since_tizen>
         public AlphaFunction AlphaFunction;
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                AlphaFunction?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 
     /// <summary>
index fcbd291..584f235 100755 (executable)
@@ -18,7 +18,6 @@ using System;
 using Tizen.NUI.BaseComponents;
 using System.Collections.Generic;
 using System.Linq;
-using System.ComponentModel;
 
 namespace Tizen.NUI
 {
@@ -830,18 +829,5 @@ namespace Tizen.NUI
                 }
             }
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected override void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                _cellPadding?.Dispose();
-            }
-            // Free native resources.
-            base.Dispose(disposing);
-        }
-
     } //LinearLayout
 } // namespace
index 16cf635..f2799d5 100755 (executable)
@@ -42,7 +42,7 @@ namespace Tizen.NUI
     /// Transition animation effect
     /// </summary>
     [EditorBrowsable(EditorBrowsableState.Never)]
-    public class TransitionAnimation : IDisposable
+    public class TransitionAnimation
     {
         private ImageViewStyle defaultImageStyle;
         private List<TransitionAnimationData> animationDataList;
@@ -136,24 +136,6 @@ namespace Tizen.NUI
                 defaultImageStyle = value;
             }
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                defaultImageStyle?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
     }
 
     /// <summary>
index 387faaa..8d95088 100755 (executable)
  *
  */
 
-using System;
-
 namespace Tizen.NUI
 {
     /// <summary>
     /// The class storing Background extra properties such as CornerRadius, ImageBorder.
     /// </summary>
-    internal class BackgroundExtraData : IDisposable
+    internal class BackgroundExtraData
     {
         internal BackgroundExtraData()
         {
@@ -55,23 +53,6 @@ namespace Tizen.NUI
         {
             return CornerRadius == 0 && Rectangle.IsNullOrZero(BackgroundImageBorder);
         }
-
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                backgroundImageBorder?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
-
     }
 }
 
index 6d07c9e..af116f8 100755 (executable)
@@ -24,7 +24,7 @@ namespace Tizen.NUI
     /// The Text Shadow for TextLabel.
     /// </summary>
     [EditorBrowsable(EditorBrowsableState.Never)]
-    public class TextShadow : ICloneable, IDisposable
+    public class TextShadow : ICloneable
     {
         private readonly PropertyMap propertyMap = null;
 
@@ -113,22 +113,5 @@ namespace Tizen.NUI
 
             return new PropertyValue(instance.propertyMap);
         }
-
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                propertyMap?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
-
     }
 }
index 425398a..6ee9e4a 100755 (executable)
@@ -14,7 +14,6 @@
  * limitations under the License.
  *
  */
-using System;
 using System.ComponentModel;
 using Tizen.NUI.BaseComponents;
 
@@ -24,7 +23,7 @@ namespace Tizen.NUI
     /// A class encapsulating the transform map of the visual.
     /// </summary>
     /// <since_tizen> 3 </since_tizen>
-    public class VisualMap : IDisposable
+    public class VisualMap
     {
         /// <summary>
         /// outputVisualMap.
@@ -677,31 +676,5 @@ namespace Tizen.NUI
             if (_visualOrigin != null) { _visualTransformMap.Add((int)VisualTransformPropertyType.Origin, new PropertyValue((int)_visualOrigin)); }
             if (_visualAnchorPoint != null) { _visualTransformMap.Add((int)VisualTransformPropertyType.AnchorPoint, new PropertyValue((int)_visualAnchorPoint)); }
         }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        protected virtual void Dispose(bool disposing)
-        {
-            if (disposing)
-            {
-                // Dispose managed resources.
-                _outputVisualMap?.Dispose();
-                _shader?.Dispose();
-                _mixColor?.Dispose();
-                _visualSize?.Dispose();
-                _visualOffset?.Dispose();
-                _visualOffsetPolicy?.Dispose();
-                _visualSizePolicy?.Dispose();
-                _visualTransformMap?.Dispose();
-            }
-            // Free native resources.
-        }
-
-        [EditorBrowsable(EditorBrowsableState.Never)]
-        public void Dispose()
-        {
-            Dispose(true);
-            GC.SuppressFinalize(this);
-        }
-
     }
 }