Hi Octavian,
On 9/1/10 7:45 PM, Octavian Voicu wrote:
> MCI_INTEGER64 is only defined for _WIN64 in DDK.
>
> MCI_INTEGER3264 is wine specific and will be needed because MCI_STATUS
> return type is MCI_INTEGER64 in 64-bit winmm.dll, and MCI_INTEGER
> in 32-bit winmm.dll.
That's not a reason to add Wine-specific defines in public header.
Jacek
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=4902
Your paranoid android.
=== W98SE (32 bit mci) ===
mci.c:926: Test failed: mci status position: 128
In order to prevent copy-paste errors, the changes were generated
using the following script, executed from git root (there are two very
long lines, might not work if they're wrapped):
for i in MCI_OPEN MCI_CLOSE MCI_ESCAPE MCI_PLAY MCI_SEEK MCI_STOP
MCI_PAUSE MCI_INFO MCI_GETDEVCAPS MCI_SPIN MCI_SET MCI_STEP MCI_RECORD
MCI_SYSINFO MCI_BREAK MCI_SOUND MCI_SAVE MCI_STATUS MCI_CUE
MCI_REALIZE MCI_WINDOW MCI_PUT MCI_WHERE MCI_FREEZE MCI_UNFREEZE
MCI_LOAD MCI_CUT MCI_COPY MCI_PASTE MCI_UPDATE MCI_RESUME MCI_DELETE;
do j=`egrep "^#define $i\\W" include/mmsystem.h | awk '{print $3}' |
sed 's/0x0/0x00000/g'`; k=`echo $i | sed 's/MCI_//g'`; sed -i
"s/${j}L, MCI_COMMAND_HEAD,/$i, 0, MCI_COMMAND_HEAD,/ig"
dlls/mciavi32/mciavi_res.rc dlls/mciqtz32/mciavi_res.rc
dlls/winmm/winmm_res.rc; done
for i in MCI_CAPTURE MCI_MONITOR MCI_RESERVE MCI_SETAUDIO MCI_SIGNAL
MCI_SETVIDEO MCI_QUALITY MCI_LIST MCI_UNDO MCI_CONFIGURE MCI_RESTORE;
do j=`egrep "^#define $i\\W" include/digitalv.h | awk '{print $3}' |
sed 's/0x0/0x00000/g'`; k=`echo $i | sed 's/MCI_//g'`; sed -i
"s/${j}L, MCI_COMMAND_HEAD,/$i, 0, MCI_COMMAND_HEAD,/ig"
dlls/mciavi32/mciavi_res.rc dlls/mciqtz32/mciavi_res.rc
dlls/winmm/winmm_res.rc; done
On 1 September 2010 02:40, Misha Koshelev <misha680(a)gmail.com> wrote:
> + while (texture <= 8)
What is the "texture == 8" case for? Also, it seems to me FVFs require
texcoords to be assigned to consecutive texture units.
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=4900
Your paranoid android.
=== W98SE (32 bit mci) ===
mci.c:637: Test failed: Expect message 0001 from play to 250 wait notify
mci.c:644: Test failed: got 0004 instead of MCI_NOTIFY_xyz 0000 from command after close
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=4899
Your paranoid android.
=== W98SE (32 bit mci) ===
mci.c:613: Test failed: Expect message 0001 from play to 250 wait notify
mci.c:620: Test failed: got 0004 instead of MCI_NOTIFY_xyz 0000 from command after close
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=4898
Your paranoid android.
=== W98SE (32 bit mci) ===
mci.c:681: Test failed: not enough time elapsed 58ms
mci.c:737: Test failed: mci status position: 58