search

sfall-team / sfall

sfall - Engine modifications for Fallout 2
https://sfall-team.github.io/sfall/
GNU General Public License v3.0
348 stars 41 forks source link

Better debug log format #195

Open burner1024 opened 6 years ago

burner1024 commented 6 years ago

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 commented 6 years ago

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

FakelsHub commented 6 years ago

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

burner1024 commented 6 years ago

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

FakelsHub commented 6 years ago

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

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

burner1024 commented 6 years ago

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

phobos2077 commented 6 years ago

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

FakelsHub commented 6 years ago

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

Во-во.

burner1024 commented 5 years ago

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 commented 5 years ago

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

burner1024 commented 5 years ago

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

FakelsHub commented 5 years ago

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