September 2013 - Wine-Devel

winspool: ERROR_INVALID_NAME in GetDefaultPrinter() ?
by Detlef Riekenberg Oct. 9, 2013

Oct. 9, 2013

Hi. >From which configuration does the "ERROR_INVALID_NAME" came from, when calling GetDefaultPrinter(NULL, &size) and no Printer is installed? This Test is Present in the current "dlls/winspool/tests/info.c". MSDN told us, that we receive an "ERROR_FILE_NOT_FOUND", if no Printer is installed: http://msdn.microsoft.com/library/en-us/gdi/prntspol_0hma.asp I get the "ERROR_FILE_NOT_FOUND" on win98se, winme, w2k and win2003 in this Situation. -- By By ... ... Detlef

3 3

winemac.drv: add fullscreen mode support for Mac OS X 10.7+
by Kevin Eaves Oct. 8, 2013

Oct. 8, 2013

Attached patch is a proof-of-concept for the fullscreen APIs provided in OS X 10.7+, which allows applications to enter fullscreen in a separate desktop space. The fullscreen button appears if a main window is resizable. Basically this feature is the zoom button on crack. It doesn't actually go fullscreen 'natively' but simply resizes the window fullscreen, and automatically moves it to its own space. As far as Wine is concerned the window resized — just like clicking the zoom button and manually assigning the window to its own space. I doubled the MaxTrackSize in user32/winpos.c, but it only needs +15 for the height.

2 2

Re: [4/5] ntdll: Make asynchronous NtWriteFile on a disk file always return STATUS_PENDING.
by Alexandre Julliard Oct. 1, 2013

Oct. 1, 2013

Dmitry Timoshkov <dmitry(a)baikal.ru> writes: > --- > dlls/ntdll/file.c | 3 ++- > dlls/ntdll/tests/file.c | 5 +++-- > 2 files changed, 5 insertions(+), 3 deletions(-) I don't see the point. Do you actually have an app that depends on this? -- Alexandre Julliard julliard(a)winehq.org

2 6

Increasing max_req_size and subsequent errors
by Hugh McMaster Sept. 30, 2013

Sept. 30, 2013

