John Lillis

Results 40 comments of John Lillis

There are still a few changes I've made but not pushed to a PR yet. I'll have this updated and moved out of draft in a week or two.

I have a few other PRs that I would ideally like to get merged ahead of this, but I'm going to say that this is ready for review at least.

Looks great. Do you think this could be easily back-ported to `admin`?

> A few people said it during the previous PR and also on Discord, they think it's not a good solution to implement the chat in the admin resource, since...

It seems there is no actual logic for the "auto" setting as of now. My preference would be just to remove it and have just the "memory" and "cpu" options.

Confirmed. I'm not sure if it's intentional or not, but if it's a recent change in behavior I'd consider it a bug. Would like some more feedback first since I...

[I was afraid this was a possibility. ](https://github.com/multitheftauto/mtasa-resources/pull/389#pullrequestreview-1005091142) Just to make sure I'm understanding fully, do you have additional map scripts in your maps that rely on OOP being enabled...

After some discussions on the dev discord the concern is that [just enabling OOP in a resource's meta.xml will cause an increase in memory usage and performance penalty of about...

If nobody else has comments after a week or two I'll just send it.

Going to go ahead and merge this since the risk of breaking things is low.