Little-Anti-Cheat icon indicating copy to clipboard operation
Little-Anti-Cheat copied to clipboard

Fix 1.11 compile errors.

Open Natanel-Shitrit opened this issue 3 years ago • 5 comments

Natanel-Shitrit avatar Sep 07 '21 16:09 Natanel-Shitrit

You're quoting https://github.com/alliedmodders/sourcepawn/issues/613 which appears to be reversed.

It shouldn't break on compile anymore, so is this PR really needed?

azalty avatar Sep 07 '21 18:09 azalty

For now the "fix" is on the dev branch, I'm using latest compiler and it's still errors, i guess it's up to the repo owner to accept it or not, i don't mind.

Natanel-Shitrit avatar Sep 07 '21 18:09 Natanel-Shitrit

Oh, so in the 1.11 development version of Sourcemod, using int instead of any in timers causes errors? Skimming through the discussion you link to, it seems like they decided not to do this? Not gonna lie, I'm a little confused... @~@

I'll happily accept this PR if this change really does end up happening to SM, but I'll wait a bit and see if it becomes necessary first. Still, thanks for the PR and the heads up :) :+1:

J-Tanzanite avatar Sep 07 '21 19:09 J-Tanzanite

Oh, so in the 1.11 development version of Sourcemod, using int instead of any in timers causes errors?

In the 1.11 development version this problem is fixed.

Skimming through the discussion you link to, it seems like they decided not to do this? Not gonna lie, I'm a little confused... @~@

The change is already made in the main branch, latest complier errors. (i had a problem with my GitHub workflow that's pulling the latest compiler and compiles plugins, this plugin failed because the errors above)

I'll happily accept this PR if this change really does end up happening to SM, but I'll wait a bit and see if it becomes necessary first. Still, thanks for the PR and the heads up :) 👍

You can accept and revert if they fix it in the future but you can let this boil until they push an update and reject it, I'm cool with both, or anything you wanna do.

Natanel-Shitrit avatar Sep 08 '21 08:09 Natanel-Shitrit

I have decided to merge this. But I would prefer if this could be applied to the development branch. Could you start a PR for that branch? If not, I can do it manually :P

Anyway, thanks for this PR, sorry for having been so slow to respond '^^

J-Tanzanite avatar Jan 03 '22 09:01 J-Tanzanite

Oh right, I forgot this PR existed, just fixed it myself in #150 Closing as this is no longer needed, sorry for the delay and for having done it myself.

J-Tanzanite avatar Dec 04 '22 15:12 J-Tanzanite