diff options
author | Andrei Karas <akaras@inbox.ru> | 2016-07-14 22:19:51 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2016-07-14 22:19:51 +0300 |
commit | 79ea125f00a9655ee3aea7c784bd012888f39cf6 (patch) | |
tree | 1825fec0dabf8fdb06cc578e640825bae93048f1 /src/map/skill.c | |
parent | b89dd1428b2d357c8bbe016573fb0ab25ade2782 (diff) | |
parent | 6c0b298c869913105e4c4d29a6580bf68cd89f55 (diff) | |
download | hercules-79ea125f00a9655ee3aea7c784bd012888f39cf6.tar.gz hercules-79ea125f00a9655ee3aea7c784bd012888f39cf6.tar.bz2 hercules-79ea125f00a9655ee3aea7c784bd012888f39cf6.tar.xz hercules-79ea125f00a9655ee3aea7c784bd012888f39cf6.zip |
Merge pull request #1361 from HerculesWS/coverity-fixes
Fixed various issues reported by coverity
Diffstat (limited to 'src/map/skill.c')
-rw-r--r-- | src/map/skill.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/map/skill.c b/src/map/skill.c index 2962cb76e..0c2f099fb 100644 --- a/src/map/skill.c +++ b/src/map/skill.c @@ -6891,7 +6891,7 @@ int skill_castend_nodamage_id(struct block_list *src, struct block_list *bl, uin return 1; } if( skill_id == AM_BERSERKPITCHER ) { - if( dstsd && dstsd->status.base_level < (unsigned int)sd->inventory_data[i]->elv ) { + if (dstsd && dstsd->status.base_level < sd->inventory_data[i]->elv) { clif->skill_fail(sd,skill_id,USESKILL_FAIL_LEVEL,0); map->freeblock_unlock(); return 1; @@ -15322,7 +15322,7 @@ void skill_weaponrefine (struct map_session_data *sd, int idx) if( sd->status.class_ == JOB_MECHANIC_T ) per += 100; else - per += 5 * ((signed int)sd->status.job_level - 50); + per += 5 * (sd->status.job_level - 50); pc->delitem(sd, i, 1, 0, DELITEM_NORMAL, LOG_TYPE_REFINE); // FIXME: is this the correct reason flag? if (per > rnd() % 1000) { |