Apply suggestions from code review

Co-authored-by: 0x5BFA <[email protected]>

Author: d2dyno1

src/Files.App/Helpers/Win32/Win32Helper.Storage.cs

@@ -952,20 +952,20 @@ public static unsafe bool SetCompressionAttributeIoctl(string lpFileName, bool i
 				return false;
 
 			uint bytesReturned = 0u;
-			var compressionFormat = isCompressed
-				? Win32PInvoke.COMPRESSION_FORMAT_DEFAULT
-				: Win32PInvoke.COMPRESSION_FORMAT_NONE;
+			COMPRESSION_FORMAT compressionFormat = isCompressed
+				? COMPRESSION_FORMAT.COMPRESSION_FORMAT_DEFAULT
+				: COMPRESSION_FORMAT.COMPRESSION_FORMAT_NONE;
 
 			var result = PInvoke.DeviceIoControl(
 				new(hFile.DangerousGetHandle()),
-				Win32PInvoke.FSCTL_SET_COMPRESSION,
+				(uint)FSCTL_SET_COMPRESSION,
 				&compressionFormat,
 				sizeof(ushort),
-				(void*)IntPtr.Zero,
+				null,
 				0u,
 				&bytesReturned);
 
-			if (result.Value == 0)
+			if (!result)
 				return false;
 
 			return true;

src/Files.App/ViewModels/Properties/Items/CombinedProperties.cs

@@ -173,23 +173,10 @@ private void ViewModel_PropertyChanged(object sender, System.ComponentModel.Prop
 
 				case "IsContentCompressed":
 					{
-						if (ViewModel.IsContentCompressed is not null)
+						List.ForEach(x =>
 						{
-							if (ViewModel.IsContentCompressed ?? false)
-							{
-								List.ForEach(x =>
-								{
-									Win32Helper.SetCompressionAttributeIoctl(x.ItemPath, true);
-								});
-							}
-							else
-							{
-								List.ForEach(x =>
-								{
-									Win32Helper.SetCompressionAttributeIoctl(x.ItemPath, false);
-								});
-							}
-						}
+							Win32Helper.SetCompressionAttributeIoctl(x.ItemPath, ViewModel.IsContentCompressed ?? false);
+						});
 					}
 					break;
 			}

src/Files.App/ViewModels/Properties/Items/FolderProperties.cs

@@ -215,13 +215,7 @@ private async void ViewModel_PropertyChanged(object sender, System.ComponentMode
 					break;
 
 				case "IsContentCompressed":
-					if (ViewModel.IsContentCompressed is not null)
-					{
-						if ((bool)ViewModel.IsContentCompressed)
-							Win32Helper.SetCompressionAttributeIoctl(Item.ItemPath, true);
-						else
-							Win32Helper.SetCompressionAttributeIoctl(Item.ItemPath, false);
-					}
+					Win32Helper.SetCompressionAttributeIoctl(Item.ItemPath, ViewModel.IsContentCompressed ?? false);
 					break;
 
 				case "ShortcutItemPath":

src/Files.App/ViewModels/Properties/Items/LibraryProperties.cs

@@ -164,13 +164,7 @@ private void ViewModel_PropertyChanged(object sender, System.ComponentModel.Prop
 					break;
 
 				case "IsContentCompressed":
-					if (ViewModel.IsContentCompressed is not null)
-					{
-						if ((bool)ViewModel.IsContentCompressed)
-							Win32Helper.SetCompressionAttributeIoctl(Library.ItemPath, true);
-						else
-							Win32Helper.SetCompressionAttributeIoctl(Library.ItemPath, false);
-					}
+					Win32Helper.SetCompressionAttributeIoctl(Item.ItemPath, ViewModel.IsContentCompressed ?? false);
 					break;
 			}
 		}