From: Changyeon Lee <cyeon.lee@samsung.com>
Date: Thu, 13 Jul 2017 13:12:02 +0000 (+0900)
Subject: tbm_surface_queue: fixed and subdivided error value
X-Git-Tag: accepted/tizen/3.0/common/20170721.115119~8
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;ds=sidebyside;h=34e51325ce713da73211d72fd348d4f86029ee27;p=platform%2Fcore%2Fuifw%2Flibtbm.git

tbm_surface_queue: fixed and subdivided error value

Change-Id: Ia81f97645628adaf651e17437e013b8d3fce5872
---

diff --git a/src/tbm_surface_queue.c b/src/tbm_surface_queue.c
index b4dfaec..3a04541 100644
--- a/src/tbm_surface_queue.c
+++ b/src/tbm_surface_queue.c
@@ -1095,7 +1095,11 @@ tbm_surface_queue_enqueue(tbm_surface_queue_h
 		pthread_mutex_unlock(&surface_queue->lock);
 
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_ALREADY_EXIST;
+
+		if (!node)
+			return TBM_SURFACE_QUEUE_ERROR_UNKNOWN_SURFACE;
+		else
+			return TBM_SURFACE_QUEUE_ERROR_ALREADY_EXIST;
 	}
 
 	if (surface_queue->impl && surface_queue->impl->enqueue)
@@ -1108,7 +1112,7 @@ tbm_surface_queue_enqueue(tbm_surface_queue_h
 		pthread_mutex_unlock(&surface_queue->lock);
 
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_UNKNOWN_SURFACE;
+		return TBM_SURFACE_QUEUE_ERROR_EMPTY;
 	}
 
 	node->type = QUEUE_NODE_TYPE_ENQUEUE;
@@ -1152,7 +1156,7 @@ tbm_surface_queue_dequeue(tbm_surface_queue_h
 		pthread_mutex_unlock(&surface_queue->lock);
 
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_INVALID_QUEUE;
+		return TBM_SURFACE_QUEUE_ERROR_EMPTY;
 	}
 
 	node->type = QUEUE_NODE_TYPE_DEQUEUE;
@@ -1256,7 +1260,11 @@ tbm_surface_queue_release(tbm_surface_queue_h
 		pthread_mutex_unlock(&surface_queue->lock);
 
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_INVALID_SURFACE;
+
+		if (!node)
+			return TBM_SURFACE_QUEUE_ERROR_UNKNOWN_SURFACE;
+		else
+			return TBM_SURFACE_QUEUE_ERROR_ALREADY_EXIST;
 	}
 
 	if (surface_queue->queue_size < surface_queue->num_attached) {
@@ -1283,7 +1291,7 @@ tbm_surface_queue_release(tbm_surface_queue_h
 
 		TBM_LOG_E("surface_queue->free_queue is empty.\n");
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_INVALID_SURFACE;
+		return TBM_SURFACE_QUEUE_ERROR_EMPTY;
 	}
 
 	node->type = QUEUE_NODE_TYPE_RELEASE;
@@ -1327,7 +1335,7 @@ tbm_surface_queue_acquire(tbm_surface_queue_h
 		pthread_mutex_unlock(&surface_queue->lock);
 
 		_tbm_surf_queue_mutex_unlock();
-		return TBM_SURFACE_QUEUE_ERROR_INVALID_QUEUE;
+		return TBM_SURFACE_QUEUE_ERROR_EMPTY;
 	}
 
 	node->type = QUEUE_NODE_TYPE_ACQUIRE;