msi: Added comment to describe that install UI level is only in lowest four bits.

Misha Koshelev mk144210 at bcm.tmc.edu
Tue Feb 6 08:39:53 CST 2007


James Hawkins had a comment on the wine-dev list asking what the 0xf was
in my previous patch that is already in git. I decided to add this
comment per his suggestion. (Wine hacking is fun and addictive :) )

Changelog:

	* msi: Added comment to describe that install UI level is only in
lowest four bits.


-------------- next part --------------
From de6e21f7c9b82f3f095a63d5ebd31ce4b55b7789 Mon Sep 17 00:00:00 2001
From: Misha Koshelev <mk144210 at bcm.tmc.edu>
Date: Tue, 6 Feb 2007 08:35:21 -0600
Subject: msi: Added comment to describe that install UI level is only in lowest four bits.
---
 dlls/msi/action.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)

diff --git a/dlls/msi/action.c b/dlls/msi/action.c
index 6c2fdf4..3822327 100644
--- a/dlls/msi/action.c
+++ b/dlls/msi/action.c
@@ -642,6 +642,9 @@ UINT MSI_InstallPackage( MSIPACKAGE *pac
     msi_apply_transforms( package );
     msi_apply_patches( package );
 
+    /* Install UI level is in the first four bits, the flags are in
+     * bits higher than four. We just want to check the install UI level
+     * and not the flags. */
     if ( (msi_get_property_int(package, szUILevel, 0) & 0xf) >= INSTALLUILEVEL_REDUCED )
     {
         package->script->InWhatSequence |= SEQUENCE_UI;
-- 
1.4.1



More information about the wine-patches mailing list