October 2009 - Wine-commits

Piotr Caban : jscript: Added Date.toGMTString implementation.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: adc2a5697b745c49ea33a14721f2c5dc1172c408 URL: http://source.winehq.org/git/wine.git/?a=commit;h=adc2a5697b745c49ea33a1472… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Oct 7 22:11:38 2009 +0200 jscript: Added Date.toGMTString implementation. --- dlls/jscript/date.c | 19 ++++++++++++------- dlls/jscript/tests/api.js | 1 + 2 files changed, 13 insertions(+), 7 deletions(-) diff --git a/dlls/jscript/date.c b/dlls/jscript/date.c index c753587..f8d3522 100644 --- a/dlls/jscript/date.c +++ b/dlls/jscript/date.c @@ -675,9 +675,8 @@ static HRESULT Date_valueOf(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISP return S_OK; } -/* ECMA-262 3rd Edition 15.9.5.42 */ -static HRESULT Date_toUTCString(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISPPARAMS *dp, - VARIANT *retv, jsexcept_t *ei, IServiceProvider *caller) +static inline HRESULT create_utc_string(script_ctx_t *ctx, vdisp_t *jsthis, + VARIANT *retv, jsexcept_t *ei) { static const WCHAR NaNW[] = { 'N','a','N',0 }; static const WCHAR formatADW[] = { '%','s',',',' ','%','d',' ','%','s',' ','%','d',' ', @@ -702,8 +701,6 @@ static HRESULT Date_toUTCString(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, int len, size, year, day; DWORD lcid_en, week_id, month_id; - TRACE("\n"); - if(!(date = date_this(jsthis))) return throw_type_error(ctx, ei, IDS_NOT_DATE, NULL); @@ -781,11 +778,19 @@ static HRESULT Date_toUTCString(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, return S_OK; } +/* ECMA-262 3rd Edition 15.9.5.42 */ +static HRESULT Date_toUTCString(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISPPARAMS *dp, + VARIANT *retv, jsexcept_t *ei, IServiceProvider *caller) +{ + TRACE("\n"); + return create_utc_string(ctx, jsthis, retv, ei); +} + static HRESULT Date_toGMTString(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, DISPPARAMS *dp, VARIANT *retv, jsexcept_t *ei, IServiceProvider *caller) { - FIXME("\n"); - return E_NOTIMPL; + TRACE("\n"); + return create_utc_string(ctx, jsthis, retv, ei); } /* ECMA-262 3rd Edition 15.9.5.3 */ diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index 268820c..ea1a5b2 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -1874,6 +1874,7 @@ testFunctions(Date.prototype, [ ["toString", 0], ["toTimeString", 0], ["toUTCString", 0], + ["toGMTString", 0], ["valueOf", 0] ]);

1 0

Piotr Caban : jscript: Fix implementation of Global.escape.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: 38dbc74a0afc1379c1f3766f9153557808334a8b URL: http://source.winehq.org/git/wine.git/?a=commit;h=38dbc74a0afc1379c1f3766f9… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Oct 7 22:11:43 2009 +0200 jscript: Fix implementation of Global.escape. --- dlls/jscript/global.c | 40 ++++++++++++++++++++-------------------- 1 files changed, 20 insertions(+), 20 deletions(-) diff --git a/dlls/jscript/global.c b/dlls/jscript/global.c index 5fd8114..ef86f26 100644 --- a/dlls/jscript/global.c +++ b/dlls/jscript/global.c @@ -301,8 +301,7 @@ static HRESULT JSGlobal_escape(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, D { BSTR ret, str; const WCHAR *ptr; - DWORD len = 0, i, size; - char buf[4]; + DWORD len = 0; HRESULT hres; TRACE("\n"); @@ -325,17 +324,13 @@ static HRESULT JSGlobal_escape(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, D return hres; for(ptr=str; *ptr; ptr++) { - if(isalnumW(*ptr) || *ptr=='*' || *ptr=='@' || *ptr=='-' || *ptr=='_' - || *ptr=='+' || *ptr=='.' || *ptr=='/') + if(*ptr > 0xff) + len += 6; + else if(isalnum((char)*ptr) || *ptr=='*' || *ptr=='@' || *ptr=='-' + || *ptr=='_' || *ptr=='+' || *ptr=='.' || *ptr=='/') len++; - else { - size = WideCharToMultiByte(CP_UTF8, 0, ptr, 1, NULL, 0, NULL, NULL)*3; - if(!size) { - FIXME("throw Error\n"); - return E_FAIL; - } - len += size; - } + else + len += 3; } ret = SysAllocStringLen(NULL, len); @@ -344,16 +339,21 @@ static HRESULT JSGlobal_escape(script_ctx_t *ctx, vdisp_t *jsthis, WORD flags, D len = 0; for(ptr=str; *ptr; ptr++) { - if(isalnumW(*ptr) || *ptr=='*' || *ptr=='@' || *ptr=='-' || *ptr=='_' - || *ptr=='+' || *ptr=='.' || *ptr=='/') + if(*ptr > 0xff) { + ret[len++] = '%'; + ret[len++] = 'u'; + ret[len++] = int_to_char(*ptr >> 12); + ret[len++] = int_to_char((*ptr >> 8) & 0xf); + ret[len++] = int_to_char((*ptr >> 4) & 0xf); + ret[len++] = int_to_char(*ptr & 0xf); + } + else if(isalnum((char)*ptr) || *ptr=='*' || *ptr=='@' || *ptr=='-' + || *ptr=='_' || *ptr=='+' || *ptr=='.' || *ptr=='/') ret[len++] = *ptr; else { - size = WideCharToMultiByte(CP_UTF8, 0, ptr, 1, buf, sizeof(buf), NULL, NULL); - for(i=0; i<size; i++) { - ret[len++] = '%'; - ret[len++] = int_to_char(buf[i] >> 4); - ret[len++] = int_to_char(buf[i] & 0xf); - } + ret[len++] = '%'; + ret[len++] = int_to_char(*ptr >> 4); + ret[len++] = int_to_char(*ptr & 0xf); } }

1 0

