diff options
| author | Lance <Lance@54d463be-8e91-2dee-dedb-b68131a5f0ec> | 2006-05-29 09:53:45 +0000 |
|---|---|---|
| committer | Lance <Lance@54d463be-8e91-2dee-dedb-b68131a5f0ec> | 2006-05-29 09:53:45 +0000 |
| commit | 9e62c33d9ee58d7dcf1c667a77b3fdd814bc0943 (patch) | |
| tree | 872a0cc972ddfc74e17afff9efc4065dc284401b /src/map/pc.c | |
| parent | 3b0f07f4b6d7ba237ba37a7d4ce627ded5581cad (diff) | |
| download | hercules-9e62c33d9ee58d7dcf1c667a77b3fdd814bc0943.tar.gz hercules-9e62c33d9ee58d7dcf1c667a77b3fdd814bc0943.tar.bz2 hercules-9e62c33d9ee58d7dcf1c667a77b3fdd814bc0943.tar.xz hercules-9e62c33d9ee58d7dcf1c667a77b3fdd814bc0943.zip | |
[Optimized]
- Removed unused checks for unsigned data type and possible logic error for char type (gcc treats char as unsigned).
git-svn-id: https://rathena.svn.sourceforge.net/svnroot/rathena/trunk@6814 54d463be-8e91-2dee-dedb-b68131a5f0ec
Diffstat (limited to 'src/map/pc.c')
| -rw-r--r-- | src/map/pc.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/map/pc.c b/src/map/pc.c index ccd066c0c..3a573f97f 100644 --- a/src/map/pc.c +++ b/src/map/pc.c @@ -3842,7 +3842,7 @@ int pc_gainexp(struct map_session_data *sd,unsigned int base_exp,unsigned int jo //Overflow checks... think we'll ever really need'em? [Skotlex]
if (base_exp > 0 && sd->status.base_exp > UINT_MAX - base_exp)
sd->status.base_exp = UINT_MAX;
- else if (base_exp < 0 && sd->status.base_exp > base_exp)
+ else if (sd->status.base_exp > base_exp)
sd->status.base_exp = 0;
else
sd->status.base_exp += base_exp;
@@ -3854,7 +3854,7 @@ int pc_gainexp(struct map_session_data *sd,unsigned int base_exp,unsigned int jo //Overflow checks... think we'll ever really need'em? [Skotlex]
if (job_exp > 0 && sd->status.job_exp > UINT_MAX - job_exp)
sd->status.job_exp = UINT_MAX;
- else if (job_exp < 0 && sd->status.job_exp > job_exp)
+ else if (sd->status.job_exp > job_exp)
sd->status.job_exp = 0;
else
sd->status.job_exp += job_exp;
@@ -4933,16 +4933,12 @@ int pc_setparam(struct map_session_data *sd,int type,int val) case SP_BASEEXP:
if(pc_nextbaseexp(sd) > 0) {
sd->status.base_exp = val;
- if(sd->status.base_exp < 0)
- sd->status.base_exp=0;
pc_checkbaselevelup(sd);
}
break;
case SP_JOBEXP:
if(pc_nextjobexp(sd) > 0) {
sd->status.job_exp = val;
- if(sd->status.job_exp < 0)
- sd->status.job_exp=0;
pc_checkjoblevelup(sd);
}
break;
|