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

Sign memory leak #26

Closed
archie426 opened this issue Oct 15, 2016 · 3 comments
Closed

Sign memory leak #26

archie426 opened this issue Oct 15, 2016 · 3 comments

Comments

@archie426
Copy link

Issue description

Write a short description about the issue
Sign memory problem

Steps to reproduce the issue

  1. ...Place a sign and take a certain amount of time writing (any other steps are welcome)

OS and versions

  • PocketMine-MP: Latest
  • PHP: Latest
  • OS: Windows 10 Pro 64-bit

Crashdump, backtrace or other files

  • No crashes are shown
@archie426
Copy link
Author

archie426 commented Oct 15, 2016

I reckon this is a bug, it happens on nukkit as well
Kill me if this was fixed

@dktapps
Copy link
Member

dktapps commented Oct 15, 2016

This is unlikely to be a memory leak and does not provide any useful information about how to reproduce. And "latest" is not a valid version.

@SOF3
Copy link
Member

SOF3 commented Oct 15, 2016

If you indeed believe this is a problem, please provide a fair test, where you duplicate the server, change to a different port, start the two servers at the same time, join both servers at the same time with two clients, and only edit a sign on one server (and to make a control setup to make the test even more fair, place a block on the other server, and place a chest when you finish editing).
(And this still isn't strictly fair enough, because the random seed is different)

Please do not casually blame anything as the source of a memory leak. A memory leak can come from anything you least expect, such as a block of fire extinguishes.

@ghost ghost mentioned this issue Dec 15, 2018
This was referenced Jul 13, 2019
@ghost ghost mentioned this issue Jul 19, 2020
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

3 participants