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

[SUGGESTION] Add more descriptive warning message than "null nbt in ETF" #302

Open
zeziba opened this issue Aug 29, 2024 · 1 comment
Open
Assignees

Comments

@zeziba
Copy link

zeziba commented Aug 29, 2024

Playing Chosen's pack Project-Architect-2 and there is log spam from ETF from

            if (lastElement == null) {
                ETFUtils2.logWarn("null nbt in ETF");
                return null;
            }

The error is as follows [16:07:05] [Render thread/WARN]: null nbt in ETF.

latest.log

This log file contains 9824 instances of this particular line as well for about 1:30hrs of play time or 109 warnings per min. This was already after several texture packs were removed to mitigate the issue and to see what was causing the spam.

Should make it more verbose so that what ever is causing the warning can be quickly found, as of right now it is not easily possible.

@Traben-0
Copy link
Owner

Traben-0 commented Sep 6, 2024

I intend to remove that log message entirely, my testing instance only had completely valid nbt and i had not accounted for the various valid reasons one might declare nbt keys that will fail

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

No branches or pull requests

2 participants