I guess this is really more of a question about possibly breaking clients of 
dutil, but I think this is worth it. In this specific case, we're getting rid 
of what's otherwise an unnecessary function so far as I can tell.
 
I was able calculate the arguments passed to WcaErrorMessage (which has macro 
functions as well), with the existing stipulation that all arguments passed to 
it are strings. If only the ANSI standard put some more weight behind variadic 
args!



Heath Stewart
Software Design Engineer
Visual Studio, Microsoft
http://blogs.msdn.com/heaths

 
From: hea...@outlook.com
To: wix-devs@lists.sourceforge.net
Date: Sat, 27 Sep 2014 16:52:17 -0700
Subject: [WiX-devs] LogExitOnFailure question




I'm working on checking in variadic macros to the public fork (these have been 
a productivity boost internally - no more having to refactor/edit error 
messages and forgetting to change the macro number at the end) but ran across a 
set I didn't change before and don't think I'll be able to (safely) now.
 
LogExitOnFailure(X) is used only twice in the entire wix source tree, and I'm 
curious as to why as opposed to LogErrorId/ExitFunction.
 
This the only one I can't do anything about, it seems, because you can't modify 
va_args (I tried out a few ideas with some limited success but it's 
runtime-dependent, so not safe/supported).
 
I wouldn't remove or change these for wix3 (I even leave the old variants like 
ExitOnFailure1, but just redirect to ExitOnFailure which uses variadic macros), 
but if we're looking at some clean-up in wix4 that would be good (I was 
planning on removing all traces of numbered functions in favor of the variadic 
ones).



Heath Stewart
Software Design Engineer
Visual Studio, Microsoft
http://blogs.msdn.com/heaths
                                          

------------------------------------------------------------------------------
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs                           
          
------------------------------------------------------------------------------
Meet PCI DSS 3.0 Compliance Requirements with EventLog Analyzer
Achieve PCI DSS 3.0 Compliant Status with Out-of-the-box PCI DSS Reports
Are you Audit-Ready for PCI DSS 3.0 Compliance? Download White paper
Comply to PCI DSS 3.0 Requirement 10 and 11.5 with EventLog Analyzer
http://pubads.g.doubleclick.net/gampad/clk?id=154622311&iu=/4140/ostg.clktrk
_______________________________________________
WiX-devs mailing list
WiX-devs@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/wix-devs

Reply via email to