[SirKnightly]

Here is the nightly for FreeBSD on 18 Nov 2013 - Revision 10115

Group: Standard
fso-FREEBSD-Standard-20131118_r10115.tar.bz2
MD5Sum

Code: [Select]

------------------------------------------------------------------------
r10048 | Goober5000 | 2013-11-11 20:26:37 -0600 (Mon, 11 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/fred2/management.cpp
   M /trunk/fs2_open/code/fred2/missionsave.cpp
   M /trunk/fs2_open/code/fred2/shiptexturesdlg.cpp
   M /trunk/fs2_open/code/mission/missionparse.cpp
   M /trunk/fs2_open/code/mission/missionparse.h

more memset refactoring... turn texture_replace back into a POD struct
------------------------------------------------------------------------
r10049 | Goober5000 | 2013-11-11 20:28:41 -0600 (Mon, 11 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/mission/missionbriefcommon.h
   M /trunk/fs2_open/code/mission/missionparse.h
   M /trunk/fs2_open/code/observer/observer.cpp

better assignment of vec3d and matrix (matrix should be set to identity, not all-0)
------------------------------------------------------------------------
r10050 | Goober5000 | 2013-11-11 20:32:22 -0600 (Mon, 11 Nov 2013) | 2 lines
Changed paths:
   M /trunk/fs2_open/code/fred2/management.cpp
   M /trunk/fs2_open/code/mission/missioncampaign.cpp
   M /trunk/fs2_open/code/mission/missioncampaign.h
   M /trunk/fs2_open/code/network/multi_campaign.cpp

make cmission and campaign into full-fledged classes since they contain non-POD types
remove a memset which hoses the multi campaign class
------------------------------------------------------------------------
r10051 | Goober5000 | 2013-11-11 20:33:55 -0600 (Mon, 11 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/model/modelanim.cpp
   M /trunk/fs2_open/code/model/modelanim.h
   M /trunk/fs2_open/code/ship/ship.cpp
   M /trunk/fs2_open/code/ship/ship.h

ship subsystems should reference an index in a global Triggered_rotations vector rather than maintaining their own class object, which wastes space
------------------------------------------------------------------------
r10052 | niffiwan | 2013-11-12 02:37:51 -0600 (Tue, 12 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/mission/missioncampaign.h

fix gcc warning: [-Wreorder]

?\226?\128?\152campaign::variables?\226?\128?\153 will be initialised after ?\226?\128?\152int campaign::num_missions?\226?\128?\153
------------------------------------------------------------------------
r10054 | The_E | 2013-11-13 01:38:49 -0600 (Wed, 13 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/model/modelanim.cpp
   M /trunk/fs2_open/code/model/modelanim.h
   M /trunk/fs2_open/code/ship/ship.cpp
   M /trunk/fs2_open/code/ship/ship.h

Revert revision 10051: If we're going to use a global vector to store animation info, we need to fill it (and the references to it) too.
------------------------------------------------------------------------
r10055 | niffiwan | 2013-11-13 05:10:58 -0600 (Wed, 13 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/sound/sound.cpp

fix gcc warning: [-Wreorder]

?\226?\128?\152game_snd::flags?\226?\128?\153 will be initialised after ?\226?\128?\152uint game_snd::signature?\226?\128?\153
------------------------------------------------------------------------
r10056 | Goober5000 | 2013-11-13 17:04:48 -0600 (Wed, 13 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/parse/sexp.cpp

don't unnecessarily retime collision checks
------------------------------------------------------------------------
r10057 | Goober5000 | 2013-11-13 18:09:12 -0600 (Wed, 13 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/model/modelanim.cpp
   M /trunk/fs2_open/code/model/modelanim.h
   M /trunk/fs2_open/code/ship/ship.cpp
   M /trunk/fs2_open/code/ship/ship.h

restore r10051, this time allocating triggered_rotation instances when we need them
------------------------------------------------------------------------
r10058 | Goober5000 | 2013-11-13 19:46:54 -0600 (Wed, 13 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/object/deadobjectdock.cpp
   M /trunk/fs2_open/code/object/deadobjectdock.h
   M /trunk/fs2_open/code/object/object.cpp
   M /trunk/fs2_open/code/object/objectdock.cpp
   M /trunk/fs2_open/code/object/objectdock.h

clear dock lists when objects are deleted, and redundantly clear them between missions and when object instances are destroyed (Mantis #2952)
------------------------------------------------------------------------
r10059 | Goober5000 | 2013-11-13 23:26:21 -0600 (Wed, 13 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/mission/missionparse.cpp
   M /trunk/fs2_open/code/object/deadobjectdock.cpp
   M /trunk/fs2_open/code/object/deadobjectdock.h
   M /trunk/fs2_open/code/object/object.cpp
   M /trunk/fs2_open/code/object/objectdock.cpp
   M /trunk/fs2_open/code/object/objectdock.h
   M /trunk/fs2_open/code/object/parseobjectdock.cpp
   M /trunk/fs2_open/code/object/parseobjectdock.h

sync functions for freeing dock instances
------------------------------------------------------------------------
r10060 | Goober5000 | 2013-11-13 23:58:12 -0600 (Wed, 13 Nov 2013) | 4 lines
Changed paths:
   M /trunk/fs2_open/code/mission/missionparse.cpp
   M /trunk/fs2_open/code/mission/missionparse.h
   M /trunk/fs2_open/code/missionui/missionshipchoice.h
   M /trunk/fs2_open/code/missionui/missionweaponchoice.h
   M /trunk/fs2_open/code/network/multi.h
   M /trunk/fs2_open/code/network/multi_ingame.cpp
   M /trunk/fs2_open/code/network/multi_observer.cpp
   M /trunk/fs2_open/code/network/multiutil.cpp
   M /trunk/fs2_open/code/network/multiutil.h
   M /trunk/fs2_open/code/object/parseobjectdock.h

turn p_object into a class
ensure that p_object dock instances are cleared between missions, not just when the game exits
handle Player_start_pobject in a saner way
the p_object Reset method is nice, but it turns out we don't need it
------------------------------------------------------------------------
r10074 | Goober5000 | 2013-11-15 20:45:12 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/object/objectdock.h
   M /trunk/fs2_open/code/object/parseobjectdock.h

change dock_function_info to a class
------------------------------------------------------------------------
r10075 | Goober5000 | 2013-11-15 20:48:50 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/object/parseobjectdock.h

change p_dock_function_info to a class as well
------------------------------------------------------------------------
r10076 | Goober5000 | 2013-11-15 20:58:29 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/osapi/outwnd.cpp

ugh, neither the constructor nor the memsets are actually needed here
------------------------------------------------------------------------
r10077 | Goober5000 | 2013-11-15 21:20:39 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/ship/ship.cpp

now that we have queued_animation_init, we can rewrite this properly
------------------------------------------------------------------------
r10078 | Goober5000 | 2013-11-15 21:22:05 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/parse/sexp.cpp
   M /trunk/fs2_open/code/ship/ship.cpp

these memsets aren't needed
------------------------------------------------------------------------
r10079 | Goober5000 | 2013-11-15 22:37:27 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/ship/ship.cpp

memset initialized the wrong level of pointer :-/
------------------------------------------------------------------------
r10080 | Goober5000 | 2013-11-15 22:39:03 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/ship/ship.cpp
   M /trunk/fs2_open/code/ship/ship.h

make man_thruster a POD struct, and initialize maneuvering thrusters in ship_info initialization
------------------------------------------------------------------------
r10081 | Goober5000 | 2013-11-15 22:39:38 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/ship/ship.h

this exited_ship constructor is redundant and interferes with the POD-ness of the struct
------------------------------------------------------------------------
r10082 | Goober5000 | 2013-11-15 23:51:59 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/ship/ship.h
   M /trunk/fs2_open/code/weapon/weapon.h
   M /trunk/fs2_open/code/weapon/weapons.cpp

remove non-working and non-supported $substitute feature for weapons (Mantis #2951)
------------------------------------------------------------------------
r10083 | Goober5000 | 2013-11-15 23:59:27 -0600 (Fri, 15 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/weapon/weapons.cpp

"new" shouldn't really ever fail
------------------------------------------------------------------------
r10084 | niffiwan | 2013-11-16 00:09:04 -0600 (Sat, 16 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/hud/hudshield.cpp

Remove a magic number
------------------------------------------------------------------------
r10085 | niffiwan | 2013-11-16 01:40:26 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/network/multimsgs.cpp

Coverity 1129273: Negative array index read

weapon_create can return -1
------------------------------------------------------------------------
r10086 | niffiwan | 2013-11-16 01:42:52 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/network/multimsgs.cpp

Coverity 1129272: Negative array index read

weapon_create can return -1
------------------------------------------------------------------------
r10087 | niffiwan | 2013-11-16 02:57:00 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/ai/aiturret.cpp

Coverity 1129270: Negative array index read

weapon_create can return -1
------------------------------------------------------------------------
r10088 | niffiwan | 2013-11-16 03:01:01 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/ai/aiturret.cpp

remove gcc warning: [-Wunused-but-set-variable]

(ret_val earmarked for future use)
------------------------------------------------------------------------
r10089 | niffiwan | 2013-11-16 03:30:50 -0600 (Sat, 16 Nov 2013) | 4 lines
Changed paths:
   M /trunk/fs2_open/code/hud/hudartillery.cpp

Coverity 1129270: Negative array index read

weapon_create can return -1
(& oops, r10087 was 1129271, not 1129270)
------------------------------------------------------------------------
r10090 | niffiwan | 2013-11-16 03:55:39 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/ship/ship.cpp

Coverity 1129274: Negative array index read

weapon_create can return -1
------------------------------------------------------------------------
r10091 | niffiwan | 2013-11-16 04:46:50 -0600 (Sat, 16 Nov 2013) | 3 lines
Changed paths:
   M /trunk/fs2_open/code/weapon/weapons.cpp

Coverity 1129269: Dereference after null check

add checks to deal with parent_objp == NULL
------------------------------------------------------------------------
r10092 | niffiwan | 2013-11-16 04:49:34 -0600 (Sat, 16 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/weapon/weapons.cpp

fix gcc warning: declaration of ?\226?\128?\152shipp?\226?\128?\153 shadows a previous local [-Wshadow]
------------------------------------------------------------------------
r10094 | Goober5000 | 2013-11-16 14:04:21 -0600 (Sat, 16 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/globalincs/systemvars.cpp

add an important note
------------------------------------------------------------------------
r10095 | Goober5000 | 2013-11-16 14:04:50 -0600 (Sat, 16 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/starfield/starfield.cpp

we can just use the STL methods here
------------------------------------------------------------------------
r10108 | Echelon9 | 2013-11-17 10:20:21 -0600 (Sun, 17 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/network/stand_gui-unix.cpp

Remove this memory event, as clear() is called after loop, which will call the destructor of each item. Picked up by AddressSanitizer as this was attempting to free() memory allocated with new()
------------------------------------------------------------------------
r10109 | Echelon9 | 2013-11-17 10:25:53 -0600 (Sun, 17 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/hud/hud.cpp

AddressSanitizer: alloc-dealloc-mismatch (operator new vs free) in hud_close()
------------------------------------------------------------------------
r10110 | Goober5000 | 2013-11-17 22:07:43 -0600 (Sun, 17 Nov 2013) | 2 lines
Changed paths:
   M /trunk/fs2_open/code/parse/sexp.cpp

rather than piggy-back the perform-actions sexp on top of the when sexp, it works better to make it its own operator
(fixes Mantis #2956)
------------------------------------------------------------------------
r10111 | Goober5000 | 2013-11-17 22:49:29 -0600 (Sun, 17 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/parse/sexp.cpp

clarify help... OP_AND will actually evaluate all arguments
------------------------------------------------------------------------
r10112 | karajorma | 2013-11-18 05:47:15 -0600 (Mon, 18 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/freespace2/freespace.cpp
   M /trunk/fs2_open/code/network/stand_gui.cpp
   M /trunk/fs2_open/code/network/stand_gui.h

Fix Mantis 2954 (Standalone can't find default language)
------------------------------------------------------------------------
r10113 | niffiwan | 2013-11-18 06:06:34 -0600 (Mon, 18 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/stats/medals.cpp

According to the medal_stuff class, some of these should be MAX_FILENAME_LEN, not NAME_LENGTH
------------------------------------------------------------------------
r10114 | niffiwan | 2013-11-18 06:06:39 -0600 (Mon, 18 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/network/stand_gui-unix.cpp

Add stub for std_init_os() so that *nix can compile
------------------------------------------------------------------------
r10115 | Echelon9 | 2013-11-18 08:12:05 -0600 (Mon, 18 Nov 2013) | 1 line
Changed paths:
   M /trunk/fs2_open/code/network/stand_gui-unix.cpp

Webui: Resolve bug that was causing standalone game name to not appear
------------------------------------------------------------------------