Hard Light Productions Forums

Modding, Mission Design, and Coding => FS2 Open Coding - The Source Code Project (SCP) => Nightly Builds => Topic started by: SirKnightly on November 18, 2013, 11:10:24 am

Title: Nightly (OS X): 18 Nov 2013 - Revision 10115
Post by: SirKnightly on November 18, 2013, 11:10:24 am
Here is the nightly for OS X on 18 Nov 2013 - Revision 10115

Group: Standard
fso-OSX-Standard-20131118_r10115.tgz (http://swc.fs2downloads.com/builds/OSX/fso-OSX-Standard-20131118_r10115.tgz)
MD5Sum (http://swc.fs2downloads.com/builds/OSX/fso-OSX-Standard-20131118_r10115.md5)

Code: [Select]
------------------------------------------------------------------------
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 ‘shipp’ 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
------------------------------------------------------------------------