Fix dispose and wrong statement in comment 39/147439/2
authorJooseok Song <seogii.song@samsung.com>
Mon, 4 Sep 2017 09:49:02 +0000 (18:49 +0900)
committerJooseok Song <seogii.song@samsung.com>
Mon, 4 Sep 2017 10:36:01 +0000 (19:36 +0900)
Change-Id: I264b7d15a9cb6147411f3d30fb07e233bdd58ba0

src/Tizen.Account.AccountManager/Tizen.Account.AccountManager/Account.cs
src/Tizen.Account.AccountManager/Tizen.Account.AccountManager/AccountEnums.cs
src/Tizen.Account.AccountManager/Tizen.Account.AccountManager/AccountErrorFactory.cs
src/Tizen.Account.AccountManager/Tizen.Account.AccountManager/AccountProvider.cs
src/Tizen.Account.AccountManager/Tizen.Account.AccountManager/AccountService.cs

index 498bb84..b09639f 100644 (file)
@@ -27,6 +27,11 @@ namespace Tizen.Account.AccountManager
     {
         private readonly SafeAccountHandle _handle;
 
+       /// <summary>
+       /// Account constructor.
+       /// </summary>
+       /// <since_tizen> 4 </since_tizen>
+       /// <param name="handle"> The account handle.</param>
         public Account(SafeAccountHandle handle)
         {
             _handle = handle;
@@ -607,16 +612,11 @@ namespace Tizen.Account.AccountManager
             GC.SuppressFinalize(this);
         }
 
-        private void Dispose(bool disposing)
+        protected virtual void Dispose(bool disposing)
         {
             if (!disposing)
             {
-                //if (_handle != IntPtr.Zero)
-                //{
-                //    _handle = IntPtr.Zero;
-                //}
-                //_handle.Dispose();
-                GC.SuppressFinalize(this);
+                _handle.Dispose();
             }
         }
     }
index 68a7a65..53942e0 100644 (file)
@@ -96,7 +96,7 @@ namespace Tizen.Account.AccountManager
         Idle,
 
         /// <summary>
-        /// Acount sync supported and sync status is running.
+        /// Account sync supported and sync status is running.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         Running
index 51c31dd..f23c792 100644 (file)
@@ -20,7 +20,7 @@ using Tizen;
 namespace Tizen.Account.AccountManager
 {
     /// <summary>
-    /// Enumeration for the types of error occured, if any.
+    /// Enumeration for the types of error occurred, if any.
     /// </summary>
     /// <since_tizen> 3 </since_tizen>
     public enum AccountError
@@ -45,7 +45,7 @@ namespace Tizen.Account.AccountManager
         /// Same user name exists in your application.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
-        Duplcated = -0x01000000 | 0x01,
+        Duplicated = -0x01000000 | 0x01,
         /// <summary>
         /// Empty data.
         /// </summary>
index ce34491..be9e077 100644 (file)
@@ -69,7 +69,7 @@ namespace Tizen.Account.AccountManager
         }
 
         /// <summary>
-        /// Serviceprovider ID of the account provider.
+        /// ServiceProvider ID of the account provider.
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         public string ServiceProviderId
@@ -365,7 +365,7 @@ namespace Tizen.Account.AccountManager
             GC.SuppressFinalize(this);
         }
 
-        private void Dispose(bool disposing)
+        protected virtual void Dispose(bool disposing)
         {
             if (!disposing)
             {
index d335a8e..8ebab51 100644 (file)
@@ -200,7 +200,7 @@ namespace Tizen.Account.AccountManager
         /// <returns>The AccountProvider instance associated with the given application ID.</returns>
         /// <privilege>http://tizen.org/privilege/account.read</privilege>
         /// <feature>http://tizen.org/feature/account</feature>
-        /// <exception cref="InvalidOperationException">In case of any DB error or record not found for given appid.</exception>
+        /// <exception cref="InvalidOperationException">In case of any DB error or record not found for given appId.</exception>
         /// <exception cref="ArgumentException"> In case of an invalid parameter.</exception>
         /// <exception cref="UnauthorizedAccessException"> In case of privilege not defined.</exception>
         /// <exception cref="NotSupportedException">The required feature is not supported.</exception>
@@ -262,6 +262,7 @@ namespace Tizen.Account.AccountManager
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         /// <param name="account">New Account instance to be added.</param>
+        /// <returns>The account ID of the account instance.</returns>
         /// <privilege>http://tizen.org/privilege/account.read</privilege>
         /// <privilege>http://tizen.org/privilege/account.write </privilege>
         /// <feature>http://tizen.org/feature/account</feature>
@@ -590,9 +591,9 @@ namespace Tizen.Account.AccountManager
         /// </summary>
         /// <since_tizen> 3 </since_tizen>
         /// <remarks>
-        /// ContentUpdate event is triggered if the MediaInformaion updated/deleted or new information is inserted.
+        /// ContentUpdate event is triggered if the MediaInformation updated/deleted or new information is inserted.
         /// </remarks>
-        /// <param name="sender"></param>
+        /// <param name="sender">An object that contains information about sender</param>
         /// <param name="e">A ContentUpdatedEventArgs object that contains information about the update operation.</param>
         /// <privilege>http://tizen.org/privilege/account.read</privilege>
         /// <feature>http://tizen.org/feature/account</feature>