View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0001511 | FSSCP | --------- | public | 2007-10-06 16:49 | 2007-10-12 19:14 |
Reporter | sefi | Assigned To | karajorma | ||
Priority | normal | Severity | trivial | Reproducibility | always |
Status | resolved | Resolution | fixed | ||
Fixed in Version | 3.6.10 | ||||
Summary | 0001511: Missing declarations in sexp.cpp | ||||
Description | There are three missing declarations in sexp.cpp. The attached patch fixes the issue. | ||||
Tags | No tags attached. | ||||
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) { |
|
Looks like I added the declaration in the 3.6.9 branch but missed it in HEAD. Fixed. |
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 |