Skip to content

Hack dtor for mysqli to cleanup resources.#45

Merged
wu-sheng merged 1 commit intoapache:masterfrom
jmjoy:mysqli
Jan 18, 2023
Merged

Hack dtor for mysqli to cleanup resources.#45
wu-sheng merged 1 commit intoapache:masterfrom
jmjoy:mysqli

Conversation

@jmjoy
Copy link
Member

@jmjoy jmjoy commented Jan 17, 2023

Hack the destructor to remove the global hash map value hold by the object id key, to cleanup resources.

This is done for other plugins, only mysqli is left.

@jmjoy jmjoy added this to the 0.3.0 milestone Jan 17, 2023
@jmjoy jmjoy marked this pull request as ready for review January 17, 2023 10:57
@jmjoy jmjoy requested review from heyanlong and wu-sheng January 17, 2023 12:01
@wu-sheng
Copy link
Member

Could you explain what is this hack about?

@jmjoy
Copy link
Member Author

jmjoy commented Jan 18, 2023

Could you explain what is this hack about?

Hack the destructor to remove the global hash map value hold by the object id key, to cleanup resources.

This is done for other plugins, only mysqli is left.

@jmjoy jmjoy changed the title Hack dtor for mysqli. Hack dtor for mysqli to cleanup resources. Jan 18, 2023
@wu-sheng wu-sheng merged commit 9d76bc3 into apache:master Jan 18, 2023
@jmjoy jmjoy deleted the mysqli branch January 18, 2023 01:17
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