Lines 1-219
Link Here
|
1 |
# Origin: https://github.com/naev/naev/commit/74b9a086a20f80f21ce2f3866b31318e651a6235 |
|
|
2 |
# Subject: Turn player_missions in an array of pointers |
3 |
|
4 |
--- src/mission.c.orig 2015-02-16 17:49:03 UTC |
5 |
+++ src/mission.c |
6 |
@@ -50,7 +50,7 @@ |
7 |
* current player missions |
8 |
*/ |
9 |
static unsigned int mission_id = 0; /**< Mission ID generator. */ |
10 |
-Mission player_missions[MISSION_MAX]; /**< Player's active missions. */ |
11 |
+Mission *player_missions[MISSION_MAX]; /**< Player's active missions. */ |
12 |
|
13 |
|
14 |
/* |
15 |
@@ -95,7 +95,7 @@ static unsigned int mission_genID (void) |
16 |
|
17 |
/* we save mission ids, so check for collisions with player's missions */ |
18 |
for (i=0; i<MISSION_MAX; i++) |
19 |
- if (id == player_missions[i].id) /* mission id was loaded from save */ |
20 |
+ if (id == player_missions[i]->id) /* mission id was loaded from save */ |
21 |
return mission_genID(); /* recursively try again */ |
22 |
return id; |
23 |
} |
24 |
@@ -240,7 +240,7 @@ int mission_alreadyRunning( MissionData* |
25 |
{ |
26 |
int i; |
27 |
for (i=0; i<MISSION_MAX; i++) |
28 |
- if (player_missions[i].data==misn) |
29 |
+ if (player_missions[i]->data == misn) |
30 |
return 1; |
31 |
return 0; |
32 |
} |
33 |
@@ -416,15 +416,15 @@ void mission_sysMark (void) |
34 |
|
35 |
for (i=0; i<MISSION_MAX; i++) { |
36 |
/* Must be a valid player mission. */ |
37 |
- if (player_missions[i].id == 0) |
38 |
+ if (player_missions[i]->id == 0) |
39 |
continue; |
40 |
/* Must have markers. */ |
41 |
- if (player_missions[i].markers == NULL) |
42 |
+ if (player_missions[i]->markers == NULL) |
43 |
continue; |
44 |
|
45 |
- n = array_size( player_missions[i].markers ); |
46 |
+ n = array_size( player_missions[i]->markers ); |
47 |
for (j=0; j<n; j++) { |
48 |
- m = &player_missions[i].markers[j]; |
49 |
+ m = &player_missions[i]->markers[j]; |
50 |
|
51 |
/* Add the individual markers. */ |
52 |
space_addMarker( m->sys, m->type ); |
53 |
@@ -562,6 +562,30 @@ void mission_cleanup( Mission* misn ) |
54 |
|
55 |
|
56 |
/** |
57 |
+ * @brief Puts the specified mission at the end of the player_missions array. |
58 |
+ * |
59 |
+ * @param pos Mission's position within player_missions |
60 |
+ */ |
61 |
+void mission_shift( int pos ) |
62 |
+{ |
63 |
+ Mission *misn; |
64 |
+ |
65 |
+ if (pos >= (MISSION_MAX-1)) |
66 |
+ return; |
67 |
+ |
68 |
+ /* Store specified mission. */ |
69 |
+ misn = player_missions[pos]; |
70 |
+ |
71 |
+ /* Move other missions down. */ |
72 |
+ memmove( &player_missions[pos], &player_missions[pos+1], |
73 |
+ sizeof(Mission*) * (MISSION_MAX - pos - 1) ); |
74 |
+ |
75 |
+ /* Put the specified mission at the end of the array. */ |
76 |
+ player_missions[MISSION_MAX - 1] = misn; |
77 |
+} |
78 |
+ |
79 |
+ |
80 |
+/** |
81 |
* @brief Frees MissionData. |
82 |
* |
83 |
* @param mission MissionData to free. |
84 |
@@ -622,8 +646,8 @@ void missions_activateClaims (void) |
85 |
int i; |
86 |
|
87 |
for (i=0; i<MISSION_MAX; i++) |
88 |
- if (player_missions[i].claims != NULL) |
89 |
- claim_activate( player_missions[i].claims ); |
90 |
+ if (player_missions[i]->claims != NULL) |
91 |
+ claim_activate( player_missions[i]->claims ); |
92 |
} |
93 |
|
94 |
|
95 |
@@ -869,9 +893,14 @@ static int mission_parse( MissionData* t |
96 |
*/ |
97 |
int missions_load (void) |
98 |
{ |
99 |
- int m; |
100 |
+ int i, m; |
101 |
uint32_t bufsize; |
102 |
- char *buf = ndata_read( MISSION_DATA_PATH, &bufsize ); |
103 |
+ char *buf; |
104 |
+ |
105 |
+ for (i=0; i<MISSION_MAX; i++) |
106 |
+ player_missions[i] = calloc(1, sizeof(Mission)); |
107 |
+ |
108 |
+ buf = ndata_read( MISSION_DATA_PATH, &bufsize ); |
109 |
|
110 |
xmlNodePtr node; |
111 |
xmlDocPtr doc = xmlParseMemory( buf, bufsize ); |
112 |
@@ -933,6 +962,10 @@ void missions_free (void) |
113 |
free( mission_stack ); |
114 |
mission_stack = NULL; |
115 |
mission_nstack = 0; |
116 |
+ |
117 |
+ /* Free the player mission stack. */ |
118 |
+ for (i=0; i<MISSION_MAX; i++) |
119 |
+ free(player_missions[i]); |
120 |
} |
121 |
|
122 |
|
123 |
@@ -944,7 +977,7 @@ void missions_cleanup (void) |
124 |
int i; |
125 |
|
126 |
for (i=0; i<MISSION_MAX; i++) |
127 |
- mission_cleanup( &player_missions[i] ); |
128 |
+ mission_cleanup( player_missions[i] ); |
129 |
} |
130 |
|
131 |
|
132 |
@@ -963,26 +996,26 @@ int missions_saveActive( xmlTextWriterPt |
133 |
xmlw_startElem(writer,"missions"); |
134 |
|
135 |
for (i=0; i<MISSION_MAX; i++) { |
136 |
- if (player_missions[i].id != 0) { |
137 |
+ if (player_missions[i]->id != 0) { |
138 |
xmlw_startElem(writer,"mission"); |
139 |
|
140 |
/* data and id are attributes because they must be loaded first */ |
141 |
- xmlw_attr(writer,"data","%s",player_missions[i].data->name); |
142 |
- xmlw_attr(writer,"id","%u",player_missions[i].id); |
143 |
+ xmlw_attr(writer,"data","%s",player_missions[i]->data->name); |
144 |
+ xmlw_attr(writer,"id","%u",player_missions[i]->id); |
145 |
|
146 |
- xmlw_elem(writer,"title","%s",player_missions[i].title); |
147 |
- xmlw_elem(writer,"desc","%s",player_missions[i].desc); |
148 |
- xmlw_elem(writer,"reward","%s",player_missions[i].reward); |
149 |
+ xmlw_elem(writer,"title","%s",player_missions[i]->title); |
150 |
+ xmlw_elem(writer,"desc","%s",player_missions[i]->desc); |
151 |
+ xmlw_elem(writer,"reward","%s",player_missions[i]->reward); |
152 |
|
153 |
/* Markers. */ |
154 |
xmlw_startElem( writer, "markers" ); |
155 |
- if (player_missions[i].markers != NULL) { |
156 |
- n = array_size( player_missions[i].markers ); |
157 |
+ if (player_missions[i]->markers != NULL) { |
158 |
+ n = array_size( player_missions[i]->markers ); |
159 |
for (j=0; j<n; j++) { |
160 |
xmlw_startElem(writer,"marker"); |
161 |
- xmlw_attr(writer,"id","%d",player_missions[i].markers[j].id); |
162 |
- xmlw_attr(writer,"type","%d",player_missions[i].markers[j].type); |
163 |
- xmlw_str(writer,"%s", system_getIndex(player_missions[i].markers[j].sys)->name); |
164 |
+ xmlw_attr(writer,"id","%d",player_missions[i]->markers[j].id); |
165 |
+ xmlw_attr(writer,"type","%d",player_missions[i]->markers[j].type); |
166 |
+ xmlw_str(writer,"%s", system_getIndex(player_missions[i]->markers[j].sys)->name); |
167 |
xmlw_endElem(writer); /* "marker" */ |
168 |
} |
169 |
} |
170 |
@@ -990,19 +1023,19 @@ int missions_saveActive( xmlTextWriterPt |
171 |
|
172 |
/* Cargo */ |
173 |
xmlw_startElem(writer,"cargos"); |
174 |
- for (j=0; j<player_missions[i].ncargo; j++) |
175 |
- xmlw_elem(writer,"cargo","%u", player_missions[i].cargo[j]); |
176 |
+ for (j=0; j<player_missions[i]->ncargo; j++) |
177 |
+ xmlw_elem(writer,"cargo","%u", player_missions[i]->cargo[j]); |
178 |
xmlw_endElem(writer); /* "cargos" */ |
179 |
|
180 |
/* OSD. */ |
181 |
- if (player_missions[i].osd > 0) { |
182 |
+ if (player_missions[i]->osd > 0) { |
183 |
xmlw_startElem(writer,"osd"); |
184 |
|
185 |
/* Save attributes. */ |
186 |
- items = osd_getItems(player_missions[i].osd, &nitems); |
187 |
- xmlw_attr(writer,"title","%s",osd_getTitle(player_missions[i].osd)); |
188 |
+ items = osd_getItems(player_missions[i]->osd, &nitems); |
189 |
+ xmlw_attr(writer,"title","%s",osd_getTitle(player_missions[i]->osd)); |
190 |
xmlw_attr(writer,"nitems","%d",nitems); |
191 |
- xmlw_attr(writer,"active","%d",osd_getActive(player_missions[i].osd)); |
192 |
+ xmlw_attr(writer,"active","%d",osd_getActive(player_missions[i]->osd)); |
193 |
|
194 |
/* Save messages. */ |
195 |
for (j=0; j<nitems; j++) |
196 |
@@ -1013,12 +1046,12 @@ int missions_saveActive( xmlTextWriterPt |
197 |
|
198 |
/* Claims. */ |
199 |
xmlw_startElem(writer,"claims"); |
200 |
- claim_xmlSave( writer, player_missions[i].claims ); |
201 |
+ claim_xmlSave( writer, player_missions[i]->claims ); |
202 |
xmlw_endElem(writer); /* "claims" */ |
203 |
|
204 |
/* Write Lua magic */ |
205 |
xmlw_startElem(writer,"lua"); |
206 |
- nxml_persistLua( player_missions[i].L, writer ); |
207 |
+ nxml_persistLua( player_missions[i]->L, writer ); |
208 |
xmlw_endElem(writer); /* "lua" */ |
209 |
|
210 |
xmlw_endElem(writer); /* "mission" */ |
211 |
@@ -1078,7 +1111,7 @@ static int missions_parseActive( xmlNode |
212 |
node = parent->xmlChildrenNode; |
213 |
do { |
214 |
if (xml_isNode(node,"mission")) { |
215 |
- misn = &player_missions[m]; |
216 |
+ misn = player_missions[m]; |
217 |
|
218 |
/* process the attributes to create the mission */ |
219 |
xmlr_attr(node,"data",buf); |