Merge pull request #2407 from stephentoub/remove_modules_throw_linux

stephentoub · stephentoub · commit fee1721e7594 · 2015-07-17T22:53:29.000-04:00
Remove unnecessary throw in ProcessManager.GetModulesInfos on Linux
diff --git a/src/System.Diagnostics.Process/src/System/Diagnostics/ProcessManager.Linux.cs b/src/System.Diagnostics.Process/src/System/Diagnostics/ProcessManager.Linux.cs
@@ -67,11 +67,6 @@ internal static ModuleInfo[] GetModuleInfos(int processId)
             }
 
             // Return the set of modules found
-            if (modules.Count == 0)
-            {
-                // Match Windows behavior when failing to enumerate modules
-                throw new Win32Exception(SR.EnumProcessModuleFailed);
-            }
             return modules.ToArray();
         }
 

Original file line number	Diff line number	Diff line change
`@@ -67,11 +67,6 @@ internal static ModuleInfo[] GetModuleInfos(int processId)`
`67`	`67`	`}`
`68`	`68`
`69`	`69`	`// Return the set of modules found`
`70`		`- if (modules.Count == 0)`
`71`		`- {`
`72`		`- // Match Windows behavior when failing to enumerate modules`
`73`		`- throw new Win32Exception(SR.EnumProcessModuleFailed);`
`74`		`- }`
`75`	`70`	`return modules.ToArray();`
`76`	`71`	`}`
`77`	`72`