Not making scale comparisons with Pawn enabled
Closed this issue · 11 comments
Hmm, it looks like the author of Pawn must have changed something. I haven't been playing WoW much lately and the version of Pawn I'm using still works. I wonder what version of Pawn broke it. If you have time and the desire, would you mind tracking down the last version of Pawn that works with AutoGear for you? If not, I probably won't get around to fixing this for a while since I have a full-time job and a newborn son these days.
What version of Pawn are you using (I am on 2.3.28 but I am not 100% sure when the issue started appearing)? I've poked at the code a little and it seems like it's an intermittent fault, sometimes PawnGetAllScalesEx()
returns the right data, and sometimes it doesn't. I wonder if there's some kind of initialization sequencing issue? I'll add some more debugging and see if I can't more consistently replicate it.
AutoGear does call PawnIsReady() before querying Pawn I think, so if that returns true when Pawn isn't actually ready, that is probably a Pawn bug. It's possible there's an AutoGear bug where it's doing something before getting true from PawnIsReady() though. I'm so busy these days that I'm only typing these comments from my phone and haven't had time to sit down and look at this on my PC yet. Thanks for looking at the code. If you do get it working, I'd be very happy to review and accept a pull request. So sorry about my unavailability.
No dramas on the availability. :) I am not sure what's happening - it's either startup sequencing or some kind of situation where Pawn sometimes doesn't return the spec correctly. You get "Rogue" but not the sub-spec. I'm going to pepper a bunch of debugging in the Pawn and scale functions to see if I can work out what's happening.
Is this still happening? It looks solved here, using TBC Classic and the latest version of Pawn. Please let me know.
Looks fixed.
I just updated Pawn from Pawn-2.5.16 to Pawn-2.5.17 and the problem returned.
I can't reproduce this with the latest alpha version of AutoGear that I released last night. Is there an error? What item is it? What's your scale name?
Ah! I missed last night's release. Looks to be working now.
I think this may be resolved now in the latest release since I moved DB initialization to the proper place (after PLAYER_ENTERING_WORLD). Is it okay now?
Oh, nice. Thanks!