Code Review
/
apps
/
agl-service-windowmanager.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
Adopt error code and message into Window Manager
[apps/agl-service-windowmanager.git]
/
src
/
applist.cpp
diff --git
a/src/applist.cpp
b/src/applist.cpp
index
8c13f5f
..
6b608d0
100644
(file)
--- a/
src/applist.cpp
+++ b/
src/applist.cpp
@@
-54,10
+54,10
@@
void AppList::removeClient(const string &appid)
client_list.erase(appid);
}
client_list.erase(appid);
}
-
bool
AppList::contains(const string &appid)
+
WMError
AppList::contains(const string &appid)
{
auto result = client_list.find(appid);
{
auto result = client_list.find(appid);
- return (client_list.end() != result) ?
true : false
;
+ return (client_list.end() != result) ?
WMError::SUCCESS : WMError::NOT_REGISTERED
;
}
void AppList::removeSurface(unsigned surface_id){
}
void AppList::removeSurface(unsigned surface_id){
@@
-150,9
+150,9
@@
const vector<struct WMAction> &AppList::getActions(unsigned req_num)
}
}
}
}
-
bool
AppList::setAction(unsigned req_num, const struct WMAction &action)
+
WMError
AppList::setAction(unsigned req_num, const struct WMAction &action)
{
{
-
bool result = false
;
+
WMError result = WMError::FAIL
;
for (auto &x : req_list)
{
if (req_num != x.seq_num)
for (auto &x : req_list)
{
if (req_num != x.seq_num)
@@
-160,16
+160,16
@@
bool AppList::setAction(unsigned req_num, const struct WMAction &action)
continue;
}
x.sync_draw_req.push_back(action);
continue;
}
x.sync_draw_req.push_back(action);
- result =
true
;
+ result =
WMError::SUCCESS
;
break;
}
return result;
}
break;
}
return result;
}
-
bool
AppList::setAction(unsigned req_num, const string &appid, const string &role, const string &area, bool visible)
+
WMError
AppList::setAction(unsigned req_num, const string &appid, const string &role, const string &area, bool visible)
{
{
-
bool result = false
;
+
WMError result = WMError::NOT_REGISTERED
;
for (auto &x : req_list)
{
if (req_num != x.seq_num)
for (auto &x : req_list)
{
if (req_num != x.seq_num)
@@
-179,7
+179,7
@@
bool AppList::setAction(unsigned req_num, const string &appid, const string &rol
WMAction action{appid, role, area, visible, false};
x.sync_draw_req.push_back(action);
WMAction action{appid, role, area, visible, false};
x.sync_draw_req.push_back(action);
- result =
true
;
+ result =
WMError::SUCCESS
;
break;
}
return result;
break;
}
return result;