Better question for wix-devs. Short answer: second "execute" was probably supposed to be "rollback" then considered not doing that and ended up leaving a redundant clause in the if. Would have to walk through the code fully to know for sure.
On Mon, Jan 14, 2013 at 6:35 AM, Bruce Cran <br...@cran.org.uk> wrote: > In plan.cpp:1166 there's a line of code with identical left and right > expressions. Is this a bug? > > // If we are going to take any action on this package, add progress for it. > if (BOOTSTRAPPER_ACTION_STATE_NONE != pRelatedBundle->package.execute || > BOOTSTRAPPER_ACTION_STATE_NONE != pRelatedBundle->package.execute) > > -- > Bruce Cran > > > ------------------------------------------------------------------------------ > Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, > MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current > with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft > MVPs and experts. SALE $99.99 this month only -- learn more at: > http://p.sf.net/sfu/learnmore_122412 > _______________________________________________ > WiX-users mailing list > WiX-users@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/wix-users > ------------------------------------------------------------------------------ Master Visual Studio, SharePoint, SQL, ASP.NET, C# 2012, HTML5, CSS, MVC, Windows 8 Apps, JavaScript and much more. Keep your skills current with LearnDevNow - 3,200 step-by-step video tutorials by Microsoft MVPs and experts. SALE $99.99 this month only -- learn more at: http://p.sf.net/sfu/learnmore_122412 _______________________________________________ WiX-users mailing list WiX-users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/wix-users
