View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0000773 | FSSCP | gameplay | public | 2006-02-01 10:17 | 2006-06-19 20:35 |
| Reporter | dfryer | Assigned To | taylor | ||
| Priority | normal | Severity | crash | Reproducibility | always |
| Status | resolved | Resolution | fixed | ||
| Summary | 0000773: Assertion failure when countermeasures deployed | ||||
| Description | While playing a multiplayer (2 participants) game of "Feint, Parry, Riposte", I immediately get an assertion failure when I use countermeasures: ASSERTION FAILED: "Weapon_info[Weapons[homing_object->instance].weapon_info_index].wi_flags & WIF_BOMB" at network/multimsgs.cpp:7798 Abort trap I have not tested with other scenarios. | ||||
| Additional Information | Jan 31st CVS, Mac OS 10.3, 'mv_core' and 'mv_effects' from 3.6.8 delta VP (others caused level load hang, a separate bug) --------------------- Basically: new countermeasure code breaks multiplayer. | ||||
| Tags | No tags attached. | ||||
|
|
I didn't even think that WMCoolmon's new countermeasure code would break networking. I'll take this once since I doubt he wants to go through the net code to get this working properly and keep compatibility. |
|
|
The countermeasure code treats countermeasures as weapons. The netcode only lets weapons home onto weapons with the "bomb" flag. edit: The only other occurance of the bomb flag in the netcode is when the weapon is fired, so i'd call this fixed. edited on: 02-01-06 19:35 |
|
|
Counter measures weren't always weapons though. Is it still retail compatible? I was waiting to check on retail compatibility before assuming that fixing the Assert() was the only issue. It's not "fixed" until it still works with older builds. |
|
|
Any news on this? |
|
|
I've only done a quick review of it and didn't find anything obviously wrong. I need to really test it with retail though and that's something that I haven't had a chance to do yet. |
|
|
Ok, this should no longer be an issue. It was a broken packet, as far as retail and older builds go, but it was also broken code which made two packets get sent for a cmeasure rather than just the one real cmeasure packet. This needs to be verified to work properly in multi games with old builds (I'll check tonight) but otherwise I think that the root cause of this has been fixed. May not be how WMCoolmon originally intended all of this to work, but needs to be retail compatible at least and not very broken like it is now. He can fix it another way if the new cmeasure code is ever completed. |
|
|
Oh nutters. :( Thanks, taylor, this is appreciated. :) Any balance bugs like this have the potential to seriously screw up ST:R, since it'll be playtested on retail yet most people will be playing it on FSO. |
|
|
Tested, and it works. May need more testing that I can provide on my own, but it worked fine in a few test runs last night. If someone notices an issue here at a later date then we can just reopen. |
|
|
Fixered. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2006-02-01 10:17 | dfryer | New Issue | |
| 2006-02-01 14:28 | taylor | Status | new => assigned |
| 2006-02-01 14:28 | taylor | Assigned To | => taylor |
| 2006-02-01 14:38 | taylor | Note Added: 0004623 | |
| 2006-02-01 16:13 | taylor | Additional Information Updated | |
| 2006-02-02 00:32 | phreak | Note Added: 0004631 | |
| 2006-02-02 00:35 | phreak | Note Edited: 0004631 | |
| 2006-02-02 00:37 | phreak | Status | assigned => resolved |
| 2006-02-02 00:37 | phreak | Resolution | open => fixed |
| 2006-02-02 00:37 | phreak | Assigned To | taylor => phreak |
| 2006-02-02 00:52 | taylor | Status | resolved => feedback |
| 2006-02-02 00:52 | taylor | Resolution | fixed => reopened |
| 2006-02-02 00:52 | taylor | Note Added: 0004632 | |
| 2006-02-08 00:26 | phreak | Assigned To | phreak => taylor |
| 2006-02-08 00:33 | taylor | Status | feedback => assigned |
| 2006-03-01 03:16 | Goober5000 | Note Added: 0004989 | |
| 2006-03-01 03:57 | taylor | Note Added: 0004994 | |
| 2006-06-18 19:50 | taylor | Note Added: 0005866 | |
| 2006-06-19 00:08 | Goober5000 | Note Added: 0005868 | |
| 2006-06-19 20:35 | taylor | Note Added: 0005876 | |
| 2006-06-19 20:35 | taylor | Status | assigned => resolved |
| 2006-06-19 20:35 | taylor | Resolution | reopened => fixed |
| 2006-06-19 20:35 | taylor | Note Added: 0005877 |
