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

Better debug log format #195

Open
burner1024 opened this issue Sep 20, 2018 · 11 comments
Open

Better debug log format #195

burner1024 opened this issue Sep 20, 2018 · 11 comments

Comments

@burner1024
Copy link
Contributor

I'm currently looking at this:

captura de pantalla de 2018-09-21 02-09-54

Which lead me to think, maybe log format can be improved? Force line breaks, at least.
Prefixing messages with actual script name and timestamp also could be useful.

@FakelsHub
Copy link
Contributor

это лог не от sfall, его непосредственно создает движок игры.
и переписывать косяковые строки в движке... ну у меня как бы есть более важные задачи)

@FakelsHub
Copy link
Contributor

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

@burner1024
Copy link
Contributor Author

я понимаю что не от sfall... переписывать руками мне кажется неоптимально. Может, можно вхучится в фунцию лога собственно? Или ее нет единой?
Так-то руками захачить я и сам смогу наверное.

@FakelsHub
Copy link
Contributor

FakelsHub commented Sep 20, 2018

единая конечно.
те ты предлагаешь проверять строку на наличие символа перевода строки?

а всякие префиксы не ну его нафиг)

@burner1024
Copy link
Contributor Author

Да, умнее было бы проверять... если только отсутствие перевода не используется где-то специально, чтобы кусками выводить строку.

@phobos2077
Copy link
Collaborator

Если правильно помню, за вывод в лог отвечает встроенная функция которая по всей видимости просто выводит строку в файл, без переноса на новую строку или имени скрипта. Вроде была еще одна функция более высокого уровня которая добавляла имя скрипта и т.п. Sfall вроде бы её как раз и юзает. Чтобы это исправить надо пройтись по всему коду и заменять вызовы первой функции на вторую, НО только там где это имеет смысл. Кароче гемор. Можно попытаться впихнуть в строку ошибки перенос, но оно того не стоит по-моему.

@FakelsHub
Copy link
Contributor

если только отсутствие перевода не используется где-то специально

Во-во.

@burner1024
Copy link
Contributor Author

Is it possible to at least add script name to "Obj is null" and "var out of range" messages? That would allow to fish them out and fix one by one eventually.
I'm aware HideObjIsNullMsg was added recently, but hiding is not quite the same as fixing.

@FakelsHub
Copy link
Contributor

"Obj is null" - не является ошибкой, поэтому скрываем как излишняя спам инфа, все остальное надо исправлять, а не скрывать.

@burner1024
Copy link
Contributor Author

Ну я то и говорю.

@FakelsHub
Copy link
Contributor

FakelsHub commented Oct 1, 2019

А понял. Посмотрю, но помойму нельзя, так просто взять и добавить.

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

No branches or pull requests

3 participants