PocketMine / PocketMine-MP

Legacy PocketMine-MP repository. Head to https://github.com/pmmp for up to date software.
https://www.pocketmine.net/
GNU Lesser General Public License v3.0
1.26k stars 661 forks source link

No stack limit on cactus #3752

Closed Nivek-C94 closed 6 years ago

Nivek-C94 commented 8 years ago

There is no stack limit on cactus and if you stack it a few high and break the bottom it craches the server due to..

Maximum function nesting level of '100' reached, aborting!

inxomnyaa commented 8 years ago

Lol.. due to BlockUpdate when broken

Nivek-C94 commented 8 years ago

yup

inxomnyaa commented 8 years ago

Same for sugar cane?

Am 03.12.2015 um 21:57 schrieb XxDXM-hippiexX notifications@github.com:

yuo

— Reply to this email directly or view it on GitHub.

inxomnyaa commented 8 years ago

https://github.com/ImagicalCorp/ImagicalMine/commit/da15fe2ef0c90e88083fd2cfd378e1b23d3f8b78 This fixes it on Sugarcane and Cacti

inxomnyaa commented 8 years ago

DON'T USE MY FIX! It makes it worse .-."

PEMapModder commented 8 years ago

I'm thinking about Level::scheduleUpdate($pos, 0)

inxomnyaa commented 8 years ago

Just wrote the code. :p because i saw your post on the redstone commit

Am 14.12.2015 um 17:36 schrieb PEMapModder notifications@github.com:

I'm thinking about Level::scheduleUpdate($pos, 0)

— Reply to this email directly or view it on GitHub.

inxomnyaa commented 8 years ago

https://github.com/ImagicalCorp/ImagicalMine/commit/3eaa032c72c7a35ad6be2ea399a06c3bc4115d03 Thanks @PEMapModder , it works just perfect.

inxomnyaa commented 8 years ago

You. This is PM. PocketMine won't disable xdebug because it shows backtrace for API + Debug for Plugins

Am 28.02.2016 um 20:18 schrieb PocketKiller notifications@github.com:

xdebug?

— Reply to this email directly or view it on GitHub.

dktapps commented 6 years ago

This has long since been fixed on PMMP's PocketMine-MP.