Backport VSS to PC version #650

caiiiycuk · 2024-12-02T13:19:07Z

Probably github don't ignore line spacing by default, please use this option to see actual diff:

stalkerg · 2025-02-14T15:04:00Z

lib/xgraph/xgraph.cpp

+		int pitch;
+		SDL_LockTexture(sdlTexture, NULL, &pixels, &pitch);
+		blitRgba((uint32_t*)pixels, get_default_render_buffer(), get_2d_rgba_render_buffer(), get_2d_render_buffer());
+		sys_frameQuant(pixels, xgrScreenSizeX, xgrScreenSizeY, 4);


Не понятно, может ли JS писать в буфер в этом случае. Для PC версии выглядит странным и избыточно.

stalkerg · 2025-02-14T15:05:18Z

lib/xtool/iniparser/iniparser.c

 {
+	const char* ininame = sys_fileOpenQuant(_ininame, /*XS::IN*/ 1);


Внутри делается duk_pop(ctx); что приводит к тому что итоговый указатель на строку может уже не существовать. Скорее всего надо копировать.

stalkerg · 2025-02-14T15:08:15Z

lib/xtool/xstream/xsenlv.cpp

 {
-
+	const char* name = sys_fileOpenQuant(_name, f);


potential undefined behavior

stalkerg · 2025-02-14T15:12:19Z

src/vss/.clang-format

папка /src/vss должна быть в /lib

Кроме того .clang-format должен быть консистентным с глобальным. Или просто быть убран.

stalkerg · 2025-02-14T15:13:46Z

lib/xtool/iniparser/iniparser.c

@@ -17,6 +17,8 @@
 #include <ctype.h>
 #include "iniparser.h"

+extern const char* sys_fileOpenQuant(const char* file, unsigned flags);


Надо избегать использовать позднего связывания и extern.

stalkerg · 2025-02-14T15:16:35Z

lib/xtool/xtcore.cpp

+	id = xtInitApplication();
+
+	if (!sys_readyQuant()) {
+		xtDoneApplication();


Это штатный выход или нет? Не кажется что тут может быть штатный выход.

stalkerg · 2025-02-14T15:25:47Z

lib/xtool/xtcore.cpp

+
+void xtClearMessageQueue(void)
+{
+	sys_tickQuant();


Почему оно тут а не основном цикле? xtClearMessageQueue вызывается так же во время разных анимаций в меню и эскейве.

stalkerg · 2025-02-14T15:27:32Z

scripting/ls.json

Этот файл разве не должен быть в data? А тут может быть example?

stalkerg · 2025-02-14T15:30:27Z

scripting/src/vss-camera.ts

разве не должен быть этот файл в некой папке example?

stalkerg · 2025-02-14T15:37:09Z

src/3d/dynamics.cpp

+					.prop("rudder", rudder)
+					.prop("tractionIncrement", traction_increment)
+					.prop("tractionDecrement", traction_decrement)
+					.prop("tractionMax", 255)


Кажется тут мы должны некую переменную передавать либо иметь константу уже в самом vss.

stalkerg · 2025-02-14T15:40:12Z

src/CMakeLists.txt

@@ -2,6 +2,7 @@ include_directories(
 	"${PROJECT_SOURCE_DIR}/lib/xtool"
 	"${PROJECT_SOURCE_DIR}/lib/xgraph"
 	"${PROJECT_SOURCE_DIR}/lib/xsound"
+	"${PROJECT_SOURCE_DIR}/src/vss/duktape-2.7.0"


Этот движок должен подключаться как зависимость и не быть частью проекта.

stalkerg · 2025-02-14T15:42:21Z

src/CMakeLists.txt

@@ -71,14 +72,24 @@ SET(vangers_SRCS
 	${WINDOWS_RES}
        actint/layout.h)

+SET(vss_SRCS


Надо отдельно сделать CMakeFile для vss и собирать отдельную .a библиотеку.

stalkerg · 2025-02-14T15:45:40Z

src/actint/actint.cpp

+	int x,y,offs = 0;
+	for(y = 0; y < SizeY; y ++){
+		for(x = 0; x < SizeX; x ++){
+			if(!(p -> flags & INV_ITEM_NO_ACTIV


не верный отступ пробелами тут

stalkerg · 2025-02-14T15:45:53Z

src/actint/actint.cpp

+	int x,y,offs = 0;
+	for(y = 0; y < SizeY; y ++){
+		for(x = 0; x < SizeX; x ++){
+			if(!(p -> flags & INV_ITEM_NO_ACTIV


лишний пробел

stalkerg · 2025-02-14T15:48:10Z

src/actint/actint.cpp

+	int x,y,offs = 0;
+	for(y = 0; y < SizeY; y ++){
+		for(x = 0; x < SizeX; x ++){
+			if(!(p -> flags & INV_ITEM_NO_ACTIV


зачем мы отправляем указатель в js и как мы с ним там работаем?

stalkerg · 2025-02-14T15:53:00Z

src/iscreen/iscreen.cpp

@@ -1343,6 +1344,17 @@ void iScreenElement::redraw(int x,int y,int sc,int sm,int hide_mode)
 	if(flags & EL_NO_SCALE && scale != 256)
 		return;

+	auto result = vss::sys()
+			.quant(vss::REDRAW_QUANT)


Зачем нужуен этот квант? Тем более это просто redraw функция а не квант.

stalkerg · 2025-02-14T15:53:42Z

src/iscreen/iscreen.cpp

+			.prop("elementType", type)
+			.send();
+
+	if (result.isPreventDefault()) {


Кажется не безопасно блокировать перерисовку из JS для элементов UI, какой был кейс?

stalkerg · 2025-02-14T15:54:46Z

src/iscreen/iscreen.cpp

-	if(x >= 0 && y >= 0 && x <= SizeX && y <= SizeY)
+	if(x >= 0 && y >= 0 && x <= SizeX && y <= SizeY) {
+		auto result = vss::sys()
+				.quant(vss::CHECK_XY_QUANT)


зачем это?

stalkerg · 2025-02-14T15:55:35Z

src/iscreen/iscreen.cpp

@@ -1878,6 +1900,16 @@ void iScreenObject::init(void)

 void iScreenObject::redraw(int mode)
 {
+	auto result = vss::sys()


похожий вопросс.

stalkerg · 2025-02-14T16:02:12Z

src/vss/duktape-2.7.0/duk_config.h

duktape не должен быть в репозитории.

stalkerg · 2025-02-14T16:03:45Z

src/vss/quant-names.h

+#define VANGERS_QUANT_NAMES_H
+
+namespace vss {
+constexpr const char* READY_QUANT = "ready";


зачем constexpr тут?

stalkerg · 2025-02-14T16:06:15Z

src/vss/sys-bridge.cpp

+       return 1;
+     },
+     1},
+    {"getLineT",


зачем это тут?

stalkerg · 2025-02-14T16:06:40Z

src/vss/sys-bridge.cpp

+       return 1;
+     },
+    1},
+    {"renderLine",


это зачем?

stalkerg · 2025-02-14T16:07:25Z

src/vss/sys-bridge.cpp

+        return 0;
+      },
+    1},
+    {"getRgbaData",


аналогично.

stalkerg · 2025-02-14T16:08:24Z

src/vss/sys-bridge.cpp

+       return 0;
+     },
+     7},
+    {"toBase64",


разве в js мы нет уже готовой функции для этого?

мы где то разве используем base64?

stalkerg · 2025-02-14T16:11:38Z

src/vss/sys-quant.cpp

+  if (duk_get_prop_string(ctx, -1, name)) {
+    value = duk_to_string(ctx, -1);
+  }
+  duk_pop(ctx);


потенциальная проблемма

caiiiycuk added 2 commits December 2, 2024 16:05

Backport VSS to PC version

12ee7c1

Build system fix

6a49a41