Initialize Views With Object Following

Reporter: RobertBColton  |  Status: open  |  Last Modified: November 17, 2018, 10:50:25 PM

This is an alternative way of fixing a glitch in Sam's game that I described in #1430. This fixes the glitch by calling the object following helper during the initialization of the views prior to the create or start events. It also required moving the view/window/screen initialization after the creation of objects and just prior to the firing of their create events.
RobertBColton  
Ok, I made a test gmk: view-following-test.zip

Testing it with this pr shows the fix here does not behave anything like GM. This pr has both views at position (7,7) during the create/start events but GM8.0 and GMSv1.4 do not and have both at (0,0).

Josh said about this:

then toast it; anyone who asks for that information can get it by the proper venues
if someone wants to know where the view is, they want the most correct info
they can hard-code a 0 or reference view_w/hview otherwise
not the PR; toast the GM behavior

codecov[bot]  

Codecov Report

Merging #1435 into master will increase coverage by 0.16%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #1435      +/-   ##
=========================================
+ Coverage   16.74%   16.9%   +0.16%     
=========================================
  Files         164     164              
  Lines       17095   17098       +3     
=========================================
+ Hits         2862    2891      +29     
+ Misses      14233   14207      -26
Impacted Files Coverage Δ
ENIGMAsystem/SHELL/Universal_System/roomsystem.cpp 26.39% <100%> (+3.72%) ⬆️
ENIGMAsystem/SHELL/Universal_System/var4.cpp 17.83% <0%> (+1.59%) ⬆️
...Asystem/SHELL/Universal_System/instance_system.cpp 53.19% <0%> (+2.12%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 950036a...ddb1334. Read the comment docs.

RobertBColton  

I should mention, now that I have #1430 ready for you to review, I don't know about this pull request at all now. The object following actually shouldn't be updated if views are not enabled or the view is not visible, neither of which is being checked by this initialization loop. I should perhaps change this pull request.
Please sign in to post comments, or you can view this issue on GitHub.