Recursive call in NoticeUI - stack overflow

bsimser / Inventory-Pro

https://devdog.io

MIT License

2 stars 0 forks source link

Closed bsimser closed 5 years ago

bsimser commented 5 years ago

Original report by CyWizz (Bitbucket: cywizz, GitHub: cywizz).

NoticeUI.AddMessage(string, [NoticeDuration]) is calling itself and causing a stack overflow.

Screenshot attached of code.

I marked it as major because it will stop execution, however any of the other overloads can be used as a workaround for now.

bsimser commented 5 years ago

Original comment by Joris Huijbregts (Bitbucket: jjahuijbregts, ).

NoticeUI recursive call fix (fixes #278)

bsimser commented 5 years ago

Original comment by Joris Huijbregts (Bitbucket: jjahuijbregts, ).

NoticeUI recursive call fix (fixes #278)