Enno Rehling
26b6ae4e30
github issue #374
...
stick some assertions in natural_armor
refactor it for readability
eliminate a static memory leak
reset num_races to zero at end of test
2015-11-07 18:21:47 +01:00
Enno Rehling
bb0fd4d810
Fehlende Initializierung, github #375
2015-11-07 18:20:37 +01:00
Enno Rehling
d30154f3a5
stick some assertions in natural_armor
...
refactor it for readability
eliminate a static memory leak
reset num_races to zero at end of test
2015-11-07 18:14:38 +01:00
Enno Rehling
d780988862
valgrind the unit tests for good measure
2015-11-07 18:04:11 +01:00
Philipp Dreher
43b2e30c46
added initial cleanup
2015-11-07 15:18:23 +01:00
Philipp Dreher
4d622224aa
added message cleanup
2015-11-07 14:47:14 +01:00
Philipp Dreher
5e85945c94
fixed the test so it finally behaves correctly
2015-11-07 14:27:48 +01:00
Enno Rehling
62262ebcac
Merge pull request #373 from ennorehling/develop
...
clean up item tests so they don't use test_create_world
2015-11-07 14:06:08 +01:00
Enno Rehling
2f8dc29396
clean up item tests so they don't use test_create_world
...
fix intermittent skillpotion test
2015-11-07 13:52:18 +01:00
Philipp Dreher
663d947b2a
New behaviour in case of low skill during building
...
Insufficient skill after hitting the next improvement stage is now
handled by leaving the for-loop and skipping to after-build-processing.
2015-11-06 21:04:35 +01:00
Philipp Dreher
4f5a02a1a3
new lua-test for castle-stages
...
(Messed up the previous commit)
New lua-test for the behaviour concerning the castle-stages and their
min-skills. (Bug 1859)
2015-11-06 20:59:45 +01:00
Philipp Dreher
b0a986e155
Revert "added a lua-test"
...
This reverts commit a452ad1517
2015-11-06 20:57:19 +01:00
Philipp Dreher
a452ad1517
added a lua-test
...
New lua-test for the bug concerning the minskill for castle-stages.
(#1859 )
2015-11-06 20:53:10 +01:00
Philipp Dreher
be5d81acd1
Revert "new test"
...
This reverts commit cb16315e7c
2015-11-06 20:51:38 +01:00
Philipp Dreher
cb16315e7c
new test
...
Added test for the handling of castles' buildstages and their individual
skillcaps.
2015-11-06 13:14:19 +01:00
Philipp Dreher
757bff87d3
slight code improvement
...
(test still not working as it should though)
2015-11-06 12:08:12 +01:00
Enno Rehling
d29429f309
Merge pull request #369 from ennorehling/coverity-leaks
...
make it easier for coverity to understand this code
2015-11-06 10:37:48 +01:00
Philipp Dreher
9e2d2b1560
corrected some mistakes in setup and cleanup of the test
2015-11-05 20:10:51 +01:00
Philipp Dreher
62805cdfa3
added new test to check the msg-behaviour
2015-11-05 15:16:52 +01:00
Enno Rehling
4ae94eaa3f
fix compilation
2015-11-05 14:06:26 +01:00
Enno Rehling
40651d57cd
22583 Time of check time of use
...
avoid use of access()
2015-11-05 14:05:58 +01:00
Enno Rehling
5f99288af1
CID 22581 Time of check time of use
...
CID 22582 Time of check time of use
CID 30743 Time of check time of use
2015-11-05 13:27:38 +01:00
Enno Rehling
8a057f183f
CID 22580 Time of check time of use
...
no need to check this first
2015-11-05 13:22:51 +01:00
Enno Rehling
797eee025d
fix cb_insert comparison (new return value for success)
2015-11-05 13:01:48 +01:00
Enno Rehling
dcc819491f
make it easier for coverity to understand this code (there are no resource leaks)
2015-11-05 12:47:34 +01:00
Enno Rehling
05239659c8
Merge pull request #368 from ennorehling/coverity-scan
...
coverity scan fixes
2015-11-05 11:53:22 +01:00
Enno Rehling
d553066590
CID 22548 Resource leak
...
github issue #334
2015-11-05 11:31:52 +01:00
Enno Rehling
fce3f63cc1
CID 22529 Resource leak
...
github issue #351
allocation call optimization for small units
2015-11-05 11:22:16 +01:00
Enno Rehling
530f729621
CID 22499 Missing break in switch
...
github issue #357
nice catch!
2015-11-05 11:18:39 +01:00
Enno Rehling
4c861aa40d
CID 22496 Infinite loop
...
github issue #356
this is only relevant to old data files, it appears.
2015-11-05 11:17:31 +01:00
Enno Rehling
58c1d83c9e
CID 22491 FORWARD_NULL
...
github issue #352
fals positive, rewriting code to appease coverity
2015-11-05 11:14:53 +01:00
Enno Rehling
62d7e9e944
CID 22473 Evaluation order violation
...
github issue #347
2015-11-05 11:11:02 +01:00
Enno Rehling
e16e669294
CID LAST_ROW 22455 Logically dead code
...
github issue #359
song of confusion was not selecting from fleeing units
2015-11-05 11:08:55 +01:00
Enno Rehling
2a7fc87656
CID 30742 Dereference before null check
...
CID 30741 ditto
2015-11-05 10:59:14 +01:00
Enno Rehling
0206ffbec6
accidentally inverted condition when fixing volcano attribute
2015-11-05 09:57:57 +01:00
Enno Rehling
c0af920d0d
Merge pull request #367 from ennorehling/coverity-scan
...
Coverity scan fixes, part n+1
2015-11-05 09:30:55 +01:00
Enno Rehling
92d1596109
CID 22444: unchecked return value from fseek
2015-11-05 09:18:57 +01:00
Enno Rehling
a8daf061de
missing include
2015-11-05 09:14:02 +01:00
Enno Rehling
91758d2e51
CID 22439 Unchecked return value from library
2015-11-05 09:00:52 +01:00
Enno Rehling
52edf15cec
CID 22438 Unchecked return value from library
2015-11-05 08:57:56 +01:00
Enno Rehling
64e8d4b094
CID 22484 Explicit null dereferenced
2015-11-05 08:50:01 +01:00
Enno Rehling
744246ab34
CID 22498 Missing break in switch
2015-11-05 08:48:03 +01:00
Enno Rehling
06513d0db4
coverity CID 22578
2015-11-05 08:46:19 +01:00
Enno Rehling
ba69f72116
CID 22502 Argument cannot be negative
...
these curses macros are hard to decipher, I think this may be a kind of false positive because stdscr is always not-null?
2015-11-05 08:40:09 +01:00
Enno Rehling
f83e71e99a
22483 Dereference after null check
2015-11-05 08:36:32 +01:00
Enno Rehling
c68829029b
22485 Dereference after null check
2015-11-05 08:35:03 +01:00
Enno Rehling
b81e56d306
CID 22486 Dereference after null check
2015-11-05 08:32:21 +01:00
Philipp Dreher
d5f8448d7d
changed the handling of an item the unit has no access to
...
Since the return of display_item() implies that the item, which is
definitely an item but not a potion at this point, is neither in the
inventory of the unit nor an item in the region or the faction's item
pool, the existing message 36 is now triggered in this case and the
previously conditional break is now done indepent of the outcome of
display_item, which shouldn't be critical (it's an item so it can't be
a spell or a race).
2015-11-04 23:48:30 +01:00
Enno Rehling
b6190b5193
CID 22527 Resource leak
2015-11-04 19:20:34 +01:00
Enno Rehling
4d1f2327fe
CID 22545: Leak of memory or pointers to system resources
...
github issue #335
volcano destruction was not causing a resuction in wages etc., because we forgot to add the attribute to the region.
2015-11-04 19:13:15 +01:00
