[Wine-devel] [Wine-patches] [eterhack] ntdll : add ThreadQuerySetWin32StartAddress in NtQueryInformationThread

[Alexander Morozov]

Subject: [PATCH] ntdll : add ThreadQuerySetWin32StartAddress in
  NtQueryInformationThread

ИМХО, лучше было бы так:
ntdll: Add ThreadQuerySetWin32StartAddress in NtQueryInformationThread 
(eterbug #номер).
Так оно больше похоже на то, как пишут в апстриме.

@@ -1038,6 +1038,28 @@ NTSTATUS WINAPI NtQueryInformationThread( HANDLE 
handle, THREADINFOCLASS class,
  #endif
              return status;
          }
+
Не стоит тут вставлять пустую строку, т.к. в других местах её нет.

+    case ThreadQuerySetWin32StartAddress:
+        {
+            if (length != sizeof(ULONG_PTR))
Точно != ? Если буфер больше, чем надо, то должна возвращаться ошибка?

@@ -5713,6 +5714,6 @@ union generic_reply
      struct set_suspend_context_reply set_suspend_context_reply;
  };

-#define SERVER_PROTOCOL_VERSION 10433
+#define SERVER_PROTOCOL_VERSION 10443
Думаю, достаточно увеличить на 1.

Если написать тест, то можно попробовать отправить в апстрим.
Если будешь делать патч для апстрима, всё автогенерируемое в него 
включать не надо.