September 2010 - Wine-commits

Hans Leidekker : msi: Use binary search to find the insert index for a row.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: 337fb86c3bb8d77987aebd36584338f9e77597c7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=337fb86c3bb8d77987aebd365… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Sep 24 17:09:16 2010 +0200 msi: Use binary search to find the insert index for a row. --- dlls/msi/table.c | 77 +++++++++++++++++++++++++++++++++--------------------- 1 files changed, 47 insertions(+), 30 deletions(-) diff --git a/dlls/msi/table.c b/dlls/msi/table.c index 464eced..9fbb0a1 100644 --- a/dlls/msi/table.c +++ b/dlls/msi/table.c @@ -1578,39 +1578,60 @@ static UINT table_validate_new( MSITABLEVIEW *tv, MSIRECORD *rec ) return ERROR_SUCCESS; } -static UINT find_insert_index( MSITABLEVIEW *tv, MSIRECORD *rec, UINT *pidx ) +static int compare_record( MSITABLEVIEW *tv, UINT row, MSIRECORD *rec ) { - UINT r, idx, j, ivalue, x; + UINT r, i, ivalue, x; - TRACE("%p %p %p\n", tv, rec, pidx); - - for (idx = 0; idx < tv->table->row_count; idx++) + for (i = 0; i < tv->num_cols; i++ ) { - for (j = 0; j < tv->num_cols; j++ ) + r = get_table_value_from_record( tv, rec, i + 1, &ivalue ); + if (r != ERROR_SUCCESS) + return 1; + + r = TABLE_fetch_int( &tv->view, row, i + 1, &x ); + if (r != ERROR_SUCCESS) { - r = get_table_value_from_record (tv, rec, j+1, &ivalue); - if (r != ERROR_SUCCESS) - break; + WARN("TABLE_fetch_int should not fail here %u\n", r); + return -1; + } + if (ivalue > x) + { + return 1; + } + else if (ivalue == x) + { + if (i < tv->num_cols - 1) continue; + return 0; + } + else + return -1; + } + return 1; +} - r = TABLE_fetch_int(&tv->view, idx, j + 1, &x); - if (r != ERROR_SUCCESS) - return r; +static int find_insert_index( MSITABLEVIEW *tv, MSIRECORD *rec ) +{ + int idx, c, low = 0, high = tv->table->row_count - 1; - if (ivalue > x) - break; - else if (ivalue == x) - continue; - else { - TRACE("Found %d.\n", idx); - *pidx = idx; - return ERROR_SUCCESS; - } + TRACE("%p %p\n", tv, rec); + + while (low <= high) + { + idx = (low + high) / 2; + c = compare_record( tv, idx, rec ); + + if (c < 0) + high = idx - 1; + else if (c > 0) + low = idx + 1; + else + { + TRACE("found %u\n", idx); + return idx; } } - - TRACE("Found %d.\n", idx); - *pidx = idx; - return ERROR_SUCCESS; + TRACE("found %u\n", high + 1); + return high + 1; } static UINT TABLE_insert_row( struct tagMSIVIEW *view, MSIRECORD *rec, UINT row, BOOL temporary ) @@ -1626,11 +1647,7 @@ static UINT TABLE_insert_row( struct tagMSIVIEW *view, MSIRECORD *rec, UINT row, return ERROR_FUNCTION_FAILED; if (row == -1) - { - r = find_insert_index(tv, rec, &row); - if( r != ERROR_SUCCESS ) - return ERROR_FUNCTION_FAILED; - } + row = find_insert_index( tv, rec ); r = table_create_new_row( view, &row, temporary ); TRACE("insert_row returned %08x\n", r);

1 0

Lauri Kenttä : msi: Update Finnish translation.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: 57a64040e3108e8b628d0843c094089b59c56dfe URL: http://source.winehq.org/git/wine.git/?a=commit;h=57a64040e3108e8b628d0843c… Author: Lauri Kenttä <lauri.kentta(a)gmail.com> Date: Fri Sep 24 19:17:33 2010 +0300 msi: Update Finnish translation. --- dlls/msi/msi_Fi.rc | 48 ++++++++++++++++++++++++------------------------ 1 files changed, 24 insertions(+), 24 deletions(-) diff --git a/dlls/msi/msi_Fi.rc b/dlls/msi/msi_Fi.rc index 7769a8a..8984172 100644 --- a/dlls/msi/msi_Fi.rc +++ b/dlls/msi/msi_Fi.rc @@ -24,39 +24,39 @@ LANGUAGE LANG_FINNISH, SUBLANG_DEFAULT STRINGTABLE { - 4 "The specified installation package could not be opened. Please check the file path and try again." + 4 "Annettua asennuspakettia ei voitu avata. Tarkista tiedoston polku ja yrit� uudelleen." 5 "Polkua %s ei l�ydy." 9 "Anna levy %s" 10 "Windows Installer %s\n\n" \ - "Usage:\n" \ - "msiexec command {required parameter} [optional parammeter]\n\n" \ - "Install a product:\n" \ - "\t/i {package|productcode} [property]\n" \ - "\t/package {package|productcode} [property]\n" \ - "\t/a package [property]\n" \ - "Repair an installation:\n" \ - "\t/f[p|o|e|d|c|a|u|m|s|v] {package|productcode}\n" \ - "Uninstall a product:\n" \ - "\t/uninstall {package|productcode} [property]\n" \ - "\t/x {package|productcode} [property]\n" \ - "Advertise a product:\n" \ - "\t/j[u|m] package [/t transform] [/g languageid]\n" \ - "Apply a patch:\n" \ - "\t/p patchpackage [property]\n" \ - "\t/p patchpackage /a package [property]\n" \ - "Log and UI Modifiers for above commands:\n" \ - "\t/l[*][i|w|e|a|r|u|c|m|o|p|v|][+|!] logfile\n" \ + "K�ytt�:\n" \ + "msiexec komento {pakollinen parametri} [valinnainen parametri]\n\n" \ + "Asenna tuote:\n" \ + "\t/i {paketti|tuotekoodi} [ominaisuus]\n" \ + "\t/package {paketti|tuotekoodi} [ominaisuus]\n" \ + "\t/a {paketti} [ominaisuus]\n" \ + "Korjaa asennus:\n" \ + "\t/f[p|o|e|d|c|a|u|m|s|v] {paketti|tuotekoodi}\n" \ + "Poista tuote:\n" \ + "\t/uninstall {paketti|tuotekoodi} [ominaisuus]\n" \ + "\t/x {paketti|tuotekoodi} [ominaisuus]\n" \ + "Mainosta (advertise) tuotetta:\n" \ + "\t/j[u|m] paketti [/t muunnos] [/g kielitunnus]\n" \ + "Asenna korjaus:\n" \ + "\t/p korjauspaketti [ominaisuus]\n" \ + "\t/p korjauspaketti /a paketti [ominaisuus]\n" \ + "Loki- ja k�ytt�liittym�asetukset edellisille komennoille:\n" \ + "\t/l[*][i|w|e|a|r|u|c|m|o|p|v|][+|!] lokitiedosto\n" \ "\t/q{|n|b|r|f|n+|b+|b-}\n" \ - "Register MSI Service:\n" \ + "Rekister�i MSI-palvelu:\n" \ "\t/y\n" \ - "Unregister MSI Service:\n" \ + "Peru MSI-palvelun rekister�inti:\n" \ "\t/z\n" \ - "Display this help:\n" \ + "N�yt� t�m� ohje:\n" \ "\t/help\n" \ "\t/?\n" - 11 "Anna kansio, joka sis�lt�� %s" + 11 "Anna kansio, jossa on %s" 12 "Ominaisuuden asennusl�hde puuttuu." 13 "Ominaisuuden verkkolevy puuttuu." 14 "Ominaisuus:" - 15 "Valitse kansio, joka sis�lt�� %s" + 15 "Valitse kansio, jossa on %s" }

1 0

Alexandre Julliard : comctl32: Return the correct hittest for the size grip in mirrored windows.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: af200111b472e5daaf04ede61fafa357bf58683b URL: http://source.winehq.org/git/wine.git/?a=commit;h=af200111b472e5daaf04ede61… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Fri Sep 24 15:37:16 2010 +0200 comctl32: Return the correct hittest for the size grip in mirrored windows. --- dlls/comctl32/status.c | 5 ++++- 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/dlls/comctl32/status.c b/dlls/comctl32/status.c index 1b69ef2..eeaf299 100644 --- a/dlls/comctl32/status.c +++ b/dlls/comctl32/status.c @@ -1047,7 +1047,10 @@ STATUSBAR_WMNCHitTest (const STATUS_INFO *infoPtr, INT x, INT y) rect.top += 2; if (PtInRect (&rect, pt)) - return HTBOTTOMRIGHT; + { + if (GetWindowLongW( infoPtr->Self, GWL_EXSTYLE ) & WS_EX_LAYOUTRTL) return HTBOTTOMLEFT; + else return HTBOTTOMRIGHT; + } } return HTERROR;

1 0

Matej Špindler : taskkill: Add Slovenian translation.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: fe7cfdcb1527f53d18bc65ae48a6db41ef5540b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fe7cfdcb1527f53d18bc65ae4… Author: Matej Špindler <spindler.matej(a)gmail.com> Date: Fri Sep 24 10:48:28 2010 +0200 taskkill: Add Slovenian translation. --- programs/taskkill/Makefile.in | 1 + programs/taskkill/Si.rc | 43 +++++++++++++++++++++++++++++++++++++++++ 2 files changed, 44 insertions(+), 0 deletions(-) diff --git a/programs/taskkill/Makefile.in b/programs/taskkill/Makefile.in index e24df64..7af915e 100644 --- a/programs/taskkill/Makefile.in +++ b/programs/taskkill/Makefile.in @@ -13,6 +13,7 @@ RC_SRCS = \ Ko.rc \ Lt.rc \ Nl.rc \ + Si.rc \ Sv.rc @MAKE_PROG_RULES@ diff --git a/programs/taskkill/Si.rc b/programs/taskkill/Si.rc new file mode 100644 index 0000000..b278544 --- /dev/null +++ b/programs/taskkill/Si.rc @@ -0,0 +1,43 @@ +/* + * Task termination utility + * Slovenian language support + * + * Copyright 2010 Matej Špindler + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include "taskkill.h" + +#pragma code_page(65001) + +LANGUAGE LANG_SLOVENIAN, SUBLANG_DEFAULT + +STRINGTABLE +{ + STRING_USAGE, "Uporaba: taskkill [/?] [/f] [/im ImeProcesa | /pid IDProcesa]\n" + STRING_INVALID_OPTION, "Napaka: Nepoznana ali nepopolna ukazna vrstica.\n" + STRING_INVALID_PARAM, "Napaka: Nepravilen parameter ukazne vrstice.\n" + STRING_MISSING_OPTION, "Napaka: Ena izmed opcija /im ali /pid mora biti izbrana.\n" + STRING_MISSING_PARAM, "Napaka: Opcija %s pričakuje parameter v ukazni vstici.\n" + STRING_MUTUAL_EXCLUSIVE, "Napaka: Opciji /im in /pid se izključujeta.\n" + STRING_CLOSE_PID_SEARCH, "Sporočilo zapri poslano vrhnjemu oknu procesa s PID %u.\n" + STRING_CLOSE_PROC_SRCH, "Sporočilo zapri poslano vrhnjemu oknu procesa \"%s\" s PID %u.\n" + STRING_TERM_PID_SEARCH, "Proces s PID %u je bil prisilno končan.\n" + STRING_TERM_PROC_SEARCH, "Proces \"%s\" s PID %u je bil prisilno končan.\n" + STRING_SEARCH_FAILED, "Napaka: Procesa \"%s\" ni mogoče najti.\n" + STRING_ENUM_FAILED, "Napaka: Ne morem dobiti seznama procesov.\n" + STRING_TERMINATE_FAILED, "Napaka: Procesa \"%s\" ni mogoče končati.\n" +}

1 0

James McKenzie : wineps.drv: Correct a typo.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: 95c3c3fdc67c3dc7cb8f5fb72050b33c64c2f106 URL: http://source.winehq.org/git/wine.git/?a=commit;h=95c3c3fdc67c3dc7cb8f5fb72… Author: James McKenzie <jjmckenzie51(a)earthlink.net> Date: Thu Sep 23 21:07:03 2010 -0700 wineps.drv: Correct a typo. --- dlls/wineps.drv/escape.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wineps.drv/escape.c b/dlls/wineps.drv/escape.c index b5a3720..988381e 100644 --- a/dlls/wineps.drv/escape.c +++ b/dlls/wineps.drv/escape.c @@ -310,7 +310,7 @@ INT CDECL PSDRV_ExtEscape( PSDRV_PDEVICE *physDev, INT nEscape, INT cbInput, LPC TRACE("END_PATH\n"); if(!physDev->pathdepth) { - ERR("END_PATH called without a BEIGN_PATH\n"); + ERR("END_PATH called without a BEGIN_PATH\n"); return -1; } TRACE("RenderMode = %d, FillMode = %d, BkMode = %d\n",

1 0

Aric Stewart : quartz: Allow for proper setting of the video window to WS_CHILD.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: 97343b0f002902c0495969f25e48ace8983bf692 URL: http://source.winehq.org/git/wine.git/?a=commit;h=97343b0f002902c0495969f25… Author: Aric Stewart <aric(a)codeweavers.com> Date: Fri Sep 24 08:53:14 2010 -0500 quartz: Allow for proper setting of the video window to WS_CHILD. If the set_WindowStyle comes before the set_Owner then the style was not being set properly (WS_CHILD cannot be set on a window with a NULL parent). So reset the window style when the set_Owner command comes in to make the new style allowable. --- dlls/quartz/videorenderer.c | 17 +++++++++++++++-- 1 files changed, 15 insertions(+), 2 deletions(-) diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c index 0af8cc6..d3ccede 100644 --- a/dlls/quartz/videorenderer.c +++ b/dlls/quartz/videorenderer.c @@ -89,6 +89,7 @@ typedef struct VideoRendererImpl BOOL bAggregatable; REFERENCE_TIME rtLastStop; MediaSeekingImpl mediaSeeking; + LONG WindowStyle; /* During pause we can hold a single sample, for use in GetCurrentImage */ IMediaSample *sample_held; @@ -611,6 +612,7 @@ HRESULT VideoRenderer_create(IUnknown * pUnkOuter, LPVOID * ppv) ZeroMemory(&pVideoRenderer->DestRect, sizeof(RECT)); ZeroMemory(&pVideoRenderer->WindowPos, sizeof(RECT)); pVideoRenderer->hWndMsgDrain = NULL; + pVideoRenderer->WindowStyle = WS_OVERLAPPED; /* construct input pin */ piInput.dir = PINDIR_INPUT; @@ -1782,8 +1784,10 @@ static HRESULT WINAPI Videowindow_put_WindowStyle(IVideoWindow *iface, if (WindowStyle & (WS_DISABLED|WS_HSCROLL|WS_ICONIC|WS_MAXIMIZE|WS_MINIMIZE|WS_VSCROLL)) return E_INVALIDARG; - + SetWindowLongA(This->hWnd, GWL_STYLE, WindowStyle); + SetWindowPos(This->hWnd,0,0,0,0,0,SWP_FRAMECHANGED|SWP_NOSIZE|SWP_NOZORDER); + This->WindowStyle = WindowStyle; return S_OK; } @@ -1794,7 +1798,7 @@ static HRESULT WINAPI Videowindow_get_WindowStyle(IVideoWindow *iface, TRACE("(%p/%p)->(%p)\n", This, iface, WindowStyle); - *WindowStyle = GetWindowLongA(This->hWnd, GWL_STYLE); + *WindowStyle = This->WindowStyle; return S_OK; } @@ -2013,6 +2017,15 @@ static HRESULT WINAPI Videowindow_put_Owner(IVideoWindow *iface, TRACE("(%p/%p)->(%08x)\n", This, iface, (DWORD) Owner); SetParent(This->hWnd, (HWND)Owner); + if (This->WindowStyle & WS_CHILD) + { + LONG old = GetWindowLongA(This->hWnd, GWL_STYLE); + if (old != This->WindowStyle) + { + SetWindowLongA(This->hWnd, GWL_STYLE, This->WindowStyle); + SetWindowPos(This->hWnd,0,0,0,0,0,SWP_FRAMECHANGED|SWP_NOSIZE|SWP_NOZORDER); + } + } return S_OK; }

1 0

Juan Lang : wintrust/tests: Test CryptSIPCreateIndirectData.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: ea715da1c6882ab89f20385cfb507c0388ff835c URL: http://source.winehq.org/git/wine.git/?a=commit;h=ea715da1c6882ab89f20385cf… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Sep 2 18:26:28 2010 -0700 wintrust/tests: Test CryptSIPCreateIndirectData. --- dlls/wintrust/tests/softpub.c | 268 ++++++++++++++++++++++++++++++++++++++--- 1 files changed, 250 insertions(+), 18 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ea715da1c6882ab89f203…

1 0

Juan Lang : crypt32: Add parameter checking to CryptSIPCreateIndirectData.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: 604e39dde5c189b6e35cc3205c34d3d8cb18a5f9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=604e39dde5c189b6e35cc3205… Author: Juan Lang <juan.lang(a)gmail.com> Date: Thu Sep 2 18:31:23 2010 -0700 crypt32: Add parameter checking to CryptSIPCreateIndirectData. --- dlls/crypt32/sip.c | 5 +++++ dlls/wintrust/tests/softpub.c | 13 ------------- 2 files changed, 5 insertions(+), 13 deletions(-) diff --git a/dlls/crypt32/sip.c b/dlls/crypt32/sip.c index e207457..d2f55f2 100644 --- a/dlls/crypt32/sip.c +++ b/dlls/crypt32/sip.c @@ -715,6 +715,11 @@ BOOL WINAPI CryptSIPCreateIndirectData(SIP_SUBJECTINFO* pSubjectInfo, DWORD* pcb TRACE("(%p %p %p)\n", pSubjectInfo, pcbIndirectData, pIndirectData); + if (!pSubjectInfo || !pSubjectInfo->pgSubjectType || !pcbIndirectData) + { + SetLastError(ERROR_INVALID_PARAMETER); + return FALSE; + } if ((sip = CRYPT_GetCachedSIP(pSubjectInfo->pgSubjectType))) ret = sip->info.pfCreate(pSubjectInfo, pcbIndirectData, pIndirectData); TRACE("returning %d\n", ret); diff --git a/dlls/wintrust/tests/softpub.c b/dlls/wintrust/tests/softpub.c index dd12000..b431ff8 100644 --- a/dlls/wintrust/tests/softpub.c +++ b/dlls/wintrust/tests/softpub.c @@ -619,28 +619,17 @@ static void test_sip_create_indirect_data(void) skip("Missing CryptSIPCreateIndirectData\n"); return; } -if (0) -{ - /* FIXME: crashes Wine */ SetLastError(0xdeadbeef); ret = CryptSIPCreateIndirectData_p(NULL, NULL, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); -} - /* FIXME: this assignment is only to avoid a crash in Wine, it isn't - * needed by native. - */ - subjinfo.pgSubjectType = &unknown; SetLastError(0xdeadbeef); ret = CryptSIPCreateIndirectData_p(&subjinfo, NULL, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); subjinfo.cbSize = sizeof(subjinfo); SetLastError(0xdeadbeef); ret = CryptSIPCreateIndirectData_p(&subjinfo, NULL, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); file = create_temp_file(temp_file); @@ -655,13 +644,11 @@ if (0) subjinfo.hFile = file; SetLastError(0xdeadbeef); ret = CryptSIPCreateIndirectData_p(&subjinfo, NULL, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); subjinfo.pgSubjectType = &unknown; SetLastError(0xdeadbeef); ret = CryptSIPCreateIndirectData_p(&subjinfo, NULL, NULL); - todo_wine ok(!ret && GetLastError() == ERROR_INVALID_PARAMETER, "expected ERROR_INVALID_PARAMETER, got %d\n", GetLastError()); count = 0xdeadbeef;

1 0

Hans Leidekker : msi: Remove unused field from MSICOMPONENT structure.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: d56d0d98a3a9603e6d149678580203c69aff2af9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d56d0d98a3a9603e6d1496785… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Sep 24 17:08:36 2010 +0200 msi: Remove unused field from MSICOMPONENT structure. --- dlls/msi/msipriv.h | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/msi/msipriv.h b/dlls/msi/msipriv.h index 99486ec..2e87918 100644 --- a/dlls/msi/msipriv.h +++ b/dlls/msi/msipriv.h @@ -399,7 +399,6 @@ typedef struct tagMSIFEATURE typedef struct tagMSICOMPONENT { struct list entry; - DWORD magic; LPWSTR Component; LPWSTR ComponentId; LPWSTR Directory;

1 0

Hans Leidekker : msi/test: Add tests for MsiGetFeatureState.
by Alexandre Julliard 24 Sep '10

24 Sep '10

Module: wine Branch: master Commit: d59c99067a3f17f3ecb75689e1355b42b5101089 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d59c99067a3f17f3ecb75689e… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Fri Sep 24 17:08:48 2010 +0200 msi/test: Add tests for MsiGetFeatureState. --- dlls/msi/tests/package.c | 13 +++++++++++++ 1 files changed, 13 insertions(+), 0 deletions(-) diff --git a/dlls/msi/tests/package.c b/dlls/msi/tests/package.c index 841ef14..7efb6a2 100644 --- a/dlls/msi/tests/package.c +++ b/dlls/msi/tests/package.c @@ -3283,6 +3283,19 @@ static void test_states(void) r = MsiDoAction( hpkg, "FileCost"); ok( r == ERROR_SUCCESS, "file cost failed\n"); + r = MsiGetFeatureState(hpkg, "one", NULL, NULL); + ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); + + action = 0xdeadbee; + r = MsiGetFeatureState(hpkg, "one", NULL, &action); + ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); + ok( state == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", state); + + state = 0xdeadbee; + r = MsiGetFeatureState( hpkg, "one", &state, NULL); + ok( r == ERROR_SUCCESS, "Expected ERROR_SUCCESS, got %d\n", r ); + ok( state == INSTALLSTATE_UNKNOWN, "Expected INSTALLSTATE_UNKNOWN, got %d\n", state); + state = 0xdeadbee; action = 0xdeadbee; r = MsiGetFeatureState(hpkg, "one", &state, &action);

1 0