[PATCH v2 2/3] ntdll/tests: Add tests for ContinueDebugEvent with DBG_REPLY_LATER status.

Rémi Bernon rbernon at codeweavers.com
Thu Feb 13 13:53:42 CST 2020


On 2/13/20 8:45 PM, Jacek Caban wrote:
> Hi Rémi,
> 
> On 10.02.2020 17:48, Rémi Bernon wrote:
>> Signed-off-by: Rémi Bernon<rbernon at codeweavers.com>
>> ---
>>   dlls/ntdll/tests/exception.c | 39 +++++++++++++++++++++++++++++++++++-
>>   1 file changed, 38 insertions(+), 1 deletion(-)
> 
> 
> The test itself looks good to me. However, looking at context and other 
> stuff in test_debugger(), it feels a bit misplaced. How about adding 
> that to kernel32 test_debugger() instead?
> 
> 
> Thanks,
> 
> Jacek
> 

I hesitated between ntdll and kernel32 and it felt easier to put it in 
the loop here. I don't mind moving it though.
-- 
Rémi Bernon <rbernon at codeweavers.com>



More information about the wine-devel mailing list