Hi,
No more MCIERR_HARDWARE in David's latest test result, hurray!
The difference with prior tests is that I temporarily eliminated 3 commands
a) resume c
b) set c video audio all off
c) set c door closed notify
We are not done yet. Users who saw that error please feed the following
scripts
Hello,
some comments about this patch
+if (hr != S_OK) return FALSE;
+else return TRUE;
could be simplified as
return SUCCEEDED(hr);
+trace(HRESULT when creating instance (%x)\n, hr);
Since tests are assumed to pass, we dont care about debug informations. So,
traces are useless in
Hi,
what happened to this patch? It's still marked new.
Is the discovery too surprising?
(Maybe it's the wrong time to ask with AJ still on vacation)
Is it not robust enough (lacking testing with network drives)?
It explains bugs #20887 and #23355.
It may explain intermittent failures in
Hi,
testbot.winehq.org job #4404 now contains another patch and binary. I removed
the call to
set c door closed
estimating that this was the most likely among the three suspects to cause the
error
(I also reduced the Sleep() time).
Please test yet once again. I've not changed anything about
On Wednesday, August 11, 2010 3:12:23 am David Adam wrote:
Hello,
some comments about this patch
+if (hr != S_OK) return FALSE;
+else return TRUE;
could be simplified as
return SUCCEEDED(hr);
SUCCEEDED encompasses more than just S_OK (S_FALSE, for instance; basically
anything
On Wed, 11 Aug 2010, joerg-cyril.hoe...@t-systems.com wrote:
Hi,
testbot.winehq.org job #4404 now contains another patch and binary. I removed
the call to
set c door closed
estimating that this was the most likely among the three suspects to cause
the error
(I also reduced the Sleep()
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=4405
Your paranoid
Am 11.08.2010 um 21:51 schrieb Rico Schüller:
--- a/dlls/d3d10/d3d10_private.h
+++ b/dlls/d3d10/d3d10_private.h
@@ -212,6 +212,17 @@ struct d3d10_effect
struct d3d10_effect_technique *techniques;
};
+extern const struct ID3D10BlobVtbl d3d10_blob_vtbl DECLSPEC_HIDDEN;
Do you need the
On 11 August 2010 20:12, David Adam david.adam.c...@gmail.com wrote:
Hello,
thanks for the comments. I've resent.
+ todo_wine ok(*(ULONG *)pDirectMusicPort != 0, IDirectMusicPort not
set\n);
Why is this cast useful?
I wasn't sure of the best way to do this. But basically I want to
Austin Lund wrote:
Why the try2? It is always nice to put why you are doing this in the
git commit notes.
AJ is on vakay and will not be back until next week so there are NO
reviews ongoing this week, except developer peer ones.
James McKenzie
On 12 August 2010 13:16, James McKenzie jjmckenzi...@earthlink.net wrote:
Austin Lund wrote:
Why the try2? It is always nice to put why you are doing this in the git
commit notes.
I was responding to review. I'm sorry if I've done something confusing.
Sorry, I failed at Gmail again. _
-- Forwarded message --
From: Paul TBBle Hampson paul.hamp...@pobox.com
Date: 12 August 2010 13:52
Subject: Re: Should we expect Liberation fonts to be installed?
To: Scott Ritchie sc...@open-vote.org
On 8 August 2010 13:02, Scott Ritchie
On 08/11/2010 05:03 AM, joerg-cyril.hoe...@t-systems.com wrote:
Hi,
what happened to this patch? It's still marked new.
Is the discovery too surprising?
(Maybe it's the wrong time to ask with AJ still on vacation)
According to this:
On 08/11/2010 09:14 PM, Austin Lund wrote:
On 11 August 2010 20:12, David Adamdavid.adam.c...@gmail.com wrote:
I wasn't sure of the best way to do this. But basically I want to
check that the Vtbl had been set. I cannot cast to
IDirectMusicPortImpl and I'm still confused by the COM macros.
On 08/11/2010 08:58 PM, Austin Lund wrote:
+static BOOL test_InitAudio(void)
+{
+hr =
IDirectMusicPerformance8_InitAudio(idmusicperformance,NULL,pDirectSound,NULL,1,128,0x3f,NULL);
+if (hr != S_OK) return FALSE;
+else return TRUE;
+}
Please put spaces after comas, as you do in
On 12 August 2010 14:23, Vitaliy Margolen wine-de...@kievinfo.com wrote:
On 08/11/2010 09:14 PM, Austin Lund wrote:
On 11 August 2010 20:12, David Adamdavid.adam.c...@gmail.com wrote:
I wasn't sure of the best way to do this. But basically I want to
check that the Vtbl had been set. I
On 12 August 2010 14:32, Vitaliy Margolen wine-de...@kievinfo.com wrote:
What are those magic numbers? Please use constants. If they are missing, add
them to appropriate header file first.
They are the numbers passed to the InitAudio method from bug 22598.
I've seen quite a few places where
On 08/11/2010 11:16 PM, Austin Lund wrote:
On 12 August 2010 14:32, Vitaliy Margolenwine-de...@kievinfo.com wrote:
What are those magic numbers? Please use constants. If they are missing, add
them to appropriate header file first.
They are the numbers passed to the InitAudio method from bug
On 08/11/2010 11:11 PM, Austin Lund wrote:
On 12 August 2010 14:23, Vitaliy Margolenwine-de...@kievinfo.com wrote:
On 08/11/2010 09:14 PM, Austin Lund wrote:
On 11 August 2010 20:12, David Adamdavid.adam.c...@gmail.comwrote:
I wasn't sure of the best way to do this. But basically I want
19 matches
Mail list logo