[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

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;