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

Added description for includes #1

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Nord1cWarr1or
Copy link

Generated by ChatGPT-3.5

* Gets the type of the specified box.
*
* @param box The index of the box.
* @param type The type of the box.
Copy link
Owner

Choose a reason for hiding this comment

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

Это выходной параметр, надо об этом сказать.

*
* @param box The index of the box.
* @param type The type of the box.
* @param len The length of the type of the box.
Copy link
Owner

Choose a reason for hiding this comment

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

Максимально допустимая длина предыдущего параметра. Текущие описание об этом ничего не говорит.

* TODO
*/
/**
* Called when an entity invalidly touches a box.
Copy link
Owner

Choose a reason for hiding this comment

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

Надо указать, когда тач будет не валидным. Сейчас форвад вызывает, если entity, которая была в соприкосновении с коробкой, перестала быть валидной.

* Called when an entity invalidly touches a box.
*
* @param box The index of the box.
* @param ent The entity that invalidly touched the box.
Copy link
Owner

Choose a reason for hiding this comment

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

Надо понимать, что этот параметр передает невалидный индекс entity. Т.е. с ним нельзя выполнять никаких действий движка.

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.

2 participants