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

fix: ensure room_id scope #298

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

moushicheng
Copy link

@moushicheng moushicheng commented Jan 10, 2025

Sometimes, the room_id exceeds the limit of 232. I originally intended to change the type of room_id, but since this type is already widely used throughout the system, it's difficult to modify. So, I changed it to 2100 to ensure that the room_id will never overflow.


有时,room_id会超出2^32的上限,我本想修改roomid的类型但是该类型已经在系统内大量使用...不好修改,所以我改成了2^100,保证roomid永远不会炸

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.

1 participant