I need to increase the $max_req_size in tools/make_requests. A new request structure for GetConsoleScreenBufferInfoEx is 80 bytes and primarily made up of 16 unsigned long integers. This is fine, but raises the error: wineserver: request.c:755: open_master_socket: Assertion `sizeof(union generic_reply) == sizeof(struct request_max_size)' failed. It looks like this error is connected to struct request_max_size { int pad[16]; }; in include/wine/server_protocol.h. But other than increasing the size of the array, I'm not sure how to fix this appropriately. Any advice will be appreciated.

1 1

Re: wldap32: Use BOOL type where appropriate
by Rico Schüller Sept. 30, 2013

Sept. 30, 2013

On 01.10.2013 00:06, Frédéric Delanoy wrote: > --- > dlls/wldap32/init.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/dlls/wldap32/init.c b/dlls/wldap32/init.c > index b5aed58..54b5b24 100644 > --- a/dlls/wldap32/init.c > +++ b/dlls/wldap32/init.c > @@ -113,13 +113,13 @@ oom: > } > > /* Determine if a URL starts with a known LDAP scheme */ > -static int has_ldap_scheme( char *url ) > +static BOOL has_ldap_scheme( char *url ) > { > if (!strncasecmp( url, "ldap://", 7 ) || > !strncasecmp( url, "ldaps://", 8 ) || > !strncasecmp( url, "ldapi://", 8 ) || > - !strncasecmp( url, "cldap://", 8 )) return 1; > - return 0; > + !strncasecmp( url, "cldap://", 8 )) return TRUE; > + return FALSE; > } > > /* Flatten an array of hostnames into a space separated string of URLs. > You probably may return the "value" of the if statement directly instead of returning TRUE / FALSE. Cheers Rico

1 0

Re: [3/3] ws2_32: Implement WSASendMsg() (try 2)
by Alexandre Julliard Sept. 30, 2013

Sept. 30, 2013

Bruno Jesus <00cpxxx(a)gmail.com> writes: > @@ -2427,6 +2427,46 @@ static void WINAPI WS2_GetAcceptExSockaddrs(PVOID buffer, DWORD data_size, DWORD > } > > /*********************************************************************** > + * WSASendMsg > + */ > +int WINAPI WSASendMsg( SOCKET s, LPWSAMSG msg, DWORD dwFlags, LPDWORD lpNumberOfBytesSent, > + LPWSAOVERLAPPED lpOverlapped, > + LPWSAOVERLAPPED_COMPLETION_ROUTINE lpCompletionRoutine) > +{ > + int ret = SOCKET_ERROR, fd, sock_type; > + socklen_t optlen = sizeof(sock_type); > + > + if (!msg) > + { > + SetLastError(WSAEFAULT); > + return SOCKET_ERROR; > + } > + > + if ( (fd = get_sock_fd( s, 0, NULL )) == -1) > + return SOCKET_ERROR; > + > + if (getsockopt(fd, SOL_SOCKET, SO_TYPE, &sock_type, &optlen) == -1) > + { > + SetLastError((errno == EBADF) ? WSAENOTSOCK : wsaErrno()); > + goto cleanup; > + } > + > + if (sock_type != SOCK_DGRAM && sock_type != SOCK_RAW) > + { > + SetLastError(WSAEINVAL); > + goto cleanup; > + } > + > + ret = WS2_sendto( s, msg->lpBuffers, msg->dwBufferCount, lpNumberOfBytesSent, > + dwFlags, msg->name, msg->namelen, > + lpOverlapped, lpCompletionRoutine ); This is redundant, WS2_sendto will already fetch the file descriptor, any necessary error checking should happen there. -- Alexandre Julliard julliard(a)winehq.org

2 1

Re: [2/3] kernel32/tests: test CopyFileEx callback and cancellation (resend)
by Nikolay Sivov Sept. 30, 2013

Sept. 30, 2013

On 9/30/2013 00:51, Daniel Jeliński wrote: > +struct progress_list { > + const DWORD progress_retval_init; /* value to return from progress routine */ > + const BOOL cancel_init; /* value to set Cancel flag to */ > + const DWORD progress_retval_end; /* value to return from progress routine */ > + const BOOL cancel_end; /* value to set Cancel flag to */ > + const DWORD progress_count; /* number of times progress is invoked */ > + const BOOL copy_retval; /* expected CopyFileEx result */ > + const DWORD lastError; /* expected CopyFileEx error code */ > +} ; I don't see a point making them 'const'. > +static DWORD WINAPI progress(LARGE_INTEGER TotalFileSize, > + LARGE_INTEGER TotalBytesTransferred, > + LARGE_INTEGER StreamSize, > + LARGE_INTEGER StreamBytesTransferred, > + DWORD dwStreamNumber, > + DWORD dwCallbackReason, > + HANDLE hSourceFile, > + HANDLE hDestinationFile, > + LPVOID lpData) > +{ > + progressInvoked++; Please pass all globals as context data with lpData, and please use 'void*' instead of LPVOID.

3 3

Re: atl90: Add new DLL
by Zhenbo Li Sept. 30, 2013

Sept. 30, 2013

2013/9/4 Zhenbo Li <litimetal(a)gmail.com>: > This patch could fix bug 34431, which is caused by missing atl90.dll > > --- > configure | 1 + > configure.ac | 1 + > dlls/atl90/Makefile.in | 7 ++++ > dlls/atl90/atl90.c | 93 > ++++++++++++++++++++++++++++++++++++++++++++++++++ > dlls/atl90/atl90.spec | 52 ++++++++++++++++++++++++++++ > include/atlbase.h | 1 + > tools/make_specfiles | 1 + > 7 files changed, 156 insertions(+) Excuse me, this is my first time to submit a patch to Wine. And I don't know that if I had made some mistakes. Would someone please review my patch and tell me what to do? Thank you. -- Have a nice day! Zhenbo Li

1 0

Re: [PATCH 1/2] setupapi: Also look in datadir/fakedlls for the re-compiled fakedlls.
by Alexandre Julliard Sept. 30, 2013

Sept. 30, 2013

Huw Davies <huw(a)codeweavers.com> writes: > --- > dlls/setupapi/fakedll.c | 13 ++++++++++++- > 1 file changed, 12 insertions(+), 1 deletion(-) I don't think that's correct. The datadir is for platform-independent files, but the fakedlls are platform specific. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: mmdevapi/tests: test MMDevPropStore_GetAt
by Marvin Sept. 30, 2013

Sept. 30, 2013

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 https://newtestbot.winehq.org/JobDetails.pl?Key=2403 Your paranoid android. === wvista (32 bit propstore) === propstore.c:108: Test failed: DEVPKEY_Device_FriendlyName not found

1 0