Stefano Guidoni wrote:
> This patch implements ACM_FORMATENUMF_SUGGEST and fixes bug 16394.
>
> >From MSDN: "The WAVEFORMATEX structure pointed to by the pwfx
> member
> of
> the ACMFORMATDETAILS structure is valid. The enumerator will
> enumerate
> all suggested destination formats for the given pwfx
> format.".
> Since
> the WAVEFORMATEX structure is valid, there is no need
> to cycle through
> all the standard formats (moreover the WAVEFORMATEX
&…
[View More]gt; format could be not
> a standard one).
> _________________________
> S. Guidoni
> ------------------------------------------------------------------------
>
>
>
Can you please use standard compression programs with standard extensions.
Your file extension, .bin will not work here.
James McKenzie
[View Less]
Nikolay Sivov <bunglehead(a)gmail.com> writes:
> @@ -775,6 +777,16 @@ STATUSBAR_SetTextT (STATUS_INFO *infoPtr, INT nPart, WORD style,
> strcpyW (ntext, text);
> } else ntext = 0;
>
> + /* replace nonprintable characters with spaces */
> + if (ntext) {
> + idx = ntext;
> + while (*idx) {
> + if(!isprintW(*idx))
> + *idx = space;
> + idx++;
This would need some test cases with various invalid chars to confirm
…
[View More]that isprintW is the right check to use.
--
Alexandre Julliard
julliard(a)winehq.org
[View Less]
Rico Schüller <kgbricola(a)web.de> writes:
> @@ -1498,7 +1498,10 @@ DWORD WINAPI mciSendStringA(LPCSTR lpstrCommand, LPSTR lpstrRet,
> MultiByteToWideChar( CP_ACP, 0, lpstrCommand, -1, lpwstrCommand, len );
> if (lpstrRet)
> {
> - lpwstrRet = HeapAlloc(GetProcessHeap(), 0, uRetLen * sizeof(WCHAR));
> + /* Use HEAP_ZERO_MEMORY, otherwise WideCharToMultiByte could crash if mciSendStringW fails,
> + * because the length is taken from …
[View More]the uninizialized data!
> + */
> + lpwstrRet = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, uRetLen * sizeof(WCHAR));
That's just hiding the bug. The string shouldn't be converted at all if
we failed to retrieve it.
--
Alexandre Julliard
julliard(a)winehq.org
[View Less]
Hi,
I am running wine 1.1.16 on a mac with leopard. Everything is working
great except for my dvd drive. Wine cannot seem to recognize it. Is
there any special settings I need to adjust in order for it to
function correctly?
On Sun, 1 Mar 2009, Austin English wrote:
> There's already one skip() in the file for some unimplemented stuff,
> so basing these fixes on that (only skipping on ERROR_NOT_SUPPORTED).
> If that's not what we should be doing (and just let it fail instead),
> feel free to ignore. I asked on #winehackers but couldn't get a clear
> answer as to what skip() should be used for...
[...]
> apiReturn = gGetIfEntry(&row);
> + if (apiReturn == ERROR_NOT_SUPPORTED)
> + …
[View More] {
> + skip("GetIfEntry is not supported\n");
> + return;
> + }
If there are Windows versions that export these functions but don't
implement them so that they systematically return ERROR_NOT_SUPPORTED,
then a win_skip() is justified.
If there are legitimate reasons for Wine to return ERROR_NOT_SUPPORTED,
besides 'we did not implement it yet for this platform', then a skip()
is justified. So you would use a skip() if the graphics card does not
support the 3D features you want to test for instance. Or to skip over
the sound capture tests if the sound card has no capture capability.
However you should not add a skip() just to hide platform-specific test
failures, or failures caused by unimplemented or partially implemented
features.
In this case it's not totally clear cut but it does look like it would
just hide unimplemented functionality. Note that Alexandre has been
reworking this area quite a bit recently so maybe your original impetus
for adding these is gone.
--
Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/
A black hole is just God dividing by zero.
[View Less]
On Sun, 1 Mar 2009, Austin English wrote:
> OpenSolaris defines _FILE_OFFSET_BITS as well, causing a conflict.
_FILE_OFFSET_BITS modifies the behavior of the Unix headers. So if it's
set already, then it's wrong to redefine it and continue to include
other Unix headers because you'll end up with inconsistent definitions.
I think what this means is that there are places that include port.h
after they have included Unix headers and that's wrong. It's these files
that should be fixed.
-…
[View More]-
Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/
It really galls me that most of the computer power in the world
is wasted on screen savers.
Chris Caldwell from the GIMPS project
http://www.mersenne.org/prime.htm
[View Less]
> + wine_set_fs( thread_data->fs );
> + wine_set_gs( thread_data->gs );
> +#endif
> +
> /* get %fs and %gs at time of the fault */
> #ifdef FS_sig
> *fs = LOWORD(FS_sig(sigcontext));
Isn't it a problem to move wine_set_{fs,gs}() just before code that
purports to 'get %fs and %gs at time of the fault'?
--
Francois Gouget <fgouget(a)free.fr> http://fgouget.free.fr/
Hiroshima '45 - Czernobyl '86 - Windows '95