Enable IDE0200 (remove unnecessary lambda expression) (dotnet#7916)

v-elnovikova · Oct 18, 2022 · 4d6bacc · 4d6bacc
1 parent 3ac5229
commit 4d6bacc
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 5 deletions.
diff --git a/eng/CodeAnalysis.src.globalconfig b/eng/CodeAnalysis.src.globalconfig
@@ -1708,7 +1708,7 @@ dotnet_diagnostic.IDE0170.severity = warning
 dotnet_diagnostic.IDE0180.severity = suggestion
 
 # IDE0200: Remove unnecessary lambda expression
-dotnet_diagnostic.IDE0200.severity = none # TODO: warning
+dotnet_diagnostic.IDE0200.severity = warning
 
 # IDE0210: Use top-level statements
 dotnet_diagnostic.IDE0210.severity = silent

diff --git a/src/System.Windows.Forms.Design/src/System/ComponentModel/Design/CollectionEditor.cs b/src/System.Windows.Forms.Design/src/System/ComponentModel/Design/CollectionEditor.cs
@@ -218,7 +218,7 @@ public override object EditValue(ITypeDescriptorContext context, IServiceProvide
             _currentContext = context;
 
             // Child modal dialog - launching in SystemAware mode.
-            CollectionForm localCollectionForm = DpiHelper.CreateInstanceInSystemAwareContext(() => CreateCollectionForm());
+            CollectionForm localCollectionForm = DpiHelper.CreateInstanceInSystemAwareContext(CreateCollectionForm);
             ITypeDescriptorContext lastContext = _currentContext;
             localCollectionForm.EditValue = value;
             _ignoreChangingEvents = false;

diff --git a/src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/CommandSet.cs b/src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/CommandSet.cs
@@ -2108,7 +2108,7 @@ protected void OnMenuPaste(object sender, EventArgs e)
                     return;   // nothing we can do here!
 
                 IDataObject dataObj = null;
-                bool clipboardOperationSuccessful = ExecuteSafely(() => Clipboard.GetDataObject(), false, out dataObj);
+                bool clipboardOperationSuccessful = ExecuteSafely(Clipboard.GetDataObject, false, out dataObj);
 
                 if (clipboardOperationSuccessful)
                 {
@@ -3505,7 +3505,7 @@ protected void OnStatusPaste(object sender, EventArgs e)
             // Not being inherited.  Now look at the contents of the data
             //
             IDataObject dataObj = null;
-            bool clipboardOperationSuccessful = ExecuteSafely(() => Clipboard.GetDataObject(), false, out dataObj);
+            bool clipboardOperationSuccessful = ExecuteSafely(Clipboard.GetDataObject, false, out dataObj);
 
             bool enable = false;
 

diff --git a/src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/EventHandlerService.cs b/src/System.Windows.Forms.Design/src/System/Windows/Forms/Design/EventHandlerService.cs
@@ -57,7 +57,7 @@ public object GetHandler(Type handlerType)
 
             Debug.Assert(_handlers.Count > 0, "Should have handlers to look through.");
 
-            object handler = _handlers.FirstOrDefault(h => handlerType.IsInstanceOfType(h));
+            object handler = _handlers.FirstOrDefault(handlerType.IsInstanceOfType);
 
             if (handler != null)
             {