[wix-devs] S_FALSE/E_NOTFOUND -> S_OK

[Bob Arnson]

I usually love the WiX native coding style but one that bugs me is:

if (E_NOTFOUND == hr)
{
    hr = S_OK;
}

Four lines really clog up the screen especially with all the repetition for typical XML parsing (especially when throwing in S_FALSE too).

What do y'all think about something like this:

#define NotFoundIsOK(hr) if (E_NOTFOUND == hr) { hr = S_OK; }

hr = XmlGetAttributeEx(pixn, L"Name", &pFoo->sczName);
NotFoundIsOK(hr);
ExitOnFailure(hr, "Failed when querying foo Name.");

_______________________________________________________________
FireGiant  |  Dedicated support for the WiX toolset  |  http://www.firegiant.com/