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

suppress the output of the sprite_change message #3910

Merged
merged 2 commits into from
Dec 22, 2024
Merged

Conversation

ya4ept
Copy link
Contributor

@ya4ept ya4ept commented Dec 18, 2024

We don't know what type 7 and 13 are. Type 7 looks like body palette or body color. Type 13 looks like body2

fix #3905

https://github.com/HerculesWS/Hercules/blob/stable/src/map/map.h#L784

enum look {
	LOOK_BASE,
	LOOK_HAIR,
	LOOK_WEAPON,
	LOOK_HEAD_BOTTOM,
	LOOK_HEAD_TOP,
	LOOK_HEAD_MID,
	LOOK_HAIR_COLOR,
	LOOK_CLOTHES_COLOR,
	LOOK_SHIELD,
	LOOK_SHOES,
	LOOK_BODY,
	LOOK_FLOOR,
	LOOK_ROBE,
	LOOK_BODY2,

#ifndef LOOK_MAX
	LOOK_MAX
#endif
};

We don't know what type 7 and 13 are. Type 7 looks like body palette or body color. Type 13 looks like body2
@dcyxxt
Copy link

dcyxxt commented Dec 19, 2024

tried "fix_sprite_change",still has the problem

@ya4ept
Copy link
Contributor Author

ya4ept commented Dec 20, 2024

@dcyxxt we need new logs

@dcyxxt
Copy link

dcyxxt commented Dec 20, 2024

@dcyxxt we need new logs

what should i do?

@ya4ept
Copy link
Contributor Author

ya4ept commented Dec 20, 2024

@dcyxxt
Copy link

dcyxxt commented Dec 20, 2024

@dcyxxt we need new logs

console_xxxx.txt
here it is

@ya4ept
Copy link
Contributor Author

ya4ept commented Dec 21, 2024

I found a bug, please test again

@dcyxxt
Copy link

dcyxxt commented Dec 21, 2024

I found a bug, please test again

fixed!thx

@ya4ept ya4ept requested a review from alisonrag December 22, 2024 11:22
@ya4ept ya4ept merged commit ac5dd5f into master Dec 22, 2024
9 checks passed
@ya4ept ya4ept deleted the fix_sprite_change branch December 22, 2024 22:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

appear You changed unknown sprite type (13)
3 participants