search

Russian-Doom / russian-doom

A limit-removing source port of Doom, Heretic and Hexen. It has numerous vanilla bug fixes, enhanced 640x400 and 1280x800 rendering resolutions, improved game palettes and offers many optional aesthetic game enhancements along with the maximum possible translation to the Russian language.
GNU General Public License v2.0
80 stars 14 forks source link

Fix `sscanf_s` check, allow `sscanf_s` on Win x64 #375

Closed Dasperal closed 2 years ago

Dasperal commented 2 years ago

Forgot that _WIN32 also exists on 64-bit Windows. May still fail on WinXP 64, don't have one to test

JNechaevsky commented 2 years ago

У меня тоже на виртуалках XP x64 нет, да и врядли она когда-то особой популярностью пользовалась, не говоря уже о физической установке. Последнее что помню после разовой установки на VM - там какой-то ужас то ли с получением прав администратора, то ли неотключаемый UAC.

Мерджим?

Dasperal commented 2 years ago

Вот тоже думаю что XP x64 встречается крайне редко. К тому же XP обычно используется на ретро железе где объём оперативной памяти ниже, а x64 имеет оверхэд по памяти из-за увеличения длинны указателя. Так же x64 билд в теории может использовать инструкции недоступные старым процессорам.

Думаю лучшим вариантом будет определить x64 билд как не совместимый с Win XP и предназначенный для современных систем.