Prevent Catalina's heli from despawning
#1254
Open
NABN00B wants to merge 2 commits from NABN00B/master
into master
pull from: NABN00B/master
merge into: mirror:master
mirror:withmorten/limits
mirror:master
mirror:lcs
mirror:miami
mirror:voodoufreak/miami
mirror:okias/dxt_on_ogl
mirror:GenkaOk/abs-function
mirror:asm95/lcs-scrollbar
mirror:ZLau92/highfps
mirror:erorcun/mpeg
mirror:alextwothousand/master
mirror:erorcun/master
mirror:alextwothousand/miami
mirror:Anarky/blur_cutscene
mirror:xeyopay170/master
mirror:Nanoposan69/master
mirror:IlDucci/miami
mirror:Tatsh/xdg
mirror:madebr/re3_conan_libsndfile
mirror:ZLau92/gamepad-dbg-menu
mirror:0x0000ff/master
mirror:Tatsh/macos-appsupport
Reviewers
Request review
No reviewers
Labels
Clear labels
Something isn't working
need for improvement or additions to documentation
This issue or pull request already exists
New feature or request
Good for newcomers
Extra attention is needed
This doesn't seem right
reLCS related
reVC related
Further information is requested
Issues happening in original game
This will not be worked on
Apply labels
bug
Something isn't working
documentation
need for improvement or additions to documentation
duplicate
This issue or pull request already exists
enhancement
New feature or request
fork idea
good first issue
Good for newcomers
help wanted
Extra attention is needed
help-desk
invalid
This doesn't seem right
is it bug or not?
lcs
reLCS related
linux
math
miami
reVC related
not our problem
openal
performance
question
Further information is requested
vanilla
Issues happening in original game
wontfix
This will not be worked on
No Label
bug
documentation
duplicate
enhancement
fork idea
good first issue
help wanted
help-desk
invalid
is it bug or not?
lcs
linux
math
miami
not our problem
openal
performance
question
vanilla
wontfix
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Set Project
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.
No due date set.
Dependencies
No dependencies set.
Reference: mirror/re3#1254
Reference in New Issue
There is no content yet.
Delete Branch 'NABN00B/master'
Deleting a branch is permanent. It CANNOT be undone. Continue?
No
Yes
Fixes possible sequence break on The Exchange.
So, how do I reproduce this? Just get into the tunnel during the last mission?
Exactly. The helicopter will despawn, and there's even a visual key for this as its radar blip gets removed. When you drive out, the helicopter will spawn above the picnic area and starts making its way to the dam again. This is a problem in itself because it can be in a weird place when the player actually triggers the fight with it.
The sequence break is that the newly spawned helicopter doesn't have any special properties so it can be blown up immediately. This immediately spawns Maria at her usual place and despawns all Colombians, essentially skipping all the fighting at the dam.
i think
if(i == HELI_CATALINA) continue;
inside the loop is cleanerWe still don't know what causes the heli to fly East if you go into a tunnel after it had landed at the dam. Perhaps we should track down and fix that too before merging this.
Shouldn't your fix address specifically that?
Only indirectly.