Hard Light Productions Forums

Modding, Mission Design, and Coding => FS2 Open Coding - The Source Code Project (SCP) => Nightly Builds => Topic started by: SirKnightly on January 20, 2022, 12:46:47 am

Title: Nightly: 20 January 2022 - Revision 33fa58400
Post by: SirKnightly on January 20, 2022, 12:46:47 am
Here is the nightly for 20 January 2022 - Revision 33fa58400



Group: Win64-SSE2
nightly_20220120_33fa58400-builds-x64-SSE2.zip (http://scp.indiegames.us/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-x64-SSE2.zip) (Mirror (https://porphyrion.feralhosting.com/datacorder/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-x64-SSE2.zip))


Group: Linux
nightly_20220120_33fa58400-builds-Linux.tar.gz (http://scp.indiegames.us/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-Linux.tar.gz) (Mirror (https://porphyrion.feralhosting.com/datacorder/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-Linux.tar.gz))


Group: Win32-SSE2
nightly_20220120_33fa58400-builds-Win32-SSE2.zip (http://scp.indiegames.us/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-Win32-SSE2.zip) (Mirror (https://porphyrion.feralhosting.com/datacorder/builds/nightly/20220120_33fa58400/nightly_20220120_33fa58400-builds-Win32-SSE2.zip))

Code: [Select]
------------------------------------------------------------------------
commit 33fa58400
Author: Joshua Glatt
Commit: GitHub

    Remove unneeded/leaky code from sexp_list_item in FRED and QtFRED (#3942)
 fred2/sexp_tree.cpp                 | 64 ++++++++++---------------------------
 fred2/sexp_tree.h                   |  7 +---
 qtfred/src/ui/widgets/sexp_tree.cpp | 63 ++++++++++--------------------------
 qtfred/src/ui/widgets/sexp_tree.h   |  7 +---
 4 files changed, 36 insertions(+), 105 deletions(-)

------------------------------------------------------------------------
commit 160b75bd9
Author: Goober5000
Commit: Goober5000

    make clang happy
 code/cmdline/cmdline.cpp | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

------------------------------------------------------------------------
commit a436d5f3d
Author: Goober5000
Commit: Goober5000

    remove redundant externs for clang
 code/mission/missionparse.h | 3 ---
 code/parse/sexp.h           | 3 ---
 2 files changed, 6 deletions(-)

------------------------------------------------------------------------
commit 6db1accdb
Author: Goober5000
Commit: Goober5000

    remove this because it is handled elsewhere, and because ships can have this flag without serving this role
 code/ship/ship.cpp | 4 ----
 1 file changed, 4 deletions(-)

------------------------------------------------------------------------
commit 9e149ff46
Author: Goober5000
Commit: Goober5000

    set the flag if a ship uses a Knossos warpin or warpout effect
 code/mission/missionparse.cpp | 6 ++++++
 1 file changed, 6 insertions(+)

------------------------------------------------------------------------
commit 5562a4f68
Author: Taylor Richards
Commit: GitHub

    update ffmpeg support for 5.0+ API changes (#3939)
 code/cutscene/ffmpeg/FFMPEGDecoder.cpp | 58 ++++++++++++++++++++--------------
 code/cutscene/ffmpeg/internal.h        |  6 ++--
 code/sound/ffmpeg/FFmpegWaveFile.cpp   | 16 +++++++---
 code/sound/ffmpeg/FFmpegWaveFile.h     |  2 +-
 4 files changed, 49 insertions(+), 33 deletions(-)

------------------------------------------------------------------------
commit 32e119457
Author: Goober5000
Commit: Goober5000

    change int to bool
 code/fireball/fireballs.cpp   | 4 ++--
 code/fireball/fireballs.h     | 2 +-
 code/mission/missionparse.cpp | 4 ++--
 code/mission/missionparse.h   | 2 +-
 code/parse/sexp.cpp           | 6 +++---
 code/parse/sexp.h             | 2 +-
 code/ship/ship.cpp            | 2 +-
 7 files changed, 11 insertions(+), 11 deletions(-)

------------------------------------------------------------------------
commit 83faf8de9
Author: Joshua Glatt
Commit: GitHub

    Remove unnecessary/leaky strdup() calls in sexp_list_item in FRED/QtFRED (#3940)
 fred2/sexp_tree.cpp                 | 4 ++--
 qtfred/src/ui/widgets/sexp_tree.cpp | 4 ++--
 2 files changed, 4 insertions(+), 4 deletions(-)

------------------------------------------------------------------------
commit 35eca2ae5
Author: Goober5000
Commit: Goober5000

    allow 3d warp to be enabled from the table
 code/fireball/fireballs.cpp    | 12 +++++++++++-
 code/fireball/fireballs.h      |  2 ++
 code/fireball/warpineffect.cpp |  2 +-
 3 files changed, 14 insertions(+), 2 deletions(-)

------------------------------------------------------------------------
commit eab2401e9
Author: Goober5000
Commit: Goober5000

    Cmdline_3dwarp is now Fireball_use_3d_warp, apparently
 code/cmdline/cmdline.cpp | 3 +--
 code/cmdline/cmdline.h   | 1 -
 2 files changed, 1 insertion(+), 3 deletions(-)

------------------------------------------------------------------------
commit d0964d738
Author: Taylor Richards
Commit: Taylor Richards

    use proper return for invalid particle effect
 code/particle/ParticleManager.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

------------------------------------------------------------------------
commit f2a99117a
Author: Taylor Richards
Commit: Taylor Richards

    always init path renderer
 code/graphics/2d.cpp | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

------------------------------------------------------------------------
commit ca73b4fef
Author: Taylor Richards
Commit: Taylor Richards

    fix crash on standalone from weapon trail cleanup
 code/weapon/trails.cpp | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

------------------------------------------------------------------------
commit e81ad7024
Author: Taylor Richards
Commit: Taylor Richards

    skip tracker init popup on standalone
 code/network/multi_fstracker.cpp | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

------------------------------------------------------------------------
commit 0ba94a554
Author: Taylor Richards
Commit: Taylor Richards

    fix memory leak from libpcp context (undocumented)
 code/network/multi_portfwd.cpp | 2 ++
 1 file changed, 2 insertions(+)

------------------------------------------------------------------------
commit d75174992
Author: Taylor Richards
Commit: Taylor Richards

    release model interp data on level close for standalones
 code/model/modelinterp.cpp | 28 +++++++++++++++++++++-------
 freespace2/freespace.cpp   |  8 ++++++++
 2 files changed, 29 insertions(+), 7 deletions(-)

------------------------------------------------------------------------
commit 45b2d2a83
Author: Taylor Richards
Commit: Taylor Richards

    release memory from object update on level close
 code/network/multi_obj.cpp | 46 ++++++++++++++++++++++++++++++++++++++++++++++
 code/network/multi_obj.h   |  2 ++
 freespace2/freespace.cpp   |  2 ++
 3 files changed, 50 insertions(+)

------------------------------------------------------------------------
commit 11d3348d6
Author: Taylor Richards
Commit: Taylor Richards

    enable/disable extra cmdline options for standalone
 freespace2/freespace.cpp | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

------------------------------------------------------------------------
commit a0769910f
Author: Taylor Richards
Commit: Taylor Richards

    free loaded models after mission on standalone
 freespace2/freespace.cpp | 4 ++++
 1 file changed, 4 insertions(+)

------------------------------------------------------------------------
commit 5e2a4c05a
Author: Taylor Richards
Commit: Taylor Richards

    make sure particles don't get processed on standalone
 code/particle/ParticleManager.cpp              | 50 ++++++++++++++------------
 code/particle/effects/SingleParticleEffect.cpp |  4 +++
 code/particle/particle.cpp                     | 11 ++++++
 code/weapon/weapons.cpp                        |  2 +-
 4 files changed, 44 insertions(+), 23 deletions(-)

------------------------------------------------------------------------
commit 422c20d39
Author: Taylor Richards
Commit: Taylor Richards

    avoid initializing stuff that standalones don't need
 code/graphics/2d.cpp     | 16 +++++++++-------
 freespace2/freespace.cpp |  8 +++++---
 2 files changed, 14 insertions(+), 10 deletions(-)

------------------------------------------------------------------------
commit 0a4d589c1
Author: Goober5000
Commit: Goober5000

    log whether command briefing is using scroll buttons
 code/missionui/missioncmdbrief.cpp | 1 +
 1 file changed, 1 insertion(+)

------------------------------------------------------------------------
commit 6ee6ddf49
Author: Taylor Richards
Commit: GitHub

    fix missing texture bug on standalone (#3937)
 code/bmpman/bmpman.cpp | 2 ++
 1 file changed, 2 insertions(+)

------------------------------------------------------------------------
commit 4ee84b71d
Author: Goober5000
Commit: Goober5000

    refactor some names
 code/io/timer.cpp        | 95 ++++++++++++++++++++++++------------------------
 code/io/timer.h          | 20 +++++-----
 freespace2/freespace.cpp | 20 +++++-----
 3 files changed, 69 insertions(+), 66 deletions(-)

------------------------------------------------------------------------
commit 9c7a9a1c2
Author: Goober5000
Commit: Goober5000

    patch a hole in the pausing and make sure timer pause/unpause is always called
 code/io/timer.cpp        |  2 ++
 freespace2/freespace.cpp | 16 ++++++++--------
 2 files changed, 10 insertions(+), 8 deletions(-)

------------------------------------------------------------------------
commit 73a408043
Author: Goober5000
Commit: Goober5000

    time compression now works
 code/io/timer.cpp | 38 ++++++++++++++++++++++++++++++--------
 1 file changed, 30 insertions(+), 8 deletions(-)

------------------------------------------------------------------------
commit 18f4f490d
Author: Goober5000
Commit: Goober5000

    time compression outline, static casts
 code/io/timer.cpp        | 54 +++++++++++++++++++++++++++---------------------
 code/io/timer.h          |  2 ++
 freespace2/freespace.cpp | 10 +++++++++
 3 files changed, 43 insertions(+), 23 deletions(-)

------------------------------------------------------------------------
commit 97f9aa225
Author: Goober5000
Commit: Goober5000

    move stuff here too
 code/io/timer.h | 79 +++++++++++++++++++++++++++++++--------------------------
 1 file changed, 43 insertions(+), 36 deletions(-)

------------------------------------------------------------------------
commit cf6a46042
Author: Goober5000
Commit: Goober5000

    move some stuff around
 code/io/timer.cpp | 214 +++++++++++++++++++++++++++---------------------------
 1 file changed, 107 insertions(+), 107 deletions(-)

------------------------------------------------------------------------
commit 00de4c3e7
Author: Goober5000
Commit: Goober5000

    timestamps should be running just about all the time
 code/io/timer.cpp        | 54 +++++++++++++++++++++++++++++++++++++++++------
 code/io/timer.h          |  7 +++++-
 code/menuui/credits.cpp  |  3 ---
 code/popup/popup.cpp     |  2 +-
 freespace2/freespace.cpp | 55 +++++++++++++++++++++++++++---------------------
 5 files changed, 85 insertions(+), 36 deletions(-)

------------------------------------------------------------------------
commit e3b4171f0
Author: Goober5000
Commit: Goober5000

    no need to keep Debug_dump_frames anymore
 freespace2/freespace.cpp | 28 +---------------------------
 1 file changed, 1 insertion(+), 27 deletions(-)

------------------------------------------------------------------------
commit a1dadb1c0
Author: Goober5000
Commit: Goober5000

    small refactor
 code/io/timer.cpp | 4 ++--
 code/io/timer.h   | 5 ++---
 2 files changed, 4 insertions(+), 5 deletions(-)

------------------------------------------------------------------------
commit 0dc3c914d
Author: Goober5000
Commit: Goober5000

    tweak the event music start so that missions will be consistent regardless of the player entry delay
 code/gamesnd/eventmusic.cpp         | 9 ++++++++-
 code/gamesnd/eventmusic.h           | 1 +
 code/scripting/api/libs/mission.cpp | 1 -
 freespace2/freespace.cpp            | 4 +++-
 4 files changed, 12 insertions(+), 3 deletions(-)

------------------------------------------------------------------------
commit 8e33644a8
Author: Goober5000
Commit: Goober5000

    since we used to fast-forward at the rate of 1/4 second per frame, simulate the same thing with the timer
 code/io/timer.cpp        | 15 +++++++++++++++
 code/io/timer.h          |  4 ++++
 freespace2/freespace.cpp |  9 ++++++++-
 3 files changed, 27 insertions(+), 1 deletion(-)

------------------------------------------------------------------------
commit 12363fe18
Author: Goober5000
Commit: Goober5000

    qtfred
 qtfred/src/fredstubs.cpp | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

------------------------------------------------------------------------
commit 88f4163ae
Author: Goober5000
Commit: Goober5000

    always give the game priority when starting/stopping time
 code/io/timer.cpp        |  4 ++--
 code/io/timer.h          |  4 ++--
 fred2/fredstubs.cpp      |  2 +-
 freespace2/freespace.cpp | 18 ++++++++++--------
 freespace2/freespace.h   |  4 ++--
 test/src/test_stubs.cpp  |  2 +-
 6 files changed, 18 insertions(+), 16 deletions(-)

------------------------------------------------------------------------
commit 1b51225c5
Author: Goober5000
Commit: Goober5000

    convert the FSO timestamp system to use SDL time
 code/io/timer.cpp        | 105 +++++++++++++++++++++++++++++++++--------------
 code/io/timer.h          |  25 ++++++-----
 code/menuui/credits.cpp  |  10 +++--
 code/network/multi.cpp   |   3 --
 fred2/fredstubs.cpp      |   2 +-
 freespace2/freespace.cpp |  84 ++++++++++++++++++-------------------
 freespace2/freespace.h   |   2 +-
 test/src/test_stubs.cpp  |   2 +-
 8 files changed, 139 insertions(+), 94 deletions(-)
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Antares on January 20, 2022, 03:41:17 am
Some wonky timing issues with this build for sure. The first time I played Last Hope in FSPort, the mission loaded with the Hope already in-system (it normally jumps in after 10 or 15 seconds); the second time I played, the Hope jumped in as soon as the mission started.

Note that restarting the mission seems to fix whatever's going on, as things happen normally; to play around with it, load another mission, then load Last Hope.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Goober5000 on January 20, 2022, 07:24:38 pm
Are you sure you have the correct build selected?  I tested the mission with the current source code and the Hope is delayed as expected.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: taylor on January 20, 2022, 09:00:33 pm
For me it jumped in after about 14-15 seconds a couple of times and ~25 all the rest of the times. The 14-15 second time generally happened with a fresh game load and the first mission played. And it was consistently ~25 seconds with a restart.

I didn't check the mission to determine what time it's actually supposed to show up so I'm not sure which number is wrong.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Goober5000 on January 20, 2022, 09:44:58 pm
The correct time is 25 seconds.  I was able to get this consistently whether it was the first thing I did after starting the game, or I restarted the mission, or I played a different mission first.

I did see it jump in earlier once, but I think I had tabbed away from the window during that session.  I wonder if the time might not be getting paused correctly when the game loses focus.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: taylor on January 20, 2022, 10:42:51 pm
The earlier jump time happened about 1 in 9 tries in my tests, so it wasn't easily reproducible. I'll do some additional testing and see if I can narrow down something that will help explain or better debug this.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Antares on January 21, 2022, 01:21:02 am
Gotcha.

Ten seconds:

(https://i.imgur.com/Xo21csE.png)

Five seconds:

(https://i.imgur.com/XmB2P4b.png)

In the process of jumping in when the mission loaded:

(https://i.imgur.com/JrX0BBa.png)

Believe it or not, this might have something to do with ship selection and/or loadout. That's the only thing I've been changing.

EDIT: No, scratch that. After further investigation, I think what's happening here is that the mission clock actually starts running during the mission briefing, not the mission itself. I've noticed some odd behavior in other missions, like starting far away from other ships that are already engaged in combat, and I think it's because the mission actually begins when the briefing starts--running underneath the briefing itself--so while you're listening to the briefing and choosing your loadout, time is ticking away and your wingmen are already mixing it up. That's why restarting the mission fixes things: it resets the mission timer to zero (and skips the briefing entirely) so everything runs normally.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: wookieejedi on January 21, 2022, 06:52:29 am
Great catch on the briefing. Can confirm on my end, too.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: taylor on January 21, 2022, 07:02:51 am
Confirmed here too.

Edit: Adding to that, I waited over a minute at the briefing and when I entered the mission the Hope was already there and off to my left and the Shivans were in mission too.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Trivial Psychic on January 22, 2022, 07:44:04 am
I'm no coder, but could this change have something to do with it?
Code: [Select]
convert the FSO timestamp system to use SDL time
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: taylor on January 22, 2022, 10:29:50 am
I'm no coder, but could this change have something to do with it?
Code: [Select]
convert the FSO timestamp system to use SDL time

Yes, but it does have conditions to compensate for this issue. Those conditions appear to be working but not having the proper affect here for some reason.
Title: Re: Nightly: 20 January 2022 - Revision 33fa58400
Post by: Goober5000 on January 22, 2022, 05:56:03 pm
I believe I have solved the problem:
https://github.com/scp-fs2open/fs2open.github.com/pull/3948