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

Снижение градуса спама в логах #712

Merged
merged 10 commits into from
Aug 22, 2022

Conversation

ssdaniel24
Copy link
Contributor

@ssdaniel24 ssdaniel24 commented Aug 22, 2022

Убраны предупреждения о неправильных текстурах, предупреждения о некоторых устаревших функциях. Исправлена ненормальная перезапись льда и сена. Переделаны или перенесены на другой уровень некоторые логи.

no squash

Copy link
Contributor

@weqqr weqqr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

game.conf Outdated Show resolved Hide resolved
mods/lord/server_message/init.lua Show resolved Hide resolved
"Irrlicht: PNG warning: iCCP: known incorrect sRGB profile"
Thanks `pngcrush` for detecting those erroneous files!
"Irrlicht: PNG warning: Interlace handling should be turned on when using png_read_image"
There is very old bug of libpng. So, this commit solves foreign bug.

Thanks `imagemagick` for finding problem image:
`magick some_img.png json: | jq '.[0].image.interlace'`
But, uh, we got another warnings:
2022-08-22 10:02:12: WARNING[Main]: NodeDefManager::deSerialize(): already defined with different ID: stairs:stair_straw
2022-08-22 10:02:12: WARNING[Main]: NodeDefManager::deSerialize(): already defined with different ID: stairs:stair_inner_straw
2022-08-22 10:02:12: WARNING[Main]: NodeDefManager::deSerialize(): already defined with different ID: stairs:stair_outer_straw
2022-08-22 10:02:12: WARNING[Main]: NodeDefManager::deSerialize(): already defined with different ID: stairs:slab_straw
@weqqr weqqr merged commit feb0976 into lord-server:dev Aug 22, 2022
@weqqr weqqr added this to the 2022.08 milestone Aug 22, 2022
@ssdaniel24 ssdaniel24 deleted the removing-warnings branch May 10, 2024 09:30
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

Successfully merging this pull request may close these issues.

3 participants