Piotr Caban : jscript: Fixed escaped characters processing.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: 50705c568303e0b938c7620c9838a386b723eb3e URL: http://source.winehq.org/git/wine.git/?a=commit;h=50705c568303e0b938c7620c9… Author: Piotr Caban <piotr.caban(a)gmail.com> Date: Wed Oct 7 22:11:48 2009 +0200 jscript: Fixed escaped characters processing. --- dlls/jscript/lex.c | 16 +++++++++------- dlls/jscript/tests/api.js | 8 ++++++++ 2 files changed, 17 insertions(+), 7 deletions(-) diff --git a/dlls/jscript/lex.c b/dlls/jscript/lex.c index 680ce55..43089d3 100644 --- a/dlls/jscript/lex.c +++ b/dlls/jscript/lex.c @@ -287,7 +287,7 @@ static BOOL unescape(WCHAR *str) i = hex_to_int(*++p); if(i == -1) return FALSE; - c += 1 << 4; + c += i << 4; i = hex_to_int(*++p); if(i == -1) @@ -297,13 +297,15 @@ static BOOL unescape(WCHAR *str) default: if(isdigitW(*p)) { c = *p++ - '0'; - while(isdigitW(*p)) - c = c*10 + (*p++ - '0'); - *pd++ = c; - continue; + if(isdigitW(*p)) { + c = c*8 + (*p++ - '0'); + if(isdigitW(*p)) + c = c*8 + (*p++ - '0'); + } + p--; } - - c = *p; + else + c = *p; } *pd++ = c; diff --git a/dlls/jscript/tests/api.js b/dlls/jscript/tests/api.js index ea1a5b2..be85566 100644 --- a/dlls/jscript/tests/api.js +++ b/dlls/jscript/tests/api.js @@ -66,6 +66,8 @@ tmp = escape("a1b c!d+e(a)*-_+./,"); ok(tmp === "a1b%20c%21d+e(a)*-_+./%2C", "escape('a1b c!d+e(a)*-_+./,') = " + tmp); tmp = escape(); ok(tmp === "undefined", "escape() = " + tmp); +tmp = escape('\u1234\123\xf3'); +ok(tmp == "%u1234S%F3", "escape('\u1234\123\xf3') = " + tmp); tmp = unescape("abc"); ok(tmp === "abc", "unescape('abc') = " + tmp); @@ -201,6 +203,12 @@ tmp = "abc".charCodeAt(true); ok(tmp === 0x62, "'abc'.charCodeAt(true) = " + tmp); tmp = "abc".charCodeAt(0,2); ok(tmp === 0x61, "'abc'.charCodeAt(0,2) = " + tmp); +tmp = "\u49F4".charCodeAt(0); +ok(tmp === 0x49F4, "'\u49F4'.charCodeAt(0) = " + tmp); +tmp = "\052".charCodeAt(0); +ok(tmp === 0x2A, "'\052'.charCodeAt(0) = " + tmp); +tmp = "\xa2".charCodeAt(0); +ok(tmp === 0xA2, "'\xa2'.charCodeAt(0) = " + tmp); tmp = "abcd".substring(1,3); ok(tmp === "bc", "'abcd'.substring(1,3) = " + tmp);

1 0

Roderick Colenbrander : winex11: Move XRENDERINFO allocation in a separate function.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: 3e5e0ee80769f027e870c2759c3c3310ae440bcc URL: http://source.winehq.org/git/wine.git/?a=commit;h=3e5e0ee80769f027e870c2759… Author: Roderick Colenbrander <thunderbird2k(a)gmail.com> Date: Tue Oct 6 20:41:33 2009 +0200 winex11: Move XRENDERINFO allocation in a separate function. --- dlls/winex11.drv/xrender.c | 32 ++++++++++++++++++++++++-------- 1 files changed, 24 insertions(+), 8 deletions(-) diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index 076b6d4..d4474dd 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -207,6 +207,23 @@ static CRITICAL_SECTION xrender_cs = { &critsect_debug, -1, 0, 0, 0, 0 }; #define NATIVE_BYTE_ORDER LSBFirst #endif +static XRENDERINFO get_xrender_info(X11DRV_PDEVICE *physDev) +{ + if(!physDev->xrender) + { + physDev->xrender = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct tagXRENDERINFO)); + + if(!physDev->xrender) + { + ERR("Unable to allocate XRENDERINFO!\n"); + return NULL; + } + physDev->xrender->cache_index = -1; + } + + return physDev->xrender; +} + static BOOL get_xrender_template(const WineXRenderFormatTemplate *fmt, XRenderPictFormat *templ, unsigned long *mask) { templ->id = 0; @@ -795,6 +812,7 @@ void X11DRV_XRender_Finalize(void) BOOL X11DRV_XRender_SelectFont(X11DRV_PDEVICE *physDev, HFONT hfont) { LFANDSIZE lfsz; + XRENDERINFO info; GetObjectW(hfont, sizeof(lfsz.lf), &lfsz.lf); TRACE("h=%d w=%d weight=%d it=%d charset=%d name=%s\n", @@ -806,15 +824,13 @@ BOOL X11DRV_XRender_SelectFont(X11DRV_PDEVICE *physDev, HFONT hfont) GetWorldTransform( physDev->hdc, &lfsz.xform ); lfsz_calc_hash(&lfsz); + info = get_xrender_info(physDev); + if (!info) return 0; + EnterCriticalSection(&xrender_cs); - if(!physDev->xrender) { - physDev->xrender = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, - sizeof(*physDev->xrender)); - physDev->xrender->cache_index = -1; - } - else if(physDev->xrender->cache_index != -1) - dec_ref_cache(physDev->xrender->cache_index); - physDev->xrender->cache_index = GetCacheEntry(physDev, &lfsz); + if(info->cache_index != -1) + dec_ref_cache(info->cache_index); + info->cache_index = GetCacheEntry(physDev, &lfsz); LeaveCriticalSection(&xrender_cs); return 0; }

1 0

