No reason for these to use inline null checks

mnadareski · mnadareski · commit 9639f40e55a5 · 2025-09-20T16:15:54.000-04:00
diff --git a/BinaryObjectScanner/Scanner.cs b/BinaryObjectScanner/Scanner.cs
@@ -347,10 +347,12 @@ private ProtectionDictionary GetInternalProtections(string fileName, Stream stre
                         }
 
                         // Prepare the returned protections
-                        subProtections?.StripFromKeys(tempPath);
-                        subProtections?.PrependToKeys(fileName);
                         if (subProtections != null)
+                        {
+                            subProtections.StripFromKeys(tempPath);
+                            subProtections.PrependToKeys(fileName);
                             protections.Append(subProtections);
+                        }
                     }
                     catch (Exception ex)
                     {

Original file line number	Diff line number	Diff line change
`@@ -347,10 +347,12 @@ private ProtectionDictionary GetInternalProtections(string fileName, Stream stre`
`347`	`347`	`}`
`348`	`348`
`349`	`349`	`// Prepare the returned protections`
`350`		`- subProtections?.StripFromKeys(tempPath);`
`351`		`- subProtections?.PrependToKeys(fileName);`
`352`	`350`	`if (subProtections != null)`
	`351`	`+ {`
	`352`	`+ subProtections.StripFromKeys(tempPath);`
	`353`	`+ subProtections.PrependToKeys(fileName);`
`353`	`354`	`protections.Append(subProtections);`
	`355`	`+ }`
`354`	`356`	`}`
`355`	`357`	`catch (Exception ex)`
`356`	`358`	`{`