October 2010 - Wine-Devel - list.winehq.org

Re: (6th try) winhlp32: Added Hebrew translation
by Paul Vriens Oct. 5, 2010

Oct. 5, 2010

Hi Yaron, I'm see this one is still not applied. Maybe because there are still some small issues? See below. On 10/03/2010 11:12 PM, Yaron Shahrabani wrote: > +LANGUAGE LANG_HEBREW, SUBLANG_DEFAULT This line should go, it's redundant (it's already stated at the beginning). This line was present in the English resources because there is a difference between SUBLANG_DEFAULT and SUBLANG_NEUTRAL in that resource file. For example 'colors' vs. 'colours'. > + > +IDD_INDEX DIALOG 0, 0, 200, 190 > +STYLE DS_MODALFRAME | WS_CAPTION | WS_SYSMENU | WS_EX_LAYOUTRTL You should move the WS_EX_LAYOUTRTL to an EXSTYLE like the IDD_SEARCH_DIALOG below. Adding it to the STYLE has no effect. > +FONT 8, "MS Shell Dlg" > +CAPTION "מפתח" > +{ > + LISTBOX IDC_INDEXLIST, 10, 10, 180, 150, LBS_NOINTEGRALHEIGHT | WS_VSCROLL | WS_BORDER > +} > + > +IDD_SEARCH DIALOG 0, 0, 200, 190 > +STYLE DS_MODALFRAME | WS_CAPTION | WS_SYSMENU > +EXSTYLE WS_EX_LAYOUTRTL > +FONT 8, "MS Shell Dlg" -- Cheers, Paul.

1 0

Re: user32/tests: Test FindWindowEx for empty window title (try 2)
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5829 Your paranoid android. === W98SE (32 bit win) === win.c:2970: Test failed: message 000f available win.c:2982: Test failed: message 000f available win.c:2991: Test failed: message 000f available win.c:3000: Test failed: message 000f available === W7PROX64 (64 bit win) === win.c:2684: Test failed: GetActiveWindow() = 0000000000000000 win.c:2684: Test failed: GetFocus() = 0000000000000000 win.c:2710: Test failed: GetActiveWindow() = 0000000000000000 win.c:2710: Test failed: GetFocus() = 0000000000000000 win.c:2769: Test failed: GetActiveWindow() = 0000000000000000 win.c:2769: Test failed: GetFocus() = 0000000000000000

1 0

Re: [PATCH] shlwapi: Handle URL_WININET_COMPATIBLITY flag in UrlCanonicalize
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5828 Your paranoid android. === W98SE (32 bit url) === url.c:746: Test failed: UrlCanonicalizeA dwFlags 0x00000000 url 'c:dir\file' Expected "file:///c:dir/file", but got "file:///c:/dir/file", index 46

1 0

winmm/tests: Don't test the non-default Midi devices if the 'PreferredOnly' Windows setting is set.
by Joerg-Cyril.Hoehle＠t-systems.com Oct. 5, 2010

Oct. 5, 2010

Hi, a few month ago when the discussion about "PreferredOnly" registry setting came up, I queried Google and found that many more users use this setting than one may believe - i.e. it is not exotic. Therefore, I don't think it's that great an idea to reduce the number of devices that the tests stress using that method: we'll get less feedback from a number of real machines. OTOH, I have no solution to the vmware MIDI trouble on Francois' machines. Regards, Jörg Höhle

1 0

Re: msi: Put the uninstall key for 32-bit packages under Wow6432Node on 64-bit.
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5824 Your paranoid android. === W7PROX64 (64 bit install) === install.c:10427: Test failed: key not created

2 1

Re: [2/3] kernel32/process: Implement GetLogicalProcessorInformation using NtQuerySystemInformation
by Vitaliy Margolen Oct. 5, 2010

Oct. 5, 2010

On 10/05/2010 04:46 AM, Rudolf Mayerhofer wrote: > --- > dlls/kernel32/process.c | 21 ++++++++++++++++++--- > 1 files changed, 18 insertions(+), 3 deletions(-) > Please exchange patches 2 and 3. And everywhere in your patches, please stick to _THE SAME_ formatting as the rest of the file you editing. Examples: > + if (status != STATUS_SUCCESS) > + { > + if( status == STATUS_INFO_LENGTH_MISMATCH ) Here you adding 2 differently formatted if () statements. > +static ULONG_PTR sysfs_cpu_topology_getmap(int cpu, const char *type) > +{ > + char buf[256]; > + FILE *fd; > + ULONG_PTR result; > + sprintf(buf, "/sys/devices/system/cpu/cpu%d/topology/%s",cpu,type); Please add empty line between variable declarations and the rest of the code. All commas should always be followed by a space. > + fd = fopen(buf,"r"); > + if(fscanf(fd,"%lx",&result) != 1) You are not checking if fopen() succeeded. > +static int sysfs_cpucache_getint(int cpu, int cache, const char *type) > +{ > + }else{ > + val = -1; > + } > + fclose(fd); > + return val; > +} > ... > + BYTE level = sysfs_cpucache_getint(cpucount,cachecount,"level"); Don't mix signed and unsigned integers. Especially that you return -1 in case of error. In that entire file curly braces go on a separate line. Vitaliy.

1 0

[1/2] user32: fix FindWindowEx behavior if window name is an empty charachter
by Louis Lenders Oct. 5, 2010

Oct. 5, 2010

>It would be better to avoid duplicate GetWindowTextW() calls. I thought a long time how to fix this, but i couldn't find another way than this one, because we need to find out whether or not the window title is empty, and afaik we can only do that by calling GetWindowText (again). If you have another way that i overlooked, please let me know Regards Louis

3 2

Re: winex11: Add support for animated cursors in X11 driver (try 5).
by Alexandre Julliard Oct. 5, 2010

Oct. 5, 2010

Erich Hoover <ehoover(a)mines.edu> writes: > + imgs[nFrames++] = image; > + if (!(imgstmp = HeapReAlloc( GetProcessHeap(), 0, imgs, nFrames*sizeof(XcursorImage*) ))) goto cleanup; > + imgs = imgstmp; This is still broken. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: ImmReleaseContext() function implementation
by Aric Stewart Oct. 5, 2010

Oct. 5, 2010

I do not believe this is correct. You would be destroying the context every time it is released and that is clearly not what we want to be doing ass ImmGetContext and ImmReleaseContext as a pair are called over and over again in a typical application. -aric On 10/4/10 1:25 PM, Mikhail Gorbunov wrote: > This patch implements (with a basic sanity check) a behaviour of > ImmReleaseContext() > function as described in msdn and in the way ImmGetContext() has already been > impleneted. > > From 56f78bcd9584350b456ca0b0514a357b7ee8cdfd Mon Sep 17 00:00:00 2001 > From: Mikhail Gorbunov<mikhail.gorbunov(a)gmail.com> > Date: Mon, 4 Oct 2010 21:47:17 +0400 > Subject: [PATCH] ImmReleaseContext() function implementation > > --- > dlls/imm32/imm.c | 24 ++++++++++++++++++------ > 1 files changed, 18 insertions(+), 6 deletions(-) > > diff --git a/dlls/imm32/imm.c b/dlls/imm32/imm.c > index 28f482c..87b4c14 100644 > --- a/dlls/imm32/imm.c > +++ b/dlls/imm32/imm.c > @@ -2009,13 +2009,25 @@ BOOL WINAPI ImmRegisterWordW( > */ > BOOL WINAPI ImmReleaseContext(HWND hWnd, HIMC hIMC) > { > - static int shown = 0; > + TRACE("(%p, %p)\n", hWnd, hIMC); > > - if (!shown) { > - FIXME("(%p, %p): stub\n", hWnd, hIMC); > - shown = 1; > - } > - return TRUE; > + if (hIMC) > + { > + InputContextData *data = hIMC; > + > + if (data->IMC.hWnd != hWnd) > + { > + FIXME("(%p, %p): data->IMC.hWnd = %p :: err\n", > + hWnd, hIMC, data->IMC.hWnd); > + return TRUE; > + } > + else > + { > + return ImmDestroyContext(hIMC); > + } > + } > + else > + return TRUE; > } > > /***********************************************************************

1 0

Re: [PATCH 1/6] urlmon: Report error on synchronous bindings that don't have a cache file and are not read in callback.
by Alexandre Julliard Oct. 5, 2010

Oct. 5, 2010

Jacek Caban <jacek(a)codeweavers.com> writes: > --- > dlls/urlmon/binding.c | 17 ++++++++++++----- > 1 files changed, 12 insertions(+), 5 deletions(-) This breaks the gameux tests: ../../../tools/runtest -q -P wine -M gameux.dll -T ../../.. -p gameux_test.exe.so gameexplorer.c && touch gameexplorer.ok gameexplorer.c:571: Test failed: IGameExplorer::AddGame failed (error 0x80004005) gameexplorer.c:589: Test failed: IGameExplorer::AddGame failed (error 0x80004005) gameexplorer.c:640: Test failed: IGameExplorer2::InstallGame failed (error 0x80004005) gameexplorer.c:643: Test failed: cannot find game with GDF path L"Z:\\home\\julliard\\wine\\wine\\dlls\\gameux\\tests\\gameux_test.exe" in scope 2 make: *** [gameexplorer.ok] Error 4 -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: kernel32/process: Implement GetLogicalProcessorInformation (rewrite - try 3)
by Rudolf Mayerhofer Oct. 5, 2010

Oct. 5, 2010

>I don't know much about kernel32/ntdll, but I'm pretty sure that you >could get this to work through ntdll. NtQuerySystemInformation with >SystemProcessorInformation/SystemLogicalProcessorInformation (see >http://undocumented.ntinternals.net/UserMode/Undocumented%20Functions/Syste…) >should help. http://native-nt-toolkit.googlecode.com/svn/trunk/ndk/extypes.h >(read, don't copy) seems to have a pretty exhaustive set of >definitions, but even that's not complete. Some tests for the NT >version would be useful. > >Mike. SystemLogicalProcessorInformation seems to be the right SystemClass for NtQuerySystemInformation. But here's the Problem. I don't have any idea how i could any information about the struct returned by calling NtQuerySystemInformation with SystemLogicalProcessorInformation (= 73). The only windows machine i have access to a the moment keeps throwing status 0xC0000005 (ACCESS_VIOLATION) at me which doesn't help at all and i'm pretty much stuck right now. If i could get my hands on the struct returned by NtQuerySystemInformation it should be easy to change the patch to do the parsing in ntdll and just call that from kernel32.

2 2

Re: [1/2] user32: fix FindWindowEx behavior if window name is an empty charachter
by Dmitry Timoshkov Oct. 5, 2010

Oct. 5, 2010

Louis Lenders <xerox_xerox2000(a)yahoo.co.uk> wrote: > @@ -1756,6 +1756,10 @@ HWND WINAPI FindWindowExW( HWND parent, HWND child, LPCWSTR className, LPCWSTR t > { > while (list[i]) > { > + /* Handle special case where empty title bar or no title bar exists, so GetWindowsText would return NULL > + but if title is also an empty char we still have a match */ > + if (!GetWindowTextW( list[i], buffer, len + 1 ) && !title[0]) break; > + > if (GetWindowTextW( list[i], buffer, len + 1 ) && !strcmpiW( buffer, title )) break; > i++; > } It would be better to avoid duplicate GetWindowTextW() calls. -- Dmitry.

1 0

Re: psapi/tests: macro cleanup
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5813 Your paranoid android. === WVISTAADM (32 bit psapi_main) === psapi_main.c:111: Test failed: cbNeeded=100 === W2K8SE (32 bit psapi_main) === psapi_main.c:111: Test failed: cbNeeded=100 === W7PRO (32 bit psapi_main) === psapi_main.c:111: Test failed: cbNeeded=96 === W7PROX64 (32 bit psapi_main) === psapi_main.c:111: Test failed: cbNeeded=108

1 0

Re: [PATCH] comctl32/tests: Add tests for negative subitems independent of local settings
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5811 Your paranoid android. === W7PROX64 (64 bit listview) === listview.c:4449: Test failed: NM_HOVER allow test: the msg sequence is not complete: expected 0087 - actual 0000 listview.c:4454: Test failed: NM_HOVER block test: the msg sequence is not complete: expected 0087 - actual 0000

1 0

Re: [2/2] user32/tests: Test FindWindowEx for empty window title
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5803 Your paranoid android. === W7PROX64 (32 bit win) === win.c:2684: Test failed: GetActiveWindow() = 00000000 win.c:2684: Test failed: GetFocus() = 00000000 win.c:2710: Test failed: GetActiveWindow() = 00000000 win.c:2710: Test failed: GetFocus() = 00000000 win.c:2769: Test failed: GetActiveWindow() = 00000000 win.c:2769: Test failed: GetFocus() = 00000000 === W7PROX64 (64 bit win) === win.c:2684: Test failed: GetActiveWindow() = 0000000000000000 win.c:2684: Test failed: GetFocus() = 0000000000000000 win.c:2710: Test failed: GetActiveWindow() = 0000000000000000 win.c:2710: Test failed: GetFocus() = 0000000000000000 win.c:2769: Test failed: GetActiveWindow() = 0000000000000000 win.c:2769: Test failed: GetFocus() = 0000000000000000

1 0

Re: [PATCH 2/3] urlmon: Added Abort tests.
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5782 Your paranoid android. === W2K3R2SESP2 (32 bit protocol) === protocol.c:824: Test failed: hrResult = 800c0005, expected: 00000000 Timeout === WVISTAADM (32 bit protocol) === protocol.c:509: Test failed: unexpected call Switch

1 0

Re: [PATCH 3/3] urlmon: Added Abort support to BindProtocol.
by testbot＠testbot.winehq.org Oct. 5, 2010

Oct. 5, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5783 Your paranoid android. === WINEBUILD (build) === Patch failed

1 0

Re: include: Added d3d10_1.idl.
by Rico Schüller Oct. 4, 2010

Oct. 4, 2010

Am 02.10.2010 15:29, schrieb Jacek Caban: +typedef enum D3D10_SRV_DIMENSION1 +{ + D3D10_1_SRV_DIMENSION_UNKNOWN = 0, ... + D3D10_1_SRV_DIMENSION_TEXTURECUBE = 9, + D3D10_1_SRV_DIMENSION_TEXTURECUBEARRAY = 10 +} D3D10_SRV_DIMENSION1; You probably might add those to d3dcommon.h and only add a typedef. It was changed in one of the sdks. Cheers Rico

2 2

Re: winspool.drv: return maximum possible size of the buffer when no buffer specified
by Alexandre Julliard Oct. 4, 2010

Oct. 4, 2010

Vitaly Perov <vitperov(a)etersoft.ru> writes: > @@ -1398,7 +1398,7 @@ static void convert_printerinfo_W_to_A(LPBYTE out, LPBYTE pPrintersW, > dmA = DEVMODEdupWtoA(piW->pDevMode); > if (dmA) { > /* align DEVMODEA to a DWORD boundary */ > - len = (4 - ( (DWORD_PTR) ptr & 3)) & 3; > + len = ptr ? (4 - ( (DWORD_PTR) ptr & 3)) & 3 : 3; That doesn't make any sense. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: psapi/tests: remove an ugly macro
by testbot＠testbot.winehq.org Oct. 4, 2010

Oct. 4, 2010

Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=5776 Your paranoid android. === WVISTAADM (32 bit psapi_main) === psapi_main.c:116: Test failed: cbNeeded=100 === W2K8SE (32 bit psapi_main) === psapi_main.c:116: Test failed: cbNeeded=100 === W7PRO (32 bit psapi_main) === psapi_main.c:116: Test failed: cbNeeded=96 === W7PROX64 (32 bit psapi_main) === psapi_main.c:116: Test failed: cbNeeded=108

1 0

Re: cmd: Add a space at the end of the first echo'ed batch line [try2]
by Alexandre Julliard Oct. 4, 2010

Oct. 4, 2010

Alexandre Goujon <ale.goujon(a)gmail.com> writes: > -(a)pwd@>echo Tests for cmd's builtin commands(a)todo_space@ > +(a)pwd@>echo Tests for cmd's builtin commands You'll need to define a @space@ symbol, normal trailing spaces will get mangled. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: winex11: choose header exclusiv
by Alexandre Julliard Oct. 4, 2010

Oct. 4, 2010

André Hentschel <nerv(a)dawncrow.de> writes: > @@ -29,8 +29,7 @@ > > #ifdef HAVE_X11_EXTENSIONS_XF86VMODE_H > #include <X11/extensions/xf86vmode.h> > -#endif > -#ifdef HAVE_X11_EXTENSIONS_XF86VMPROTO_H > +#elif defined (HAVE_X11_EXTENSIONS_XF86VMPROTO_H) > #include <X11/extensions/xf86vmproto.h> > #endif This doesn't make sense, all it will most likely achieve is to disable the gamma support. -- Alexandre Julliard julliard(a)winehq.org

1 0

mono packaging, for wine packagers or anyone else who wants to follow along
by Vincent Povirk Oct. 4, 2010

Oct. 4, 2010

Buried in the 1.3.4 announcement is a tiny, insignificant little change that makes it possible for distributors to package Mono for Wine: mscoree: Search for Mono in some predefined paths before using the registry. As happy as I am to have sneaked this in under the radar, as it were, I guess I'm going to have to tell someone if it's ever going to be useful. I have no plans to make an official Wine Mono distribution. It should be easy enough for anyone to build. We currently do not require any changes in Mono for use with Wine. When we do (and it will be soon), I will try very hard to get the things we need merged upstream. Any managed libraries we need instead of or in addition to those shipped with Mono will have to go in Wine. We don't need a fork yet, and I hope we can keep it that way. Mono installs are generally self-contained and do not interfere with each other. Mono does not have to be installed into a bottle before it can be used. Therefore, I have not programmed any sort of install process into Wine. It will simply search for Mono as needed. If you are a distributor, here is how I recommend you distribute Mono for use with Wine: Download and extract the mono 2.6.7 source tarball from http://ftp.novell.com/pub/mono/sources/mono/mono-2.6.7.tar.bz2 Use the instructions at http://wiki.winehq.org/Mono to cross-compile Mono 2.6.7 using mingw32. Tweak the script if you have to (or just want to), but consider contributing any fixes you have to make back to winezeug. This will create a directory named mono-2.6.7. The contents of that directory should be installed to /usr/share/mono/mono-1.0. To make sure this worked, you should try a simple "Hello World" .NET application in Wine. Wikipedia has an example at https://secure.wikimedia.org/wikipedia/en/wiki/C_sharp_programming_language… that you should be able to save as a .cs file and compile to a .exe file using gmcs. If all is well, you should be able to run that exe file in a clean Wine prefix. In the future, we'll need a mono-2.0 directory as well, so we can keep using newer versions of Mono and get .NET 4.0 support. But we're not there yet. In case all these numbers have not confused you yet, the current version of Mono is 2.6.7. This version uses the Mono Embedding API version 1.0, which is why it needs to be placed in a directory named mono-1.0. It normally includes .NET 1.1 and 2.0 runtimes. It can optionally be compiled with a 4.0 runtime as well, but I don't recommend it. I'm expecting to see Mono 2.8 released "soon". That version will use the Embedding API version 2.0, which Wine can't support yet, but when the time comes it will need to be in a directory named mono-2.0. It will include .NET 2.0 and 4.0 runtimes. .NET 3.0 and 3.5 are not runtimes, which is why they were not included in that list. They use the .NET 2.0 runtime. While Mono 2.8 (and eventually 3.0) is the way forward for .NET 2.0 and 4.0 support, we'll need to also have the 2.6 series available so that we can have a .NET 1.1 runtime. Wine will detect which versions are available and what runtimes they have using a system that I'm working on and hope to have ready "soon", and it will select the correct runtime for whatever it's trying to run, or whatever the program selects using the .NET embedding API. (Also coming soon: the .NET embedding API.) I'm treating Mono as an optional dependency, much like the many native libraries Wine already uses. If it's not there, Wine will gracefully report failure to applications that try to use it, which probably won't work, and print some things to the console. I plan to send some changes soon that happen to cause a Wine without Mono to fail some tests, so I do consider an install that's missing Mono to be broken. But for the most part, the user experience will remain the same as it was before if Mono is missing.

2 1

Question on use of TextMetricsW
by James McKenzie Oct. 4, 2010

Oct. 4, 2010

All: I am running code like the following in a richedit control to set the font to Courier 12 point font: + static const WCHAR courier[] = {'C','o','u','r','i','e','r',0}; + testFont1 = CreateFontW (12,0,0,0,FW_LIGHT, 0, 0, 0, ANSI_CHARSET, + OUT_DEFAULT_PRECIS, CLIP_DEFAULT_PRECIS, DEFAULT_QUALITY, DEFAULT_PITCH | + FF_DONTCARE, courier); + + /*Set test font to be Courier font */ + SendMessageW(hwndRichEdit, WM_SETFONT, (WPARAM)testFont1, MAKELPARAM(TRUE, 0)); + /*Verify font is set */ + SendMessageW(hwndRichEdit, EM_GETCHARFORMAT, SCF_DEFAULT, (LPARAM) &CharFont1Unicode); + GetObjectW(testFont1, sizeof(LOGFONTW), &sentLogFont); + ok (!lstrcmpW(sentLogFont.lfFaceName, CharFont1Unicode.szFaceName), "WM_SETFONT set wrong font. " \ + "Font was set to %s or %s, was supposed to be \"Courier\".\n", wine_dbgstr_w(CharFont1Unicode.szFaceName), wine_dbgstr_w(sentLogFont.lfFaceName)); + + ret = GetTextMetricsW (hdc, &tmw); + ok (ret, "GetTextMetricsW failed\n"); + + ok (7 == tmw.tmAveCharWidth, "Average Character Width for Courier 12 is %d\n", tmw.tmAveCharWidth); I also set the font to MS Sans Serif, and two different font sizes for Arial. However, the tmAveCharWidth value remains at 7 and I am of the opinion that this should change for each font change and also for the font size change. Am I doing something improper? Should I clear the tmw structure between calls to GetTextMetricsW? When I run this on native WindowsXP (which I have access to) the average character width changes to 9, but remains the same for all four font types. TTIA. James McKenzie

2 3

Re: kernel32/process: Implement GetLogicalProcessorInformation (rewrite - try 3)
by Mike Kaplinskiy Oct. 3, 2010

Oct. 3, 2010

On Sun, Oct 3, 2010 at 10:25 AM, Rudolf Mayerhofer <rm(a)eightyfive.net> wrote: >>A rewrite of my previously sent patches in order to avoid changing >>SYSTEM_CPU_INFORMATION. > >>This patch supersedes my previously sent patches: >>[PATCH1/3] include/winternl: Add Logical Processor Information to >>SYSTEM_CPU_INFORMATION (try 3) >>[PATCH2/3] kernel32/process: Implement GetLogicalProcessorInformation() (try > 2) >>[PATCH3/3] ntdll: Retrieve Logical Processor Information from SysFS (try 4) > > Try 2 removes a FIXME which is not needed. > Try 3 changes a FIXME to a WARN when an unknown cache type is found > > > --- > dlls/kernel32/process.c | 272 > ++++++++++++++++++++++++++++++++++++++++++++++- > 1 files changed, 270 insertions(+), 2 deletions(-) > > > > > I don't know much about kernel32/ntdll, but I'm pretty sure that you could get this to work through ntdll. NtQuerySystemInformation with SystemProcessorInformation/SystemLogicalProcessorInformation (see http://undocumented.ntinternals.net/UserMode/Undocumented%20Functions/Syste…) should help. http://native-nt-toolkit.googlecode.com/svn/trunk/ndk/extypes.h (read, don't copy) seems to have a pretty exhaustive set of definitions, but even that's not complete. Some tests for the NT version would be useful. Mike.

1 0