[Wine-devel] [Wine-patches] ntdll: show backtrace (eterbug #1990)

[Alexander Morozov]

On Fri, 10 Feb 2012 20:21:34 +0400, Roman Dadkov wrote:
> нужна была для баги 5892

В текущем виде мне это не хочется прикладывать.

1. ИМХО, WINESEHBLOCK лучше SEHBLOCK в плане возможных конфликтов с 
другими
переменными окружения.

2. Оформление патча не очень. Есть пробел в конце одной из строк, 
coding style
не соответствует используемому рядом.

3. Выводимое сообщение выглядит как-то не очень:
Handled exception.(D)ebugger, (c)ontinue, (s)top:s
ИМХО, перед (D) и после : стоит вставить пробелы.

3. Если выбрать (D)ebugger, то после закрытия окна Wine с сообщением об 
ошибке выводится
Handled exception.(D)ebugger, (c)ontinue, (s)top:Invalid option 
selected.(D)ebugger, (c)ontinue, (s)top:
Далее при выборе d спрашивает снова до бесконечности. При других 
вариантах зацикливается, ничего не
спрашивая.
Проверял на таком примере, собранном VC++ 6.0:

#include <stdio.h>
#include <windows.h>

int main()
{
     __try
     {
         int a, k;

         puts("start");
         a = 0;
         k = 3 / a;
     }
     __except (EXCEPTION_EXECUTE_HANDLER)
     {
         puts("except");
     }
     return 0;
}