Skip to content
This repository has been archived by the owner on Apr 17, 2022. It is now read-only.

Loading save bug #4815

Closed
wzdev-ci opened this issue Jul 22, 2018 · 12 comments
Closed

Loading save bug #4815

wzdev-ci opened this issue Jul 22, 2018 · 12 comments

Comments

@wzdev-ci
Copy link
Contributor

resolution_fixed type_bug | by Berserk Cyborg


Was testing some reticule stuff and noticed there is a new save-load bug where Warzone essentially hangs indefinitely. Campaign save attached (created with current master).

[fpathRoute:386] starting new job for droid 4129 0x1021

Issue migrated from trac:4815 at 2022-04-16 13:07:59 -0700

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg uploaded file LoadBug.zip (70.5 KiB)

@wzdev-ci
Copy link
Contributor Author

pastdue changed status from new to assigned

@wzdev-ci
Copy link
Contributor Author

pastdue changed owner from `` to pastdue

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 4, 2018

WZ2100ModsFAn changed _comment0 which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 4, 2018

WZ2100ModsFAn changed _comment1 which not transferred by tractive

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 4, 2018

WZ2100ModsFAn commented


So this #4815 are to be fixed. then we can get a release?

I see there hasn't been activity since a week ago.

http://forums.wz2100.net/viewtopic.php?f=43&p=142346#p142346

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 9, 2018

Berserk Cyborg commented


If it helps, I see it always gets stuck in the while loop a little bit above that log.

// Check if waiting for a result
while (psMove->Status == MOVEWAITROUTE)
{
    ...
    PATHRESULT result = I->second.get();      <- hangs at this line

@wzdev-ci
Copy link
Contributor Author

pastdue commented


@BerSerK Cyborg:

Please give this patch a try: https://github.com/past-due/warzone2100/commit/a4e8092c2216b1756afd00516b3481c3d84dd7a2.patch

@wzdev-ci
Copy link
Contributor Author

wzdev-ci commented Aug 15, 2018

Berserk Cyborg commented


Replying to Warzone2100/old-trac-import#4815 (comment:4):

@BerSerK Cyborg:

Please give this patch a try: https://github.com/past-due/warzone2100/commit/a4e8092c2216b1756afd00516b3481c3d84dd7a2.patch

Tried ~20+ saves (on home map and away missions) and have not encountered any problems yet. Thanks!

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg changed status from assigned to closed

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg changed resolution from `` to fixed

@wzdev-ci
Copy link
Contributor Author

Berserk Cyborg commented


Fixed in 9384a8274fded4e8b091d0d1740dfd61d7bdae37.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant