author | erik <erik> | 2007-01-24 19:50:48 (UTC) |
---|---|---|
committer | erik <erik> | 2007-01-24 19:50:48 (UTC) |
commit | 035bbc5bf689839c8d8e7be37f347b0dd900fccf (patch) (unidiff) | |
tree | cd668db7b1fedbf6fa2d54fb312a90e6f157e82a | |
parent | a017bf21dd89159052f2f7a3fbc043a24956c08c (diff) | |
download | opie-035bbc5bf689839c8d8e7be37f347b0dd900fccf.zip opie-035bbc5bf689839c8d8e7be37f347b0dd900fccf.tar.gz opie-035bbc5bf689839c8d8e7be37f347b0dd900fccf.tar.bz2 |
Eac one the files in this commit had an instance where a code path is
never traversed because of conditionals operating on values that
never change.
-rw-r--r-- | noncore/apps/opie-reader/decompress.cpp | 7 | ||||
-rw-r--r-- | noncore/games/solitaire/canvascard.cpp | 2 | ||||
-rw-r--r-- | noncore/games/tetrix/ohighscoredlg.cpp | 9 |
3 files changed, 2 insertions, 16 deletions
diff --git a/noncore/apps/opie-reader/decompress.cpp b/noncore/apps/opie-reader/decompress.cpp index 9bfec49..6034e01 100644 --- a/noncore/apps/opie-reader/decompress.cpp +++ b/noncore/apps/opie-reader/decompress.cpp | |||
@@ -1,95 +1,88 @@ | |||
1 | #include <string.h> | 1 | #include <string.h> |
2 | #include "decompress.h" | 2 | #include "decompress.h" |
3 | #include <zlib.h> | 3 | #include <zlib.h> |
4 | #include <stdlib.h> | 4 | #include <stdlib.h> |
5 | 5 | ||
6 | size_t UnZip(UInt8* compressedbuffer, size_t reclen, UInt8* tgtbuffer, size_t bsize) | 6 | size_t UnZip(UInt8* compressedbuffer, size_t reclen, UInt8* tgtbuffer, size_t bsize) |
7 | { | 7 | { |
8 | z_stream zstream; | 8 | z_stream zstream; |
9 | memset(&zstream,sizeof(zstream),0); | 9 | memset(&zstream,sizeof(zstream),0); |
10 | zstream.next_in = compressedbuffer; | 10 | zstream.next_in = compressedbuffer; |
11 | zstream.next_out = tgtbuffer; | 11 | zstream.next_out = tgtbuffer; |
12 | zstream.avail_out = bsize; | 12 | zstream.avail_out = bsize; |
13 | zstream.avail_in = reclen; | 13 | zstream.avail_in = reclen; |
14 | 14 | ||
15 | int keylen = 0; | ||
16 | |||
17 | zstream.zalloc = Z_NULL; | 15 | zstream.zalloc = Z_NULL; |
18 | zstream.zfree = Z_NULL; | 16 | zstream.zfree = Z_NULL; |
19 | zstream.opaque = Z_NULL; | 17 | zstream.opaque = Z_NULL; |
20 | 18 | ||
21 | // printf("Initialising\n"); | 19 | // printf("Initialising\n"); |
22 | 20 | ||
23 | inflateInit(&zstream); | 21 | inflateInit(&zstream); |
24 | int err = 0; | 22 | int err = 0; |
25 | do { | 23 | do { |
26 | if ( zstream.avail_in == 0 && 0 < keylen ) { | ||
27 | zstream.next_in = compressedbuffer + keylen; | ||
28 | zstream.avail_in = reclen - keylen; | ||
29 | keylen = 0; | ||
30 | } | ||
31 | zstream.next_out = tgtbuffer; | 24 | zstream.next_out = tgtbuffer; |
32 | zstream.avail_out = bsize; | 25 | zstream.avail_out = bsize; |
33 | 26 | ||
34 | err = inflate( &zstream, Z_SYNC_FLUSH ); | 27 | err = inflate( &zstream, Z_SYNC_FLUSH ); |
35 | 28 | ||
36 | ////qDebug("err:%d - %u", err, zstream.avail_in); | 29 | ////qDebug("err:%d - %u", err, zstream.avail_in); |
37 | 30 | ||
38 | } while ( err == Z_OK ); | 31 | } while ( err == Z_OK ); |
39 | 32 | ||
40 | inflateEnd(&zstream); | 33 | inflateEnd(&zstream); |
41 | return zstream.total_out; | 34 | return zstream.total_out; |
42 | } | 35 | } |
43 | 36 | ||
44 | #if defined(__STATIC) && defined(USENEF) | 37 | #if defined(__STATIC) && defined(USENEF) |
45 | #include "Model.h" | 38 | #include "Model.h" |
46 | size_t (*getdecompressor(char* _s))(UInt8*, size_t, UInt8*, size_t) | 39 | size_t (*getdecompressor(char* _s))(UInt8*, size_t, UInt8*, size_t) |
47 | { | 40 | { |
48 | if (strcmp(_s, "PluckerDecompress3") == 0) | 41 | if (strcmp(_s, "PluckerDecompress3") == 0) |
49 | { | 42 | { |
50 | return PluckerDecompress3; | 43 | return PluckerDecompress3; |
51 | } | 44 | } |
52 | if (strcmp(_s, "PluckerDecompress4") == 0) | 45 | if (strcmp(_s, "PluckerDecompress4") == 0) |
53 | { | 46 | { |
54 | return PluckerDecompress4; | 47 | return PluckerDecompress4; |
55 | } | 48 | } |
56 | if (strcmp(_s, "RebDecompress") == 0) | 49 | if (strcmp(_s, "RebDecompress") == 0) |
57 | { | 50 | { |
58 | return RebDecompress; | 51 | return RebDecompress; |
59 | } | 52 | } |
60 | return NULL; | 53 | return NULL; |
61 | } | 54 | } |
62 | #else | 55 | #else |
63 | 56 | ||
64 | #include "qfileinfo.h" | 57 | #include "qfileinfo.h" |
65 | 58 | ||
66 | #include <dlfcn.h> | 59 | #include <dlfcn.h> |
67 | 60 | ||
68 | size_t (*getdecompressor(char* _s))(UInt8*, size_t, UInt8*, size_t) | 61 | size_t (*getdecompressor(char* _s))(UInt8*, size_t, UInt8*, size_t) |
69 | { | 62 | { |
70 | #ifdef USEQPE | 63 | #ifdef USEQPE |
71 | #ifdef OPIE | 64 | #ifdef OPIE |
72 | QString codecpath(getenv("OPIEDIR")); | 65 | QString codecpath(getenv("OPIEDIR")); |
73 | #else | 66 | #else |
74 | QString codecpath(getenv("QTDIR")); | 67 | QString codecpath(getenv("QTDIR")); |
75 | #endif | 68 | #endif |
76 | codecpath += "/plugins/reader/support/libpluckerdecompress.so"; | 69 | codecpath += "/plugins/reader/support/libpluckerdecompress.so"; |
77 | #else | 70 | #else |
78 | QString codecpath(getenv("READERDIR")); | 71 | QString codecpath(getenv("READERDIR")); |
79 | codecpath += "/support/libpluckerdecompress.so"; | 72 | codecpath += "/support/libpluckerdecompress.so"; |
80 | #endif | 73 | #endif |
81 | qDebug("Codec:%s", (const char*)codecpath); | 74 | qDebug("Codec:%s", (const char*)codecpath); |
82 | if (QFile::exists(codecpath)) | 75 | if (QFile::exists(codecpath)) |
83 | { | 76 | { |
84 | qDebug("Codec:%s", (const char*)codecpath); | 77 | qDebug("Codec:%s", (const char*)codecpath); |
85 | void* handle = dlopen(codecpath, RTLD_LAZY); | 78 | void* handle = dlopen(codecpath, RTLD_LAZY); |
86 | if (handle == 0) | 79 | if (handle == 0) |
87 | { | 80 | { |
88 | qDebug("Can't find codec:%s", dlerror()); | 81 | qDebug("Can't find codec:%s", dlerror()); |
89 | return NULL; | 82 | return NULL; |
90 | } | 83 | } |
91 | return (size_t (*)(UInt8*, size_t, UInt8*, size_t))dlsym(handle, _s); | 84 | return (size_t (*)(UInt8*, size_t, UInt8*, size_t))dlsym(handle, _s); |
92 | } | 85 | } |
93 | return NULL; | 86 | return NULL; |
94 | } | 87 | } |
95 | #endif | 88 | #endif |
diff --git a/noncore/games/solitaire/canvascard.cpp b/noncore/games/solitaire/canvascard.cpp index cd298ef..3ed9ea0 100644 --- a/noncore/games/solitaire/canvascard.cpp +++ b/noncore/games/solitaire/canvascard.cpp | |||
@@ -121,183 +121,183 @@ CanvasCard::CanvasCard( eValue v, eSuit s, bool f, QCanvas *canvas ) : | |||
121 | setSize( cardsFaces->width(), cardsFaces->height() ); | 121 | setSize( cardsFaces->width(), cardsFaces->height() ); |
122 | setPen( NoPen ); | 122 | setPen( NoPen ); |
123 | flipping = FALSE; | 123 | flipping = FALSE; |
124 | } | 124 | } |
125 | 125 | ||
126 | 126 | ||
127 | void CanvasCard::setCardBack(int b) | 127 | void CanvasCard::setCardBack(int b) |
128 | { | 128 | { |
129 | if ( cardBack != b ) { | 129 | if ( cardBack != b ) { |
130 | 130 | ||
131 | cardBack = b; | 131 | cardBack = b; |
132 | 132 | ||
133 | if ( cardsBacks ) | 133 | if ( cardsBacks ) |
134 | delete cardsBacks; | 134 | delete cardsBacks; |
135 | 135 | ||
136 | if ( qt_screen->deviceWidth() < 200 ) { | 136 | if ( qt_screen->deviceWidth() < 200 ) { |
137 | switch (cardBack) { | 137 | switch (cardBack) { |
138 | case 0: | 138 | case 0: |
139 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0001_small" ) ); break; | 139 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0001_small" ) ); break; |
140 | case 1: | 140 | case 1: |
141 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0002_small" ) ); break; | 141 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0002_small" ) ); break; |
142 | case 2: | 142 | case 2: |
143 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0003_small" ) ); break; | 143 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0003_small" ) ); break; |
144 | case 3: | 144 | case 3: |
145 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0004_small" ) ); break; | 145 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0004_small" ) ); break; |
146 | case 4: | 146 | case 4: |
147 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0005_small" ) ); break; | 147 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0005_small" ) ); break; |
148 | } | 148 | } |
149 | } else { | 149 | } else { |
150 | switch (cardBack) { | 150 | switch (cardBack) { |
151 | case 0: | 151 | case 0: |
152 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0001" ) ); break; | 152 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0001" ) ); break; |
153 | case 1: | 153 | case 1: |
154 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0002" ) ); break; | 154 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0002" ) ); break; |
155 | case 2: | 155 | case 2: |
156 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0003" ) ); break; | 156 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0003" ) ); break; |
157 | case 3: | 157 | case 3: |
158 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0004" ) ); break; | 158 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0004" ) ); break; |
159 | case 4: | 159 | case 4: |
160 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0005" ) ); break; | 160 | cardsBacks = new QPixmap( Opie::Core::OResource::loadPixmap( "cards/card_back0005" ) ); break; |
161 | } | 161 | } |
162 | } | 162 | } |
163 | 163 | ||
164 | if ( !isFacing() ) | 164 | if ( !isFacing() ) |
165 | redraw(); | 165 | redraw(); |
166 | } | 166 | } |
167 | } | 167 | } |
168 | 168 | ||
169 | 169 | ||
170 | void CanvasCard::draw(QPainter &painter) | 170 | void CanvasCard::draw(QPainter &painter) |
171 | { | 171 | { |
172 | int ix = (int)x(), iy = (int)y(); | 172 | int ix = (int)x(), iy = (int)y(); |
173 | 173 | ||
174 | QPainter *p = &painter; | 174 | QPainter *p = &painter; |
175 | QPixmap *unscaledCard = NULL; | 175 | QPixmap *unscaledCard = NULL; |
176 | 176 | ||
177 | if ((scaleX <= 0.98) || (scaleY <= 0.98)) | 177 | if ((scaleX <= 0.98) || (scaleY <= 0.98)) |
178 | { | 178 | { |
179 | p = new QPainter(); | 179 | p = new QPainter(); |
180 | unscaledCard = new QPixmap( cardsFaces->width(), cardsFaces->height() ); | 180 | unscaledCard = new QPixmap( cardsFaces->width(), cardsFaces->height() ); |
181 | p->begin(unscaledCard); | 181 | p->begin(unscaledCard); |
182 | ix = 0; | 182 | ix = 0; |
183 | iy = 0; | 183 | iy = 0; |
184 | } | 184 | } |
185 | 185 | ||
186 | if ( isFacing() ) { | 186 | if ( isFacing() ) { |
187 | 187 | ||
188 | int w = cardsFaces->width(), h = cardsFaces->height(); | 188 | int w = cardsFaces->width(), h = cardsFaces->height(); |
189 | 189 | ||
190 | p->setBrush( QColor( 0xFF, 0xFF, 0xFF ) ); | 190 | p->setBrush( QColor( 0xFF, 0xFF, 0xFF ) ); |
191 | 191 | ||
192 | if ( isRed() == TRUE ) | 192 | if ( isRed() == TRUE ) |
193 | p->setPen( QColor( 0xFF, 0, 0 ) ); | 193 | p->setPen( QColor( 0xFF, 0, 0 ) ); |
194 | else | 194 | else |
195 | p->setPen( QColor( 0, 0, 0 ) ); | 195 | p->setPen( QColor( 0, 0, 0 ) ); |
196 | 196 | ||
197 | if ( qt_screen->deviceWidth() < 200 ) { | 197 | if ( qt_screen->deviceWidth() < 200 ) { |
198 | p->drawPixmap( ix + 0, iy + 0, *cardsFaces ); | 198 | p->drawPixmap( ix + 0, iy + 0, *cardsFaces ); |
199 | p->drawPixmap( ix + 3, iy + 3, *cardsChars, 5*(getValue()-1), 0, 5, 6 ); | 199 | p->drawPixmap( ix + 3, iy + 3, *cardsChars, 5*(getValue()-1), 0, 5, 6 ); |
200 | p->drawPixmap( ix + 11, iy + 3, *cardsSuits, 5*(getSuit()-1), 0, 5, 6 ); | 200 | p->drawPixmap( ix + 11, iy + 3, *cardsSuits, 5*(getSuit()-1), 0, 5, 6 ); |
201 | p->drawPixmap( ix + w-3-5, iy + h-3-6, *cardsCharsUpsideDown, 5*(12-getValue()+1), 0, 5, 6 ); | 201 | p->drawPixmap( ix + w-3-5, iy + h-3-6, *cardsCharsUpsideDown, 5*(12-getValue()+1), 0, 5, 6 ); |
202 | p->drawPixmap( ix + w-11-5, iy + h-3-6, *cardsSuitsUpsideDown, 5*(3-getSuit()+1), 0, 5, 6 ); | 202 | p->drawPixmap( ix + w-11-5, iy + h-3-6, *cardsSuitsUpsideDown, 5*(3-getSuit()+1), 0, 5, 6 ); |
203 | } else { | 203 | } else { |
204 | p->drawPixmap( ix + 0, iy + 0, *cardsFaces ); | 204 | p->drawPixmap( ix + 0, iy + 0, *cardsFaces ); |
205 | p->drawPixmap( ix + 4, iy + 4, *cardsChars, 7*(getValue()-1), 0, 7, 7 ); | 205 | p->drawPixmap( ix + 4, iy + 4, *cardsChars, 7*(getValue()-1), 0, 7, 7 ); |
206 | p->drawPixmap( ix + 12, iy + 4, *cardsSuits, 7*(getSuit()-1), 0, 7, 8 ); | 206 | p->drawPixmap( ix + 12, iy + 4, *cardsSuits, 7*(getSuit()-1), 0, 7, 8 ); |
207 | p->drawPixmap( ix + w-4-7, iy + h-4-7, *cardsCharsUpsideDown, 7*(12-getValue()+1), 0, 7, 7 ); | 207 | p->drawPixmap( ix + w-4-7, iy + h-4-7, *cardsCharsUpsideDown, 7*(12-getValue()+1), 0, 7, 7 ); |
208 | p->drawPixmap( ix + w-12-7, iy + h-5-7, *cardsSuitsUpsideDown, 7*(3-getSuit()+1), 0, 7, 8 ); | 208 | p->drawPixmap( ix + w-12-7, iy + h-5-7, *cardsSuitsUpsideDown, 7*(3-getSuit()+1), 0, 7, 8 ); |
209 | } | 209 | } |
210 | 210 | ||
211 | } else { | 211 | } else { |
212 | 212 | ||
213 | p->drawPixmap( ix, iy, *cardsBacks ); | 213 | p->drawPixmap( ix, iy, *cardsBacks ); |
214 | 214 | ||
215 | } | 215 | } |
216 | 216 | ||
217 | if (p != &painter) | 217 | if (p != &painter && unscaledCard) |
218 | { | 218 | { |
219 | p->end(); | 219 | p->end(); |
220 | QPixmap *scaledCard = CreateScaledPixmap( unscaledCard, scaleX, scaleY ); | 220 | QPixmap *scaledCard = CreateScaledPixmap( unscaledCard, scaleX, scaleY ); |
221 | int xoff = scaledCard->width() / 2; | 221 | int xoff = scaledCard->width() / 2; |
222 | int yoff = scaledCard->height() / 2; | 222 | int yoff = scaledCard->height() / 2; |
223 | painter.drawPixmap( (int)x() + xOff - xoff, (int)y() + yOff - yoff, *scaledCard ); | 223 | painter.drawPixmap( (int)x() + xOff - xoff, (int)y() + yOff - yoff, *scaledCard ); |
224 | delete p; | 224 | delete p; |
225 | delete unscaledCard; | 225 | delete unscaledCard; |
226 | delete scaledCard; | 226 | delete scaledCard; |
227 | } | 227 | } |
228 | } | 228 | } |
229 | 229 | ||
230 | 230 | ||
231 | static const double flipLift = 1.5; | 231 | static const double flipLift = 1.5; |
232 | 232 | ||
233 | 233 | ||
234 | void CanvasCard::flipTo(int x2, int y2, int steps) | 234 | void CanvasCard::flipTo(int x2, int y2, int steps) |
235 | { | 235 | { |
236 | flipSteps = steps; | 236 | flipSteps = steps; |
237 | 237 | ||
238 | #ifdef SLOW_HARDWARE | 238 | #ifdef SLOW_HARDWARE |
239 | move(x2,y2); | 239 | move(x2,y2); |
240 | Card::flipTo(x2,y2,steps); | 240 | Card::flipTo(x2,y2,steps); |
241 | #else | 241 | #else |
242 | int x1 = (int)x(); | 242 | int x1 = (int)x(); |
243 | int y1 = (int)y(); | 243 | int y1 = (int)y(); |
244 | double dx = x2 - x1; | 244 | double dx = x2 - x1; |
245 | double dy = y2 - y1; | 245 | double dy = y2 - y1; |
246 | 246 | ||
247 | flipping = TRUE; | 247 | flipping = TRUE; |
248 | destX = x2; | 248 | destX = x2; |
249 | destY = y2; | 249 | destY = y2; |
250 | animSteps = flipSteps; | 250 | animSteps = flipSteps; |
251 | setVelocity(dx/animSteps, dy/animSteps-flipLift); | 251 | setVelocity(dx/animSteps, dy/animSteps-flipLift); |
252 | setAnimated(TRUE); | 252 | setAnimated(TRUE); |
253 | #endif | 253 | #endif |
254 | } | 254 | } |
255 | 255 | ||
256 | 256 | ||
257 | void CanvasCard::advance(int stage) | 257 | void CanvasCard::advance(int stage) |
258 | { | 258 | { |
259 | if ( stage==1 ) { | 259 | if ( stage==1 ) { |
260 | if ( animSteps-- <= 0 ) { | 260 | if ( animSteps-- <= 0 ) { |
261 | scaleX = 1.0; | 261 | scaleX = 1.0; |
262 | scaleY = 1.0; | 262 | scaleY = 1.0; |
263 | flipping = FALSE; | 263 | flipping = FALSE; |
264 | setVelocity(0,0); | 264 | setVelocity(0,0); |
265 | setAnimated(FALSE); | 265 | setAnimated(FALSE); |
266 | move(destX,destY); // exact | 266 | move(destX,destY); // exact |
267 | } else { | 267 | } else { |
268 | if ( flipping ) { | 268 | if ( flipping ) { |
269 | if ( animSteps > flipSteps / 2 ) { | 269 | if ( animSteps > flipSteps / 2 ) { |
270 | // animSteps = flipSteps .. flipSteps/2 (flip up) -> 1..0 | 270 | // animSteps = flipSteps .. flipSteps/2 (flip up) -> 1..0 |
271 | scaleX = ((double)animSteps/flipSteps-0.5)*2; | 271 | scaleX = ((double)animSteps/flipSteps-0.5)*2; |
272 | } else { | 272 | } else { |
273 | // animSteps = flipSteps/2 .. 0 (flip down) -> 0..1 | 273 | // animSteps = flipSteps/2 .. 0 (flip down) -> 0..1 |
274 | scaleX = 1-((double)animSteps/flipSteps)*2; | 274 | scaleX = 1-((double)animSteps/flipSteps)*2; |
275 | } | 275 | } |
276 | if ( animSteps == flipSteps / 2-1 ) { | 276 | if ( animSteps == flipSteps / 2-1 ) { |
277 | setYVelocity(yVelocity()+flipLift*2); | 277 | setYVelocity(yVelocity()+flipLift*2); |
278 | setFace( !isFacing() ); | 278 | setFace( !isFacing() ); |
279 | } | 279 | } |
280 | } | 280 | } |
281 | } | 281 | } |
282 | } | 282 | } |
283 | QCanvasRectangle::advance(stage); | 283 | QCanvasRectangle::advance(stage); |
284 | } | 284 | } |
285 | 285 | ||
286 | 286 | ||
287 | void CanvasCard::animatedMove(int x2, int y2, int steps) | 287 | void CanvasCard::animatedMove(int x2, int y2, int steps) |
288 | { | 288 | { |
289 | destX = x2; | 289 | destX = x2; |
290 | destY = y2; | 290 | destY = y2; |
291 | 291 | ||
292 | double x1 = x(), y1 = y(), dx = x2 - x1, dy = y2 - y1; | 292 | double x1 = x(), y1 = y(), dx = x2 - x1, dy = y2 - y1; |
293 | 293 | ||
294 | // Ensure a good speed | 294 | // Ensure a good speed |
295 | while ( fabs(dx/steps)+fabs(dy/steps) < 5.0 && steps > 4 ) | 295 | while ( fabs(dx/steps)+fabs(dy/steps) < 5.0 && steps > 4 ) |
296 | steps--; | 296 | steps--; |
297 | 297 | ||
298 | setAnimated(TRUE); | 298 | setAnimated(TRUE); |
299 | setVelocity(dx/steps, dy/steps); | 299 | setVelocity(dx/steps, dy/steps); |
300 | 300 | ||
301 | animSteps = steps; | 301 | animSteps = steps; |
302 | } | 302 | } |
303 | 303 | ||
diff --git a/noncore/games/tetrix/ohighscoredlg.cpp b/noncore/games/tetrix/ohighscoredlg.cpp index 66f4917..504385f 100644 --- a/noncore/games/tetrix/ohighscoredlg.cpp +++ b/noncore/games/tetrix/ohighscoredlg.cpp | |||
@@ -81,126 +81,119 @@ void OHighscore::getList() | |||
81 | pPlayerData->level = 0; | 81 | pPlayerData->level = 0; |
82 | 82 | ||
83 | playerData.append( pPlayerData ); | 83 | playerData.append( pPlayerData ); |
84 | } | 84 | } |
85 | } | 85 | } |
86 | 86 | ||
87 | } | 87 | } |
88 | 88 | ||
89 | void OHighscore::checkIfItIsANewhighscore( int points) | 89 | void OHighscore::checkIfItIsANewhighscore( int points) |
90 | { | 90 | { |
91 | if ( points > lowest ) | 91 | if ( points > lowest ) |
92 | isNewhighscore = true; | 92 | isNewhighscore = true; |
93 | else | 93 | else |
94 | isNewhighscore = false; | 94 | isNewhighscore = false; |
95 | } | 95 | } |
96 | 96 | ||
97 | void OHighscore::insertData( QString name , int punkte , int playerLevel ) | 97 | void OHighscore::insertData( QString name , int punkte , int playerLevel ) |
98 | { | 98 | { |
99 | Config cfg ( "tetrix" ); | 99 | Config cfg ( "tetrix" ); |
100 | t_playerData * Run; | 100 | t_playerData * Run; |
101 | int index = 0; | 101 | int index = 0; |
102 | int entryNumber = 1; | 102 | int entryNumber = 1; |
103 | 103 | ||
104 | for ( Run=playerData.first(); | 104 | for ( Run=playerData.first(); |
105 | Run != 0; | 105 | Run != 0; |
106 | index ++, Run=playerData.next() ) { | 106 | index ++, Run=playerData.next() ) { |
107 | 107 | ||
108 | if ( punkte > Run->points ) | 108 | if ( punkte > Run->points ) |
109 | { | 109 | { |
110 | t_playerData* temp = new t_playerData; | 110 | t_playerData* temp = new t_playerData; |
111 | temp->sName = name; | 111 | temp->sName = name; |
112 | temp->points = punkte; | 112 | temp->points = punkte; |
113 | temp->level = playerLevel; | 113 | temp->level = playerLevel; |
114 | 114 | ||
115 | playerData.insert( index, temp ); | 115 | playerData.insert( index, temp ); |
116 | 116 | ||
117 | //now we have to delete the last entry | 117 | //now we have to delete the last entry |
118 | playerData.remove( playerData.count() ); | 118 | playerData.remove( playerData.count() ); |
119 | 119 | ||
120 | ///////////////////////////////////////// | 120 | ///////////////////////////////////////// |
121 | //this block just rewrites the highscore | 121 | //this block just rewrites the highscore |
122 | for ( t_playerData * Run2=playerData.first(); | 122 | for ( t_playerData * Run2=playerData.first(); |
123 | Run2 != 0; | 123 | Run2 != 0; |
124 | Run2=playerData.next() ) { | 124 | Run2=playerData.next() ) { |
125 | cfg.setGroup( QString::number( entryNumber ) ); | 125 | cfg.setGroup( QString::number( entryNumber ) ); |
126 | cfg.writeEntry( "Name" , Run2->sName ); | 126 | cfg.writeEntry( "Name" , Run2->sName ); |
127 | cfg.writeEntry( "Points" , Run2->points ); | 127 | cfg.writeEntry( "Points" , Run2->points ); |
128 | cfg.writeEntry( "Level" , Run2->level ); | 128 | cfg.writeEntry( "Level" , Run2->level ); |
129 | entryNumber++; | 129 | entryNumber++; |
130 | } | 130 | } |
131 | //////////////////////////////////////// | 131 | //////////////////////////////////////// |
132 | 132 | ||
133 | return; | 133 | return; |
134 | } | 134 | } |
135 | } | 135 | } |
136 | } | 136 | } |
137 | 137 | ||
138 | QString OHighscore::getName() | 138 | QString OHighscore::getName() |
139 | { | 139 | { |
140 | QString name; | 140 | QString name; |
141 | QDialog *d = new QDialog ( this, 0, true ); | 141 | QDialog *d = new QDialog ( this, 0, true ); |
142 | d->setCaption( tr( "Enter your name!" )); | 142 | d->setCaption( tr( "Enter your name!" )); |
143 | QLineEdit *ed = new QLineEdit ( d ); | 143 | QLineEdit *ed = new QLineEdit ( d ); |
144 | ( new QVBoxLayout ( d, 3, 3 ))->addWidget ( ed ); | 144 | ( new QVBoxLayout ( d, 3, 3 ))->addWidget ( ed ); |
145 | ed->setFocus ( ); | 145 | ed->setFocus ( ); |
146 | 146 | ||
147 | if ( d->exec() == QDialog::Accepted ) { | 147 | if ( d->exec() == QDialog::Accepted ) { |
148 | name = ed->text(); | 148 | name = ed->text(); |
149 | } | 149 | } |
150 | //delete d; | 150 | //delete d; |
151 | return name; | 151 | return name; |
152 | } | 152 | } |
153 | 153 | ||
154 | OHighscoreDialog::OHighscoreDialog(OHighscore *highscore, QWidget *parent, const char *name, bool modal) : QDialog(parent, name, modal) | 154 | OHighscoreDialog::OHighscoreDialog(OHighscore *highscore, QWidget *parent, const char *name, bool modal) : QDialog(parent, name, modal) |
155 | { | 155 | { |
156 | hs_ = highscore; | 156 | hs_ = highscore; |
157 | setCaption( tr( "Highscores" ) ); | 157 | setCaption( tr( "Highscores" ) ); |
158 | vbox_layout = new QVBoxLayout( this, 4 , 4 ); | 158 | vbox_layout = new QVBoxLayout( this, 4 , 4 ); |
159 | list = new QListView( this ); | 159 | list = new QListView( this ); |
160 | list->setSorting( -1 ); | 160 | list->setSorting( -1 ); |
161 | list->addColumn( tr( "#" )); | 161 | list->addColumn( tr( "#" )); |
162 | list->addColumn( tr( "Name" )); | 162 | list->addColumn( tr( "Name" )); |
163 | list->addColumn( tr( "Points" )); | 163 | list->addColumn( tr( "Points" )); |
164 | list->addColumn( tr( "Level" )); | 164 | list->addColumn( tr( "Level" )); |
165 | 165 | ||
166 | createHighscoreListView(); | 166 | createHighscoreListView(); |
167 | 167 | ||
168 | vbox_layout->addWidget( list ); | 168 | vbox_layout->addWidget( list ); |
169 | QPEApplication::showDialog( this ); | 169 | QPEApplication::showDialog( this ); |
170 | } | 170 | } |
171 | 171 | ||
172 | void OHighscoreDialog::createHighscoreListView() | 172 | void OHighscoreDialog::createHighscoreListView() |
173 | { | 173 | { |
174 | int pos = 10; | 174 | int pos = 10; |
175 | int points_ = 0; | 175 | int points_ = 0; |
176 | int level_ = 0; | 176 | int level_ = 0; |
177 | QListViewItem * Prev = 0; | ||
178 | 177 | ||
179 | for ( t_playerData * Run = hs_->playerData.first(); | 178 | for ( t_playerData * Run = hs_->playerData.first(); |
180 | Run != 0; | 179 | Run != 0; |
181 | Run=hs_->playerData.next() ) | 180 | Run=hs_->playerData.next() ) |
182 | { | 181 | { |
183 | QListViewItem *item; | 182 | QListViewItem *item; |
184 | 183 | ||
185 | if( Prev ) { | 184 | item = new QListViewItem( list ); |
186 | // after previous | ||
187 | item = new QListViewItem( list, Prev ); | ||
188 | Prev = item; | ||
189 | } else { | ||
190 | item = new QListViewItem( list ); | ||
191 | 185 | ||
192 | } | ||
193 | item->setText( 0 , QString::number( pos ) ); //number | 186 | item->setText( 0 , QString::number( pos ) ); //number |
194 | item->setText( 1 , Run->sName ); //name | 187 | item->setText( 1 , Run->sName ); //name |
195 | if ( Run->points == -1 ) | 188 | if ( Run->points == -1 ) |
196 | points_ = 0; | 189 | points_ = 0; |
197 | else points_ = Run->points; | 190 | else points_ = Run->points; |
198 | if ( Run->level == -1 ) | 191 | if ( Run->level == -1 ) |
199 | level_ = 0; | 192 | level_ = 0; |
200 | else level_ = Run->level; | 193 | else level_ = Run->level; |
201 | item->setText( 2 , QString::number( points_ ) ); //points | 194 | item->setText( 2 , QString::number( points_ ) ); //points |
202 | item->setText( 3 , QString::number( level_ ) ); //level | 195 | item->setText( 3 , QString::number( level_ ) ); //level |
203 | pos--; | 196 | pos--; |
204 | } | 197 | } |
205 | } | 198 | } |
206 | 199 | ||