Alexandre Julliard : winex11: Rename the XRENDERINFO type to make it clear it's a pointer.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: 344cf57f1bb569c7593d21d14e9effb493457ff0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=344cf57f1bb569c7593d21d14… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Oct 8 12:10:59 2009 +0200 winex11: Rename the XRENDERINFO type to make it clear it's a pointer. --- dlls/winex11.drv/x11drv.h | 4 ++-- dlls/winex11.drv/xrender.c | 8 ++++---- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/dlls/winex11.drv/x11drv.h b/dlls/winex11.drv/x11drv.h index 3faffdf..eee5849 100644 --- a/dlls/winex11.drv/x11drv.h +++ b/dlls/winex11.drv/x11drv.h @@ -138,7 +138,7 @@ typedef struct /* X physical font */ typedef UINT X_PHYSFONT; -typedef struct tagXRENDERINFO *XRENDERINFO; +struct xrender_info; /* X physical device */ typedef struct @@ -163,7 +163,7 @@ typedef struct Drawable gl_drawable; Pixmap pixmap; /* Pixmap for a GLXPixmap gl_drawable */ int gl_copy; - XRENDERINFO xrender; + struct xrender_info *xrender; } X11DRV_PDEVICE; diff --git a/dlls/winex11.drv/xrender.c b/dlls/winex11.drv/xrender.c index d4474dd..8b2f0dd 100644 --- a/dlls/winex11.drv/xrender.c +++ b/dlls/winex11.drv/xrender.c @@ -140,7 +140,7 @@ typedef struct INT next; } gsCacheEntry; -struct tagXRENDERINFO +struct xrender_info { int cache_index; Picture pict; @@ -207,11 +207,11 @@ static CRITICAL_SECTION xrender_cs = { &critsect_debug, -1, 0, 0, 0, 0 }; #define NATIVE_BYTE_ORDER LSBFirst #endif -static XRENDERINFO get_xrender_info(X11DRV_PDEVICE *physDev) +static struct xrender_info *get_xrender_info(X11DRV_PDEVICE *physDev) { if(!physDev->xrender) { - physDev->xrender = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(struct tagXRENDERINFO)); + physDev->xrender = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, sizeof(*physDev->xrender)); if(!physDev->xrender) { @@ -812,7 +812,7 @@ void X11DRV_XRender_Finalize(void) BOOL X11DRV_XRender_SelectFont(X11DRV_PDEVICE *physDev, HFONT hfont) { LFANDSIZE lfsz; - XRENDERINFO info; + struct xrender_info *info; GetObjectW(hfont, sizeof(lfsz.lf), &lfsz.lf); TRACE("h=%d w=%d weight=%d it=%d charset=%d name=%s\n",

1 0

Nikolay Sivov : comctl32/monthcal: Cleanup top drawing helper namespace, reduce parameter count in prev/next draw helpers.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: bd58b130bac9586a8134f564ff9dfb70cb55366f URL: http://source.winehq.org/git/wine.git/?a=commit;h=bd58b130bac9586a8134f564f… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Oct 7 22:13:07 2009 +0400 comctl32/monthcal: Cleanup top drawing helper namespace, reduce parameter count in prev/next draw helpers. --- dlls/comctl32/monthcal.c | 189 +++++++++++++++++++++++---------------------- 1 files changed, 97 insertions(+), 92 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=bd58b130bac9586a8134f…

1 0

Nikolay Sivov : comctl32/monthcal: Fix the way a first day of week is handled.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: c8f35a136d51b949df252d932e9b10c2e2f85419 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c8f35a136d51b949df252d932… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Oct 8 00:45:05 2009 +0400 comctl32/monthcal: Fix the way a first day of week is handled. --- dlls/comctl32/comctl32.h | 2 +- dlls/comctl32/datetime.c | 2 +- dlls/comctl32/monthcal.c | 80 ++++++++++++++++++++++++++++++---------------- 3 files changed, 54 insertions(+), 30 deletions(-) diff --git a/dlls/comctl32/comctl32.h b/dlls/comctl32/comctl32.h index e0d1970..4cad907 100644 --- a/dlls/comctl32/comctl32.h +++ b/dlls/comctl32/comctl32.h @@ -231,7 +231,7 @@ extern void UPDOWN_Unregister(void); int MONTHCAL_MonthLength(int month, int year); -int MONTHCAL_CalculateDayOfWeek(DWORD day, DWORD month, DWORD year); +int MONTHCAL_CalculateDayOfWeek(WORD day, WORD month, WORD year); extern void THEMING_Initialize(void); extern void THEMING_Uninitialize(void); diff --git a/dlls/comctl32/datetime.c b/dlls/comctl32/datetime.c index 891ca91..097e442 100644 --- a/dlls/comctl32/datetime.c +++ b/dlls/comctl32/datetime.c @@ -88,7 +88,7 @@ typedef struct /* in monthcal.c */ extern int MONTHCAL_MonthLength(int month, int year); -extern int MONTHCAL_CalculateDayOfWeek(DWORD day, DWORD month, DWORD year); +extern int MONTHCAL_CalculateDayOfWeek(WORD day, WORD month, WORD year); /* this list of defines is closely related to `allowedformatchars' defined * in datetime.c; the high nibble indicates the `base type' of the format diff --git a/dlls/comctl32/monthcal.c b/dlls/comctl32/monthcal.c index 745d99b..2450a5c 100644 --- a/dlls/comctl32/monthcal.c +++ b/dlls/comctl32/monthcal.c @@ -97,8 +97,10 @@ typedef struct INT delta; /* scroll rate; # of months that the */ /* control moves when user clicks a scroll button */ int visible; /* # of months visible */ - int firstDay; /* Start month calendar with firstDay's day */ - int firstDayHighWord; /* High word only used externally */ + int firstDay; /* Start month calendar with firstDay's day, + stored in SYSTEMTIME format */ + BOOL firstDaySet; /* first week day differs from locale defined */ + int monthRange; MONTHDAYSTATE *monthdayState; SYSTEMTIME todaysDate; @@ -177,10 +179,12 @@ static inline void MONTHCAL_NotifySelect(const MONTHCAL_INFO *infoPtr) int MONTHCAL_MonthLength(int month, int year) { const int mdays[] = {31, 28, 31, 30, 31, 30, 31, 31, 30, 31, 30, 31, 0}; - /*Wrap around, this eases handling*/ + /* Wrap around, this eases handling. Getting length only we shouldn't care + about year change here cause January and December have + the same day quantity */ if(month == 0) month = 12; - if(month == 13) + else if(month == 13) month = 1; /* if we have a leap year add 1 day to February */ @@ -372,9 +376,17 @@ static void MONTHCAL_CopyDate(const SYSTEMTIME *from, SYSTEMTIME *to) 0 = Sunday. */ -/* returns the day in the week(0 == sunday, 6 == saturday) */ -/* day(1 == 1st, 2 == 2nd... etc), year is the year value */ -int MONTHCAL_CalculateDayOfWeek(DWORD day, DWORD month, DWORD year) +/* Returns the day in the week + * + * PARAMETERS + * [i] day : day of month [1, 31] + * [I] month : month number [1, 12] + * [I] year : year value + * + * RETURN VALUE + * day of week in SYSTEMTIME format: (0 == sunday,..., 6 == saturday) + */ +int MONTHCAL_CalculateDayOfWeek(WORD day, WORD month, WORD year) { year-=(month < 3); @@ -751,14 +763,14 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT dayrect.right += infoPtr->width_increment; } -/* draw day numbers; first, the previous month */ - + /* draw day numbers; first, the previous month */ firstDay = MONTHCAL_CalculateDayOfWeek(1, infoPtr->curSel.wMonth, infoPtr->curSel.wYear); - day = MONTHCAL_MonthLength(prevMonth, infoPtr->curSel.wYear) + - (infoPtr->firstDay + 7 - firstDay)%7 + 1; + day = MONTHCAL_MonthLength(prevMonth, infoPtr->curSel.wYear) + + (infoPtr->firstDay - firstDay)%7 + 1; + if (day > MONTHCAL_MonthLength(prevMonth, infoPtr->curSel.wYear)) - day -=7; + day -= 7; startofprescal = day; mask = 1<<(day-1); @@ -921,7 +933,7 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT { /* calculate all those exceptions for january */ weeknum1 = MONTHCAL_CalculateDayOfWeek(1, 1, infoPtr->curSel.wYear); - if ((infoPtr->firstDay + 7 - weeknum1) % 7 > mindays) + if ((infoPtr->firstDay - weeknum1) % 7 > mindays) weeknum = 1; else { @@ -932,7 +944,7 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT weeknum += startofprescal + 7; weeknum /= 7; weeknum1 = MONTHCAL_CalculateDayOfWeek(1, 1, infoPtr->curSel.wYear - 1); - if ((infoPtr->firstDay + 7 - weeknum1) % 7 > mindays) weeknum++; + if ((infoPtr->firstDay - weeknum1) % 7 > mindays) weeknum++; } } else @@ -944,7 +956,7 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT weeknum += startofprescal + 7; weeknum /= 7; weeknum1 = MONTHCAL_CalculateDayOfWeek(1, 1, infoPtr->curSel.wYear); - if ((infoPtr->firstDay + 7 - weeknum1) % 7 > mindays) weeknum++; + if ((infoPtr->firstDay - weeknum1) % 7 > mindays) weeknum++; } dayrect = infoPtr->weeknums; @@ -1089,10 +1101,15 @@ MONTHCAL_SetMonthDelta(MONTHCAL_INFO *infoPtr, INT delta) } -static LRESULT +static inline LRESULT MONTHCAL_GetFirstDayOfWeek(const MONTHCAL_INFO *infoPtr) { - return MAKELONG(infoPtr->firstDay, infoPtr->firstDayHighWord); + int day; + + /* convert from SYSTEMTIME to locale format */ + day = (infoPtr->firstDay >= 0) ? (infoPtr->firstDay+6)%7 : infoPtr->firstDay; + + return MAKELONG(day, infoPtr->firstDaySet); } @@ -1115,34 +1132,42 @@ MONTHCAL_GetFirstDayOfWeek(const MONTHCAL_INFO *infoPtr) static LRESULT MONTHCAL_SetFirstDayOfWeek(MONTHCAL_INFO *infoPtr, INT day) { - int prev = MAKELONG(infoPtr->firstDay, infoPtr->firstDayHighWord); + LRESULT prev = MONTHCAL_GetFirstDayOfWeek(infoPtr); + int new_day; - TRACE("day %d\n", day); + TRACE("%d\n", day); if(day == -1) { - int localFirstDay; WCHAR buf[80]; GetLocaleInfoW(LOCALE_USER_DEFAULT, LOCALE_IFIRSTDAYOFWEEK, buf, countof(buf)); TRACE("%s %d\n", debugstr_w(buf), strlenW(buf)); - localFirstDay = atoiW(buf); + new_day = atoiW(buf); - infoPtr->firstDay = localFirstDay; - infoPtr->firstDayHighWord = FALSE; + infoPtr->firstDaySet = FALSE; } else if(day >= 7) { - infoPtr->firstDay = 6; /* max first day allowed */ - infoPtr->firstDayHighWord = TRUE; + new_day = 6; /* max first day allowed */ + infoPtr->firstDaySet = TRUE; } else { - infoPtr->firstDay = day; - infoPtr->firstDayHighWord = TRUE; + /* Native behaviour for that case is broken: invalid date number >31 + got displayed at (0,0) position, current month starts always from + (1,0) position. Should be implemnted here as well. */ + if (day < -1) + FIXME("No bug compatibility for day=%d\n", day); + + new_day = day; + infoPtr->firstDaySet = TRUE; } + /* convert from locale to SYSTEMTIME format */ + infoPtr->firstDay = (new_day >= 0) ? (++new_day) % 7 : new_day; + InvalidateRect(infoPtr->hwndSelf, NULL, FALSE); return prev; @@ -2248,7 +2273,6 @@ MONTHCAL_Create(HWND hwnd, LPCREATESTRUCTW lpcs) /* FIXME: calculate systemtime ->> localtime(substract timezoneinfo) */ GetLocalTime(&infoPtr->todaysDate); - infoPtr->firstDayHighWord = FALSE; MONTHCAL_SetFirstDayOfWeek(infoPtr, -1); infoPtr->maxSelCount = (infoPtr->dwStyle & MCS_MULTISELECT) ? 7 : 1;

1 0

Nikolay Sivov : comctl32: Use user32 control names from public header constants instead of defining over and over again .
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: f37130c83bb7effc62670f15d79e84119502271f URL: http://source.winehq.org/git/wine.git/?a=commit;h=f37130c83bb7effc62670f15d… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Oct 8 01:04:08 2009 +0400 comctl32: Use user32 control names from public header constants instead of defining over and over again. --- dlls/comctl32/comboex.c | 6 ++---- dlls/comctl32/ipaddress.c | 3 +-- dlls/comctl32/listview.c | 5 ++--- dlls/comctl32/monthcal.c | 4 +--- dlls/comctl32/treeview.c | 3 +-- dlls/comctl32/updown.c | 6 ++---- 6 files changed, 9 insertions(+), 18 deletions(-) diff --git a/dlls/comctl32/comboex.c b/dlls/comctl32/comboex.c index 1484ba8..d67b918 100644 --- a/dlls/comctl32/comboex.c +++ b/dlls/comctl32/comboex.c @@ -968,8 +968,6 @@ static INT COMBOEX_SetItemHeight (COMBOEX_INFO const *infoPtr, INT index, UINT h static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) { - static const WCHAR COMBOBOX[] = { 'C', 'o', 'm', 'b', 'o', 'B', 'o', 'x', 0 }; - static const WCHAR EDIT[] = { 'E', 'D', 'I', 'T', 0 }; static const WCHAR NIL[] = { 0 }; COMBOEX_INFO *infoPtr; LOGFONTW mylogfont; @@ -1011,7 +1009,7 @@ static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) /* We also need to place the edit control at the proper location */ /* (allow space for the icons). */ - infoPtr->hwndCombo = CreateWindowW (COMBOBOX, NIL, + infoPtr->hwndCombo = CreateWindowW (WC_COMBOBOXW, NIL, /* following line added to match native */ WS_CLIPSIBLINGS | WS_CLIPCHILDREN | WS_VSCROLL | CBS_NOINTEGRALHEIGHT | CBS_DROPDOWNLIST | @@ -1045,7 +1043,7 @@ static LRESULT COMBOEX_Create (HWND hwnd, CREATESTRUCTA const *cs) * It is created only for CBS_DROPDOWN style */ if ((cs->style & CBS_DROPDOWNLIST) == CBS_DROPDOWN) { - infoPtr->hwndEdit = CreateWindowExW (0, EDIT, NIL, + infoPtr->hwndEdit = CreateWindowExW (0, WC_EDITW, NIL, WS_CHILD | WS_VISIBLE | WS_CLIPSIBLINGS | ES_AUTOHSCROLL, 0, 0, 0, 0, /* will set later */ infoPtr->hwndCombo, diff --git a/dlls/comctl32/ipaddress.c b/dlls/comctl32/ipaddress.c index 42ac703..03b8d1c 100644 --- a/dlls/comctl32/ipaddress.c +++ b/dlls/comctl32/ipaddress.c @@ -189,7 +189,6 @@ static LRESULT IPADDRESS_Draw (const IPADDRESS_INFO *infoPtr, HDC hdc) static LRESULT IPADDRESS_Create (HWND hwnd, const CREATESTRUCTA *lpCreate) { - static const WCHAR EDIT[] = { 'E', 'd', 'i', 't', 0 }; IPADDRESS_INFO *infoPtr; RECT rcClient, edit; int i, fieldsize; @@ -230,7 +229,7 @@ static LRESULT IPADDRESS_Create (HWND hwnd, const CREATESTRUCTA *lpCreate) edit.left = rcClient.left + i*fieldsize + 6; edit.right = rcClient.left + (i+1)*fieldsize - 2; part->EditHwnd = - CreateWindowW (EDIT, NULL, WS_CHILD | WS_VISIBLE | ES_CENTER, + CreateWindowW (WC_EDITW, NULL, WS_CHILD | WS_VISIBLE | ES_CENTER, edit.left, edit.top, edit.right - edit.left, edit.bottom - edit.top, hwnd, (HMENU) 1, (HINSTANCE)GetWindowLongPtrW(hwnd, GWLP_HINSTANCE), NULL); diff --git a/dlls/comctl32/listview.c b/dlls/comctl32/listview.c index e1e8f96..6e73b55 100644 --- a/dlls/comctl32/listview.c +++ b/dlls/comctl32/listview.c @@ -11309,7 +11309,6 @@ static LRESULT CALLBACK EditLblWndProcA(HWND hwnd, UINT uMsg, WPARAM wParam, LPA */ static HWND CreateEditLabelT(LISTVIEW_INFO *infoPtr, LPCWSTR text, DWORD style, BOOL isW) { - WCHAR editName[5] = { 'E', 'd', 'i', 't', '\0' }; HWND hedit; HINSTANCE hinst = (HINSTANCE)GetWindowLongPtrW(infoPtr->hwndSelf, GWLP_HINSTANCE); @@ -11319,9 +11318,9 @@ static HWND CreateEditLabelT(LISTVIEW_INFO *infoPtr, LPCWSTR text, DWORD style, /* Window will be resized and positioned after LVN_BEGINLABELEDIT */ if (isW) - hedit = CreateWindowW(editName, text, style, 0, 0, 0, 0, infoPtr->hwndSelf, 0, hinst, 0); + hedit = CreateWindowW(WC_EDITW, text, style, 0, 0, 0, 0, infoPtr->hwndSelf, 0, hinst, 0); else - hedit = CreateWindowA("Edit", (LPCSTR)text, style, 0, 0, 0, 0, infoPtr->hwndSelf, 0, hinst, 0); + hedit = CreateWindowA(WC_EDITA, (LPCSTR)text, style, 0, 0, 0, 0, infoPtr->hwndSelf, 0, hinst, 0); if (!hedit) return 0; diff --git a/dlls/comctl32/monthcal.c b/dlls/comctl32/monthcal.c index 2450a5c..6394600 100644 --- a/dlls/comctl32/monthcal.c +++ b/dlls/comctl32/monthcal.c @@ -1681,10 +1681,8 @@ MONTHCAL_RButtonUp(MONTHCAL_INFO *infoPtr, LPARAM lParam) /* creates updown control and edit box */ static void MONTHCAL_EditYear(MONTHCAL_INFO *infoPtr) { - static const WCHAR EditW[] = { 'E','D','I','T',0 }; - infoPtr->hWndYearEdit = - CreateWindowExW(0, EditW, 0, WS_VISIBLE | WS_CHILD | ES_READONLY, + CreateWindowExW(0, WC_EDITW, 0, WS_VISIBLE | WS_CHILD | ES_READONLY, infoPtr->titleyear.left + 3, infoPtr->titlebtnnext.top, infoPtr->titleyear.right - infoPtr->titleyear.left + 4, infoPtr->textHeight, infoPtr->hwndSelf, diff --git a/dlls/comctl32/treeview.c b/dlls/comctl32/treeview.c index a38434d..29db19a 100644 --- a/dlls/comctl32/treeview.c +++ b/dlls/comctl32/treeview.c @@ -3674,7 +3674,6 @@ TREEVIEW_EditLabel(TREEVIEW_INFO *infoPtr, HTREEITEM hItem) HDC hdc; HFONT hOldFont=0; TEXTMETRICW textMetric; - static const WCHAR EditW[] = {'E','d','i','t',0}; TRACE("%p %p\n", hwnd, hItem); if (!TREEVIEW_ValidItem(infoPtr, hItem)) @@ -3721,7 +3720,7 @@ TREEVIEW_EditLabel(TREEVIEW_INFO *infoPtr, HTREEITEM hItem) infoPtr->editItem = hItem; hwndEdit = CreateWindowExW(WS_EX_LEFT, - EditW, + WC_EDITW, 0, WS_CHILD | WS_BORDER | ES_AUTOHSCROLL | WS_CLIPSIBLINGS | ES_WANTRETURN | diff --git a/dlls/comctl32/updown.c b/dlls/comctl32/updown.c index bb78903..a212073 100644 --- a/dlls/comctl32/updown.c +++ b/dlls/comctl32/updown.c @@ -557,8 +557,6 @@ UPDOWN_Buddy_SubclassProc(HWND hwnd, UINT uMsg, WPARAM wParam, LPARAM lParam) */ static HWND UPDOWN_SetBuddy (UPDOWN_INFO* infoPtr, HWND bud) { - static const WCHAR editW[] = { 'E', 'd', 'i', 't', 0 }; - static const WCHAR listboxW[] = { 'L', 'i', 's', 't', 'b', 'o', 'x', 0 }; RECT budRect; /* new coord for the buddy */ int x, width; /* new x position and width for the up-down */ WNDPROC baseWndProc; @@ -586,9 +584,9 @@ static HWND UPDOWN_SetBuddy (UPDOWN_INFO* infoPtr, HWND bud) /* Store buddy window class type */ infoPtr->BuddyType = BUDDY_TYPE_UNKNOWN; if (GetClassNameW(bud, buddyClass, COUNT_OF(buddyClass))) { - if (lstrcmpiW(buddyClass, editW) == 0) + if (lstrcmpiW(buddyClass, WC_EDITW) == 0) infoPtr->BuddyType = BUDDY_TYPE_EDIT; - else if (lstrcmpiW(buddyClass, listboxW) == 0) + else if (lstrcmpiW(buddyClass, WC_LISTBOXW) == 0) infoPtr->BuddyType = BUDDY_TYPE_LISTBOX; }

1 0

Nikolay Sivov : comctl32/monthcal: Implement MCM_GETMONTHRANGE for GMR_DAYSTATE flag and a single calendar control .
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: c68324dfedf2f1e25e984f3d87c9419db90ce1f5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c68324dfedf2f1e25e984f3d8… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Thu Oct 8 14:02:18 2009 +0400 comctl32/monthcal: Implement MCM_GETMONTHRANGE for GMR_DAYSTATE flag and a single calendar control. --- dlls/comctl32/monthcal.c | 73 ++++++++++++++++++++++++++++++++------- dlls/comctl32/tests/monthcal.c | 17 +++++---- 2 files changed, 70 insertions(+), 20 deletions(-) diff --git a/dlls/comctl32/monthcal.c b/dlls/comctl32/monthcal.c index 6394600..a44ae31 100644 --- a/dlls/comctl32/monthcal.c +++ b/dlls/comctl32/monthcal.c @@ -395,7 +395,7 @@ int MONTHCAL_CalculateDayOfWeek(WORD day, WORD month, WORD year) } /* properly updates date to point on next month */ -void inline MONTHCAL_GetNextMonth(SYSTEMTIME *date) +static inline void MONTHCAL_GetNextMonth(SYSTEMTIME *date) { if(++date->wMonth > 12) { @@ -407,7 +407,7 @@ void inline MONTHCAL_GetNextMonth(SYSTEMTIME *date) } /* properly updates date to point on prev month */ -void inline MONTHCAL_GetPrevMonth(SYSTEMTIME *date) +static inline void MONTHCAL_GetPrevMonth(SYSTEMTIME *date) { if(--date->wMonth < 1) { @@ -418,6 +418,45 @@ void inline MONTHCAL_GetPrevMonth(SYSTEMTIME *date) date->wYear); } +/* Returns full date for a first currently visible day */ +static void MONTHCAL_GetMinDate(const MONTHCAL_INFO *infoPtr, SYSTEMTIME *date) +{ + int firstDay; + + firstDay = MONTHCAL_CalculateDayOfWeek(1, infoPtr->curSel.wMonth, infoPtr->curSel.wYear); + + *date = infoPtr->curSel; + MONTHCAL_GetPrevMonth(date); + + date->wDay = MONTHCAL_MonthLength(date->wMonth, date->wYear) + + (infoPtr->firstDay - firstDay) % 7 + 1; + + if(date->wDay > MONTHCAL_MonthLength(date->wMonth, date->wYear)) + date->wDay -= 7; + + /* fix day of week */ + date->wDayOfWeek = MONTHCAL_CalculateDayOfWeek(date->wDay, date->wMonth, + date->wYear); +} + +/* Returns full date for a last currently visible day */ +static void MONTHCAL_GetMaxDate(const MONTHCAL_INFO *infoPtr, SYSTEMTIME *date) +{ + SYSTEMTIME st; + + *date = infoPtr->curSel; + MONTHCAL_GetNextMonth(date); + + MONTHCAL_GetMinDate(infoPtr, &st); + /* Use month length to get max day. 42 means max day count in calendar area */ + date->wDay = 42 - (MONTHCAL_MonthLength(st.wMonth, st.wYear) - st.wDay + 1) - + MONTHCAL_MonthLength(infoPtr->curSel.wMonth, infoPtr->curSel.wYear); + + /* fix day of week */ + date->wDayOfWeek = MONTHCAL_CalculateDayOfWeek(date->wDay, date->wMonth, + date->wYear); +} + /* From a given point, calculate the row (weekpos), column(daypos) and day in the calendar. day== 0 mean the last day of tha last month */ @@ -661,7 +700,7 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT RECT *titlemonth=&infoPtr->titlemonth; RECT *titleyear=&infoPtr->titleyear; RECT dayrect; - int i, j, m, mask, day, firstDay, prevMonth; + int i, j, m, mask, day, prevMonth; int textHeight = infoPtr->textHeight; SIZE size; HBRUSH hbr; @@ -673,6 +712,7 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT RECT rcTemp; RECT rcDay; /* used in MONTHCAL_CalcDayRect() */ int startofprescal; + SYSTEMTIME st; oldTextColor = SetTextColor(hdc, comctl32_color.clrWindowText); @@ -764,13 +804,8 @@ static void MONTHCAL_Refresh(MONTHCAL_INFO *infoPtr, HDC hdc, const PAINTSTRUCT } /* draw day numbers; first, the previous month */ - firstDay = MONTHCAL_CalculateDayOfWeek(1, infoPtr->curSel.wMonth, infoPtr->curSel.wYear); - - day = MONTHCAL_MonthLength(prevMonth, infoPtr->curSel.wYear) + - (infoPtr->firstDay - firstDay)%7 + 1; - - if (day > MONTHCAL_MonthLength(prevMonth, infoPtr->curSel.wYear)) - day -= 7; + MONTHCAL_GetMinDate(infoPtr, &st); + day = st.wDay; startofprescal = day; mask = 1<<(day-1); @@ -1181,7 +1216,8 @@ MONTHCAL_GetMonthRange(const MONTHCAL_INFO *infoPtr, DWORD flag, SYSTEMTIME *st) if(st) { - if(flag == GMR_VISIBLE) + switch (flag) { + case GMR_VISIBLE: { /*FIXME: currently multicalendar feature isn't implelented, so entirely visible month is current */ @@ -1193,9 +1229,20 @@ MONTHCAL_GetMonthRange(const MONTHCAL_INFO *infoPtr, DWORD flag, SYSTEMTIME *st) st[1].wDay = MONTHCAL_MonthLength(st[1].wMonth, st[1].wYear); st[1].wDayOfWeek = MONTHCAL_CalculateDayOfWeek(st[1].wDay, st[1].wMonth, st[1].wYear); + /* a single current month used */ + return 1; + } + case GMR_DAYSTATE: + { + /*FIXME: currently multicalendar feature isn't implelented, + min date from previous month and max date from next one returned */ + MONTHCAL_GetMinDate(infoPtr, &st[0]); + MONTHCAL_GetMaxDate(infoPtr, &st[1]); + break; + } + default: + WARN("Unknown flag value, got %d\n", flag); } - else - FIXME("only GMR_VISIBLE flag supported, got %d\n", flag); } return infoPtr->monthRange; diff --git a/dlls/comctl32/tests/monthcal.c b/dlls/comctl32/tests/monthcal.c index 57142a6..d3f6681 100644 --- a/dlls/comctl32/tests/monthcal.c +++ b/dlls/comctl32/tests/monthcal.c @@ -1374,10 +1374,10 @@ static void test_monthcal_monthrange(void) todo_wine { expect(2, res); } - expect(2000, st_visible[0].wYear); - expect(11, st_visible[0].wMonth); - expect(1, st_visible[0].wDay); - expect(2000, st_visible[1].wYear); + expect(2000, st_visible[0].wYear); + expect(11, st_visible[0].wMonth); + expect(1, st_visible[0].wDay); + expect(2000, st_visible[1].wYear); todo_wine { expect(12, st_visible[1].wMonth); @@ -1386,9 +1386,12 @@ static void test_monthcal_monthrange(void) res = SendMessage(hwnd, MCM_GETMONTHRANGE, GMR_DAYSTATE, (LPARAM)st_daystate); todo_wine { expect(4, res); - expect(2000, st_daystate[0].wYear); - expect(10, st_daystate[0].wMonth); - expect(29, st_daystate[0].wDay); + } + expect(2000, st_daystate[0].wYear); + expect(10, st_daystate[0].wMonth); + expect(29, st_daystate[0].wDay); + + todo_wine { expect(2001, st_daystate[1].wYear); expect(1, st_daystate[1].wMonth); expect(6, st_daystate[1].wDay);

1 0

Alexandre Julliard : kernel32: Move Get/SetProcessDword to kernel16.c.
by Alexandre Julliard 08 Oct '09

08 Oct '09

Module: wine Branch: master Commit: e0ce6dc32eb81660b19bfd6dfd739fb296b38159 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e0ce6dc32eb81660b19bfd6df… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Oct 7 20:49:22 2009 +0200 kernel32: Move Get/SetProcessDword to kernel16.c. --- dlls/kernel32/kernel16.c | 115 ++++++++++++++++++++++++++++++++++++++++ dlls/kernel32/krnl386.exe.spec | 4 +- dlls/kernel32/process.c | 107 +------------------------------------ 3 files changed, 119 insertions(+), 107 deletions(-) diff --git a/dlls/kernel32/kernel16.c b/dlls/kernel32/kernel16.c index e298aa3..3acb932 100644 --- a/dlls/kernel32/kernel16.c +++ b/dlls/kernel32/kernel16.c @@ -25,6 +25,7 @@ #include "winbase.h" #include "winternl.h" #include "wownt32.h" +#include "wine/winuser16.h" #include "kernel_private.h" #include "kernel16_private.h" @@ -32,6 +33,10 @@ WINE_DEFAULT_DEBUG_CHANNEL(module); +extern DWORD WINAPI GetProcessFlags( DWORD processid ); + +static DWORD process_dword; + /************************************************************************** * DllEntryPoint (KERNEL.669) */ @@ -580,6 +585,116 @@ DWORD WINAPI MapProcessHandle( HANDLE hProcess ) } /*********************************************************************** + * SetProcessDword (KERNEL.484) + * 'Of course you cannot directly access Windows internal structures' + */ +void WINAPI SetProcessDword16( DWORD dwProcessID, INT offset, DWORD value ) +{ + TRACE("(%d, %d)\n", dwProcessID, offset ); + + if (dwProcessID && dwProcessID != GetCurrentProcessId()) + { + ERR("%d: process %x not accessible\n", offset, dwProcessID); + return; + } + + switch ( offset ) + { + case GPD_APP_COMPAT_FLAGS: + case GPD_LOAD_DONE_EVENT: + case GPD_HINSTANCE16: + case GPD_WINDOWS_VERSION: + case GPD_THDB: + case GPD_PDB: + case GPD_STARTF_SHELLDATA: + case GPD_STARTF_HOTKEY: + case GPD_STARTF_SHOWWINDOW: + case GPD_STARTF_SIZE: + case GPD_STARTF_POSITION: + case GPD_STARTF_FLAGS: + case GPD_PARENT: + case GPD_FLAGS: + ERR("Not allowed to modify offset %d\n", offset ); + break; + case GPD_USERDATA: + process_dword = value; + break; + default: + ERR("Unknown offset %d\n", offset ); + break; + } +} + +/*********************************************************************** + * GetProcessDword (KERNEL.485) + * 'Of course you cannot directly access Windows internal structures' + */ +DWORD WINAPI GetProcessDword16( DWORD dwProcessID, INT offset ) +{ + DWORD x, y; + STARTUPINFOW siw; + + TRACE("(%d, %d)\n", dwProcessID, offset ); + + if (dwProcessID && dwProcessID != GetCurrentProcessId()) + { + ERR("%d: process %x not accessible\n", offset, dwProcessID); + return 0; + } + + switch ( offset ) + { + case GPD_APP_COMPAT_FLAGS: + return GetAppCompatFlags16(0); + case GPD_LOAD_DONE_EVENT: + return 0; + case GPD_HINSTANCE16: + return GetTaskDS16(); + case GPD_WINDOWS_VERSION: + return GetExeVersion16(); + case GPD_THDB: + return (DWORD_PTR)NtCurrentTeb() - 0x10 /* FIXME */; + case GPD_PDB: + return (DWORD_PTR)NtCurrentTeb()->Peb; /* FIXME: truncating a pointer */ + case GPD_STARTF_SHELLDATA: /* return stdoutput handle from startupinfo ??? */ + GetStartupInfoW(&siw); + return HandleToULong(siw.hStdOutput); + case GPD_STARTF_HOTKEY: /* return stdinput handle from startupinfo ??? */ + GetStartupInfoW(&siw); + return HandleToULong(siw.hStdInput); + case GPD_STARTF_SHOWWINDOW: + GetStartupInfoW(&siw); + return siw.wShowWindow; + case GPD_STARTF_SIZE: + GetStartupInfoW(&siw); + x = siw.dwXSize; + if ( (INT)x == CW_USEDEFAULT ) x = CW_USEDEFAULT16; + y = siw.dwYSize; + if ( (INT)y == CW_USEDEFAULT ) y = CW_USEDEFAULT16; + return MAKELONG( x, y ); + case GPD_STARTF_POSITION: + GetStartupInfoW(&siw); + x = siw.dwX; + if ( (INT)x == CW_USEDEFAULT ) x = CW_USEDEFAULT16; + y = siw.dwY; + if ( (INT)y == CW_USEDEFAULT ) y = CW_USEDEFAULT16; + return MAKELONG( x, y ); + case GPD_STARTF_FLAGS: + GetStartupInfoW(&siw); + return siw.dwFlags; + case GPD_PARENT: + return 0; + case GPD_FLAGS: + return GetProcessFlags(0); + case GPD_USERDATA: + return process_dword; + default: + ERR("Unknown offset %d\n", offset ); + return 0; + } +} + +/*********************************************************************** * FreeLibrary32 (KERNEL.486) */ BOOL WINAPI FreeLibrary32_16( HINSTANCE module ) diff --git a/dlls/kernel32/krnl386.exe.spec b/dlls/kernel32/krnl386.exe.spec index b43c874..60560c5 100644 --- a/dlls/kernel32/krnl386.exe.spec +++ b/dlls/kernel32/krnl386.exe.spec @@ -386,8 +386,8 @@ 481 pascal -ret16 _LeaveWin16Lock() _LeaveWin16Lock 482 pascal LoadSystemLibrary32(str) LoadLibrary32_16 # FIXME! 483 pascal MapProcessHandle(long) MapProcessHandle -484 pascal SetProcessDword(long s_word long) SetProcessDword -485 pascal GetProcessDword(long s_word) GetProcessDword +484 pascal SetProcessDword(long s_word long) SetProcessDword16 +485 pascal GetProcessDword(long s_word) GetProcessDword16 486 pascal FreeLibrary32(long) FreeLibrary32_16 487 pascal GetModuleFileName32(long str word) GetModuleFileName32_16 488 pascal GetModuleHandle32(str) GetModuleHandle32_16 diff --git a/dlls/kernel32/process.c b/dlls/kernel32/process.c index 4d8f2f7..df09558 100644 --- a/dlls/kernel32/process.c +++ b/dlls/kernel32/process.c @@ -74,7 +74,6 @@ static UINT process_error_mode; static DWORD shutdown_flags = 0; static DWORD shutdown_priority = 0x280; -static DWORD process_dword; static BOOL is_wow64; HMODULE kernel32_handle = 0; @@ -2537,114 +2536,12 @@ DWORD WINAPI GetProcessFlags( DWORD processid ) /*********************************************************************** - * GetProcessDword (KERNEL.485) * GetProcessDword (KERNEL32.18) - * 'Of course you cannot directly access Windows internal structures' */ DWORD WINAPI GetProcessDword( DWORD dwProcessID, INT offset ) { - DWORD x, y; - STARTUPINFOW siw; - - TRACE("(%d, %d)\n", dwProcessID, offset ); - - if (dwProcessID && dwProcessID != GetCurrentProcessId()) - { - ERR("%d: process %x not accessible\n", offset, dwProcessID); - return 0; - } - - switch ( offset ) - { - case GPD_APP_COMPAT_FLAGS: - return GetAppCompatFlags16(0); - case GPD_LOAD_DONE_EVENT: - return 0; - case GPD_HINSTANCE16: - return GetTaskDS16(); - case GPD_WINDOWS_VERSION: - return GetExeVersion16(); - case GPD_THDB: - return (DWORD_PTR)NtCurrentTeb() - 0x10 /* FIXME */; - case GPD_PDB: - return (DWORD_PTR)NtCurrentTeb()->Peb; /* FIXME: truncating a pointer */ - case GPD_STARTF_SHELLDATA: /* return stdoutput handle from startupinfo ??? */ - GetStartupInfoW(&siw); - return HandleToULong(siw.hStdOutput); - case GPD_STARTF_HOTKEY: /* return stdinput handle from startupinfo ??? */ - GetStartupInfoW(&siw); - return HandleToULong(siw.hStdInput); - case GPD_STARTF_SHOWWINDOW: - GetStartupInfoW(&siw); - return siw.wShowWindow; - case GPD_STARTF_SIZE: - GetStartupInfoW(&siw); - x = siw.dwXSize; - if ( (INT)x == CW_USEDEFAULT ) x = CW_USEDEFAULT16; - y = siw.dwYSize; - if ( (INT)y == CW_USEDEFAULT ) y = CW_USEDEFAULT16; - return MAKELONG( x, y ); - case GPD_STARTF_POSITION: - GetStartupInfoW(&siw); - x = siw.dwX; - if ( (INT)x == CW_USEDEFAULT ) x = CW_USEDEFAULT16; - y = siw.dwY; - if ( (INT)y == CW_USEDEFAULT ) y = CW_USEDEFAULT16; - return MAKELONG( x, y ); - case GPD_STARTF_FLAGS: - GetStartupInfoW(&siw); - return siw.dwFlags; - case GPD_PARENT: - return 0; - case GPD_FLAGS: - return GetProcessFlags(0); - case GPD_USERDATA: - return process_dword; - default: - ERR("Unknown offset %d\n", offset ); - return 0; - } -} - -/*********************************************************************** - * SetProcessDword (KERNEL.484) - * 'Of course you cannot directly access Windows internal structures' - */ -void WINAPI SetProcessDword( DWORD dwProcessID, INT offset, DWORD value ) -{ - TRACE("(%d, %d)\n", dwProcessID, offset ); - - if (dwProcessID && dwProcessID != GetCurrentProcessId()) - { - ERR("%d: process %x not accessible\n", offset, dwProcessID); - return; - } - - switch ( offset ) - { - case GPD_APP_COMPAT_FLAGS: - case GPD_LOAD_DONE_EVENT: - case GPD_HINSTANCE16: - case GPD_WINDOWS_VERSION: - case GPD_THDB: - case GPD_PDB: - case GPD_STARTF_SHELLDATA: - case GPD_STARTF_HOTKEY: - case GPD_STARTF_SHOWWINDOW: - case GPD_STARTF_SIZE: - case GPD_STARTF_POSITION: - case GPD_STARTF_FLAGS: - case GPD_PARENT: - case GPD_FLAGS: - ERR("Not allowed to modify offset %d\n", offset ); - break; - case GPD_USERDATA: - process_dword = value; - break; - default: - ERR("Unknown offset %d\n", offset ); - break; - } + FIXME( "(%d, %d): not supported\n", dwProcessID, offset ); + return 0; }

1 0