Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Registration class FS_Path for Lua #459

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

Graff46
Copy link
Contributor

@Graff46 Graff46 commented Jun 5, 2023

Некоторые методы экспортированные из CLocatorAPI в Lua (например .def("get_path", &CLocatorAPI::get_path), метод get_path) возвращают экземпляры FS_Path, но Lua "не понимает" этот тип и бросает ошибку. Я вернул регистрацию FS_Path для Lua, но без методов.

xrSimpodin and others added 3 commits March 10, 2023 13:08
Некоторые методы экспортированные из `CLocatorAPI` в Lua (например `.def("get_path", &CLocatorAPI::get_path)`, метод `get_path`) возвращают экземпляры `FS_Path`, но Lua  "не понимает" этот тип и бросает ошибку. Я вернул регистрацию `FS_Path` для Lua, но без методов.
@Graff46
Copy link
Contributor Author

Graff46 commented Jun 5, 2023

Возможно стоит раскомментировать методы?

@joye-ramone
Copy link
Member

там 2 метода get_path и append_path
вопрос - зачем они в скриптах ?
я только update_path испольшую
get_path вообще возращяет ссылку на внутренний объект движка непотяно зачем

@Graff46
Copy link
Contributor Author

Graff46 commented Jun 5, 2023

там 2 метода get_path и append_path вопрос - зачем они в скриптах ? я только update_path испольшую get_path вообще возращяет ссылку на внутренний объект движка непотяно зачем

Хз что там наделали. Мне надо было использовать append_path, пол ночи искал ошибку. Если что-то есть, оно должно быть рабочим или этого не должно быть.

@joye-ramone
Copy link
Member

joye-ramone commented Jun 5, 2023

вместо append_path испольщуй update_path - тоже самое делает без FS_Path

а тебе для твоих махинаций с FS
ну да. тебе надо чинить. но обычно это не надо и так только накосячиить можно

@Graff46
Copy link
Contributor Author

Graff46 commented Jun 5, 2023

вместо append_path испольщуй update_path - тоже самое делает без FS_Path

Разве? append_path Добавляет новый путь с алиасом (если бы он был в fs_game.ltx). А update_path возвращает абсолютный путь от алиаса и относительного пути.
Вот так у меня работает:

local fs = getFS()
fs:append_path('$game_weathers$', fs:update_path('$game_config$', 'environment\\weathers'), '', 1)

@joye-ramone
Copy link
Member

Да. да. Я вспомнил что у тебя там с работы с путями для сюжета. ну тогда да тебе надо вернуть это все

@Graff46
Copy link
Contributor Author

Graff46 commented Jun 5, 2023

Да. да. Я вспомнил что у тебя там с работы с путями для сюжета. ну тогда да тебе надо вернуть это все

Не для сюжета. У меня одновременно 2 погоды ТЧшная и ЗПшная, я хочу переключать их ингейм. Я написал за ночь рабочий код, но возможно с большими утечками, скрытыми багами, костылями и тд.
Вот коммит: Graff46@5f993f9

@Graff46
Copy link
Contributor Author

Graff46 commented Jun 5, 2023

Вы тогда или добавьте эти правки или удалите 2 метода возвращающие экземпляры путей, а то кто нибудь заюзает в своих модах и тоже будет пол дня причину искать.

@joye-ramone
Copy link
Member

да. я б просто убрал методы пока. у тебя очень спец кейс. не везде такое надо.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants