sfall icon indicating copy to clipboard operation
sfall copied to clipboard

Better debug log format

Open burner1024 opened this issue 5 years ago • 11 comments

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.

burner1024 avatar Sep 20 '18 19:09 burner1024

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

FakelsHub avatar Sep 20 '18 19:09 FakelsHub

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

FakelsHub avatar Sep 20 '18 20:09 FakelsHub

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

burner1024 avatar Sep 20 '18 20:09 burner1024

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

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

FakelsHub avatar Sep 20 '18 23:09 FakelsHub

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

burner1024 avatar Sep 21 '18 07:09 burner1024

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

phobos2077 avatar Sep 21 '18 14:09 phobos2077

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

Во-во.

FakelsHub avatar Sep 21 '18 14:09 FakelsHub

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.

burner1024 avatar Oct 01 '19 11:10 burner1024

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

FakelsHub avatar Oct 01 '19 12:10 FakelsHub

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

burner1024 avatar Oct 01 '19 12:10 burner1024

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

FakelsHub avatar Oct 01 '19 12:10 FakelsHub