From 7da487c042cc89750b2797de817f9d07c6d957b6 Mon Sep 17 00:00:00 2001 From: Jay Cho Date: Wed, 15 Apr 2026 16:08:47 +0900 Subject: [PATCH] [Tizen.Applications.Common] Replace manual null checks with ArgumentNullException.ThrowIfNull() --- .../Tizen.Applications/Application.cs | 5 +---- .../Tizen.Applications/ApplicationManager.cs | 5 +---- .../Tizen.Applications/Bundle.cs | 20 ++++--------------- .../Tizen.Applications/CoreApplication.cs | 10 ++-------- .../Tizen.Applications/CoreTask.cs | 10 ++-------- .../Tizen.Applications/ReceivedAppControl.cs | 5 +---- 6 files changed, 11 insertions(+), 44 deletions(-) diff --git a/src/Tizen.Applications.Common/Tizen.Applications/Application.cs b/src/Tizen.Applications.Common/Tizen.Applications/Application.cs index 832d2ee5248..b598e636e29 100644 --- a/src/Tizen.Applications.Common/Tizen.Applications/Application.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/Application.cs @@ -118,10 +118,7 @@ public string Version /// 3 public virtual void Run(string[] args) { - if (args == null) - { - throw new ArgumentNullException(nameof(args)); - } + ArgumentNullException.ThrowIfNull(args); s_CurrentApplication = this; } diff --git a/src/Tizen.Applications.Common/Tizen.Applications/ApplicationManager.cs b/src/Tizen.Applications.Common/Tizen.Applications/ApplicationManager.cs index 6457ac89bf5..fbda2c31cfc 100755 --- a/src/Tizen.Applications.Common/Tizen.Applications/ApplicationManager.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/ApplicationManager.cs @@ -879,10 +879,7 @@ public static IEnumerable GetRecentApplications() [EditorBrowsable(EditorBrowsableState.Never)] public static IEnumerable GetRuaStatTags(string caller) { - if (caller == null) - { - throw new ArgumentNullException(nameof(caller)); - } + ArgumentNullException.ThrowIfNull(caller); List tags = new List(); Interop.ApplicationManager.RuaStatTagIterCallback callback = (string ruaStatTag, IntPtr userData) => diff --git a/src/Tizen.Applications.Common/Tizen.Applications/Bundle.cs b/src/Tizen.Applications.Common/Tizen.Applications/Bundle.cs index 11f3f8298b0..9b6b5220a7c 100644 --- a/src/Tizen.Applications.Common/Tizen.Applications/Bundle.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/Bundle.cs @@ -68,10 +68,7 @@ public Bundle() /// 3 public Bundle(SafeBundleHandle handle) { - if (handle == null) - { - throw new ArgumentNullException(nameof(handle)); - } + ArgumentNullException.ThrowIfNull(handle); if (handle.IsInvalid) { @@ -230,10 +227,7 @@ public bool Contains(string key) /// 3 public void AddItem(string key, byte[] value) { - if (value == null) - { - throw new ArgumentNullException(nameof(value)); - } + ArgumentNullException.ThrowIfNull(value); AddItem(key, value, 0, value.Length); } @@ -260,10 +254,7 @@ public void AddItem(string key, byte[] value, int offset, int count) { if (!_keys.Contains(key)) { - if (value == null) - { - throw new ArgumentNullException(nameof(value)); - } + ArgumentNullException.ThrowIfNull(value); if (offset < 0) { throw new ArgumentOutOfRangeException(nameof(offset), offset, "Cannot be less than 0"); @@ -773,10 +764,7 @@ protected virtual void Dispose(bool disposing) [EditorBrowsable(EditorBrowsableState.Never)] public static Bundle ImportFromArgv(string[] argv) { - if (argv == null) - { - throw new ArgumentNullException(nameof(argv)); - } + ArgumentNullException.ThrowIfNull(argv); return new Bundle(Interop.Bundle.ImportFromArgv(argv.Length, argv)); } diff --git a/src/Tizen.Applications.Common/Tizen.Applications/CoreApplication.cs b/src/Tizen.Applications.Common/Tizen.Applications/CoreApplication.cs index e76a5b8b1cb..6bd4df8d016 100644 --- a/src/Tizen.Applications.Common/Tizen.Applications/CoreApplication.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/CoreApplication.cs @@ -349,10 +349,7 @@ protected virtual void OnTimeZoneChanged(TimeZoneChangedEventArgs e) [EditorBrowsable(EditorBrowsableState.Never)] public static void Post(Action runner) { - if (runner == null) - { - throw new ArgumentNullException(nameof(runner)); - } + ArgumentNullException.ThrowIfNull(runner); GSourceManager.Post(runner, true); } @@ -372,10 +369,7 @@ public static void Post(Action runner) [EditorBrowsable(EditorBrowsableState.Never)] public static async Task Post(Func runner) { - if (runner == null) - { - throw new ArgumentNullException(nameof(runner)); - } + ArgumentNullException.ThrowIfNull(runner); var task = new TaskCompletionSource(); GSourceManager.Post(() => { task.SetResult(runner()); }, true); diff --git a/src/Tizen.Applications.Common/Tizen.Applications/CoreTask.cs b/src/Tizen.Applications.Common/Tizen.Applications/CoreTask.cs index 9ab78271d40..acf18c745e6 100755 --- a/src/Tizen.Applications.Common/Tizen.Applications/CoreTask.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/CoreTask.cs @@ -125,10 +125,7 @@ public virtual void OnUIEvent(UIEventArgs e) /// 10 public static void Post(Action runner) { - if (runner == null) - { - throw new ArgumentNullException(nameof(runner)); - } + ArgumentNullException.ThrowIfNull(runner); GSourceManager.Post(runner); } @@ -147,10 +144,7 @@ public static void Post(Action runner) public static async Task Post(Func runner) { - if (runner == null) - { - throw new ArgumentNullException(nameof(runner)); - } + ArgumentNullException.ThrowIfNull(runner); var task = new TaskCompletionSource(); GSourceManager.Post(() => { task.SetResult(runner()); }); diff --git a/src/Tizen.Applications.Common/Tizen.Applications/ReceivedAppControl.cs b/src/Tizen.Applications.Common/Tizen.Applications/ReceivedAppControl.cs index 47a667d849b..f28c645ca85 100644 --- a/src/Tizen.Applications.Common/Tizen.Applications/ReceivedAppControl.cs +++ b/src/Tizen.Applications.Common/Tizen.Applications/ReceivedAppControl.cs @@ -140,10 +140,7 @@ public bool IsReplyRequest /// 3 public void ReplyToLaunchRequest(AppControl replyRequest, AppControlReplyResult result) { - if (replyRequest == null) - { - throw new ArgumentNullException(nameof(replyRequest)); - } + ArgumentNullException.ThrowIfNull(replyRequest); Interop.AppControl.ErrorCode err = Interop.AppControl.ReplyToLaunchRequest(replyRequest.SafeAppControlHandle, this.SafeAppControlHandle, (int)result); if (err != Interop.AppControl.ErrorCode.None) throw new InvalidOperationException("Failed to reply. Err = " + err);