Board index Microprose's Magic: The Gathering ManaLink 3.0 Bug Reports Archived Reports Pending Reports
[fixed] Wrong dynamic iids released at eot
Moderators: BAgate, drool66, Aswan jaguar, gmzombie, stassy, CCGHQ Admins
[fixed] Wrong dynamic iids released at eot
by Aswan jaguar » 09 Jan 2023, 09:00
Describe the Bug:
I don't know if you can find anything from this error as it is from my dev, I have only a change on engine.c from what we have on git which doesn't seem to be the cause of this issue. I have no clue what is though, happened for AI as usual.
I am attaching the deck if you can't find from above, to play a bit.
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Based on commit accf88a ("Recommit aaf84de which was reverted back by commit 6bdf6a2", 2023-01-05)/ Gauntlet
What exactly should be the correct behavior/interaction?
Are any other cards possibly affected by this bug?
-
I don't know if you can find anything from this error as it is from my dev, I have only a change on engine.c from what we have on git which doesn't seem to be the cause of this issue. I have no clue what is though, happened for AI as usual.
- blue dump.txt
- (272 Bytes) Downloaded 117 times
I am attaching the deck if you can't find from above, to play a bit.
- Copy of Grand Illusion.txt
- (574 Bytes) Downloaded 128 times
Which update are you using? (date, name)Which type? (duel, gauntlet, sealed deck)
Based on commit accf88a ("Recommit aaf84de which was reverted back by commit 6bdf6a2", 2023-01-05)/ Gauntlet
What exactly should be the correct behavior/interaction?
Are any other cards possibly affected by this bug?
-
Last edited by drool66 on 21 Jan 2023, 19:47, edited 1 time in total.
Reason: fixed
Reason: fixed
---
Trying to squash some bugs and playtesting.
Trying to squash some bugs and playtesting.
-
Aswan jaguar - Super Tester Elite
- Posts: 8080
- Joined: 13 May 2010, 12:17
- Has thanked: 732 times
- Been thanked: 458 times
Re: Dump error - No clue what is the error about
by drool66 » 10 Jan 2023, 04:30
Please attach ManalinkEh.dbg for the dump file to be useful.
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1163
- Joined: 25 Nov 2010, 22:38
- Has thanked: 186 times
- Been thanked: 267 times
Re: Dump error - No clue what is the error about
by Aswan jaguar » 10 Jan 2023, 08:05
Try this savegame in gauntlet if it loads for you it will be more helpful and easier for you to debug.
- blue deck dump.rar
- (3.65 KiB) Downloaded 98 times
---
Trying to squash some bugs and playtesting.
Trying to squash some bugs and playtesting.
-
Aswan jaguar - Super Tester Elite
- Posts: 8080
- Joined: 13 May 2010, 12:17
- Has thanked: 732 times
- Been thanked: 458 times
Re: Dump error - No clue what is the error about
by drool66 » 10 Jan 2023, 16:11
I'm getting a lot of CTDs but no dumps yet
[EDIT] Seems to be related to Phyrexian Metamorph and Phyrexian casting costs
[EDIT] Actually, has something to do with cloning
[EDIT] Seems to be related to Phyrexian Metamorph and Phyrexian casting costs
[EDIT] Actually, has something to do with cloning
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1163
- Joined: 25 Nov 2010, 22:38
- Has thanked: 186 times
- Been thanked: 267 times
Re: [fixed] Wrong dynamic iids released at eot
by drool66 » 21 Jan 2023, 19:51
Fixed. ending_phase() was releasing the original_internal_iid's of everything instead of the current internal_card_id (the dynamic iids actually being used) so any dynamic iid not stored in a dummy3 was being improperly released. I made the same mistake in token_generation::resolve_iid()
EDIT: Fixed in commit 43cf98d ("Check current iids for dynamic iid release candidates", 2023-01-23)
EDIT: Fixed in commit 43cf98d ("Check current iids for dynamic iid release candidates", 2023-01-23)
Last edited by Aswan jaguar on 05 Feb 2023, 14:46, edited 1 time in total.
Reason: add commit HASH
Reason: add commit HASH
The latest images for Manalink will be here.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
The latest Manalink installation directory will be here. Well, not quite, anymore. Check the latest patches.
-
drool66 - Programmer
- Posts: 1163
- Joined: 25 Nov 2010, 22:38
- Has thanked: 186 times
- Been thanked: 267 times
5 posts
• Page 1 of 1
Who is online
Users browsing this forum: No registered users and 3 guests