-
Notifications
You must be signed in to change notification settings - Fork 40
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
Comments
это лог не от sfall, его непосредственно создает движок игры. |
Если ты поспособствуешь и выпишешь строки где нет перевода сток которые тебе встречаются, то я могу тебе скриптик для игры накатать который будет фиксить в экзешнике отсутствующий символ перевода строки. |
я понимаю что не от sfall... переписывать руками мне кажется неоптимально. Может, можно вхучится в фунцию лога собственно? Или ее нет единой? |
единая конечно. а всякие префиксы не ну его нафиг) |
Да, умнее было бы проверять... если только отсутствие перевода не используется где-то специально, чтобы кусками выводить строку. |
Если правильно помню, за вывод в лог отвечает встроенная функция которая по всей видимости просто выводит строку в файл, без переноса на новую строку или имени скрипта. Вроде была еще одна функция более высокого уровня которая добавляла имя скрипта и т.п. Sfall вроде бы её как раз и юзает. Чтобы это исправить надо пройтись по всему коду и заменять вызовы первой функции на вторую, НО только там где это имеет смысл. Кароче гемор. Можно попытаться впихнуть в строку ошибки перенос, но оно того не стоит по-моему. |
Во-во. |
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. |
"Obj is null" - не является ошибкой, поэтому скрываем как излишняя спам инфа, все остальное надо исправлять, а не скрывать. |
Ну я то и говорю. |
А понял. Посмотрю, но помойму нельзя, так просто взять и добавить. |
I'm currently looking at this:
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.
The text was updated successfully, but these errors were encountered: