View Issue Details

IDProjectCategoryView StatusLast Update
0001511FSSCP---------public2007-10-12 19:14
Reportersefi Assigned Tokarajorma  
PrioritynormalSeveritytrivialReproducibilityalways
Status resolvedResolutionfixed 
Fixed in Version3.6.10 
Summary0001511: Missing declarations in sexp.cpp
DescriptionThere are three missing declarations in sexp.cpp. The attached patch fixes the issue.
TagsNo tags attached.

Activities

2007-10-06 16:49

 

fs2_build.patch (843 bytes)   
--- code/parse/sexp.cpp	2007-10-06 14:09:53.000000000 +0200
+++ code/parse/sexp.cpp	2007-10-06 14:16:33.000000000 +0200
@@ -10600,9 +10600,9 @@ void sexp_toggle_builtin_messages (int n
 		{
 			// Since trying to determine whose wingman in a stand alone multiplayer game opens a can of worms
 			// Any Wingman silences all ships in wings regardless of whose side they're on. 
-			for (wingnum = 0; wingnum < Num_wings; wingnum++ ) {
-				for ( shipnum = 0; shipnum < Wings[wingnum].current_count; shipnum++ ) {
-					ship_index = Wings[wingnum].ship_index[shipnum];
+			for (unsigned wingnum = 0; wingnum < Num_wings; wingnum++ ) {
+				for (unsigned shipnum = 0; shipnum < Wings[wingnum].current_count; shipnum++ ) {
+					int ship_index = Wings[wingnum].ship_index[shipnum];
 					Assert( ship_index != -1 );
 
 					if (enable_messages) {
fs2_build.patch (843 bytes)   

karajorma

2007-10-12 19:14

administrator   ~0008559

Looks like I added the declaration in the 3.6.9 branch but missed it in HEAD. Fixed.

Issue History

Date Modified Username Field Change
2007-10-06 16:49 sefi New Issue
2007-10-06 16:49 sefi File Added: fs2_build.patch
2007-10-12 19:00 karajorma Status new => assigned
2007-10-12 19:00 karajorma Assigned To => karajorma
2007-10-12 19:14 karajorma Status assigned => resolved
2007-10-12 19:14 karajorma Fixed in Version => 3.6.10
2007-10-12 19:14 karajorma Resolution open => fixed
2007-10-12 19:14 karajorma Note Added: 0008559