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 |