Merge lp:~unity-api-team/unity-scopes-shell/gcc-5 into lp:unity-scopes-shell

Proposed by Pete Woods
Status: Approved
Approved by: dobey
Approved revision: 239
Proposed branch: lp:~unity-api-team/unity-scopes-shell/gcc-5
Merge into: lp:unity-scopes-shell
Diff against target: 350 lines (+32/-32)
16 files modified
src/scope-harness/matcher/category-list-matcher.cpp (+1/-1)
src/scope-harness/matcher/category-matcher.cpp (+3/-3)
src/scope-harness/matcher/child-department-matcher.cpp (+2/-2)
src/scope-harness/matcher/match-result.cpp (+2/-2)
src/scope-harness/matcher/preview-column-matcher.cpp (+2/-2)
src/scope-harness/matcher/preview-matcher.cpp (+2/-2)
src/scope-harness/matcher/preview-widget-matcher.cpp (+3/-3)
src/scope-harness/matcher/result-matcher.cpp (+1/-1)
src/scope-harness/matcher/scope-uri.cpp (+2/-2)
src/scope-harness/preview/preview-widget-list.cpp (+2/-2)
src/scope-harness/preview/preview-widget.cpp (+2/-2)
src/scope-harness/registry/custom-registry.cpp (+2/-2)
src/scope-harness/results/category.cpp (+2/-2)
src/scope-harness/results/child-department.cpp (+2/-2)
src/scope-harness/results/department.cpp (+2/-2)
src/scope-harness/results/result.cpp (+2/-2)
To merge this branch: bzr merge lp:~unity-api-team/unity-scopes-shell/gcc-5
Reviewer Review Type Date Requested Status
Marcus Tomlinson (community) Needs Information
dobey (community) Approve
Matthias Klose Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+266227@code.launchpad.net

Commit message

Fix GCC 5 issues

Description of the change

Fix GCC 5 issues

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Matthias Klose (doko) wrote :

package builds again, but hangs in the tests (likely needs a rebuilt location-service)

review: Approve
Revision history for this message
dobey (dobey) :
review: Approve
Revision history for this message
Marcus Tomlinson (marcustomlinson) wrote :

Hey Pete, considering this MP is still floating around, I assume that it was not actually required. Can we delete this?

review: Needs Information

Unmerged revisions

239. By Pete Woods

Change move -> std::move to remove ambiguity

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'src/scope-harness/matcher/category-list-matcher.cpp'
--- src/scope-harness/matcher/category-list-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/category-list-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -118,7 +118,7 @@
118118
119CategoryListMatcher& CategoryListMatcher::category(CategoryMatcher&& categoryMatcher)119CategoryListMatcher& CategoryListMatcher::category(CategoryMatcher&& categoryMatcher)
120{120{
121 p->m_categories.emplace_back(move(categoryMatcher));121 p->m_categories.emplace_back(std::move(categoryMatcher));
122 return *this;122 return *this;
123}123}
124124
125125
=== modified file 'src/scope-harness/matcher/category-matcher.cpp'
--- src/scope-harness/matcher/category-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/category-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -180,7 +180,7 @@
180180
181CategoryMatcher::CategoryMatcher(CategoryMatcher&& other)181CategoryMatcher::CategoryMatcher(CategoryMatcher&& other)
182{182{
183 *this = move(other);183 *this = std::move(other);
184}184}
185185
186CategoryMatcher& CategoryMatcher::operator=(const CategoryMatcher& other)186CategoryMatcher& CategoryMatcher::operator=(const CategoryMatcher& other)
@@ -199,7 +199,7 @@
199199
200CategoryMatcher& CategoryMatcher::operator=(CategoryMatcher&& other)200CategoryMatcher& CategoryMatcher::operator=(CategoryMatcher&& other)
201{201{
202 p = move(other.p);202 p = std::move(other.p);
203 return *this;203 return *this;
204}204}
205205
@@ -253,7 +253,7 @@
253253
254CategoryMatcher& CategoryMatcher::result(ResultMatcher&& resultMatcher)254CategoryMatcher& CategoryMatcher::result(ResultMatcher&& resultMatcher)
255{255{
256 p->m_results.emplace_back(move(resultMatcher));256 p->m_results.emplace_back(std::move(resultMatcher));
257 return *this;257 return *this;
258}258}
259259
260260
=== modified file 'src/scope-harness/matcher/child-department-matcher.cpp'
--- src/scope-harness/matcher/child-department-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/child-department-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -88,7 +88,7 @@
8888
89ChildDepartmentMatcher::ChildDepartmentMatcher(ChildDepartmentMatcher&& other)89ChildDepartmentMatcher::ChildDepartmentMatcher(ChildDepartmentMatcher&& other)
90{90{
91 *this = move(other);91 *this = std::move(other);
92}92}
9393
94ChildDepartmentMatcher& ChildDepartmentMatcher::operator=(const ChildDepartmentMatcher& other)94ChildDepartmentMatcher& ChildDepartmentMatcher::operator=(const ChildDepartmentMatcher& other)
@@ -102,7 +102,7 @@
102102
103ChildDepartmentMatcher& ChildDepartmentMatcher::operator=(ChildDepartmentMatcher&& other)103ChildDepartmentMatcher& ChildDepartmentMatcher::operator=(ChildDepartmentMatcher&& other)
104{104{
105 p = move(other.p);105 p = std::move(other.p);
106 return *this;106 return *this;
107}107}
108108
109109
=== modified file 'src/scope-harness/matcher/match-result.cpp'
--- src/scope-harness/matcher/match-result.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/match-result.cpp 2015-07-29 13:31:53 +0000
@@ -43,7 +43,7 @@
4343
44MatchResult::MatchResult(MatchResult&& other)44MatchResult::MatchResult(MatchResult&& other)
45{45{
46 *this = move(other);46 *this = std::move(other);
47}47}
4848
49MatchResult::MatchResult(const MatchResult& other) :49MatchResult::MatchResult(const MatchResult& other) :
@@ -61,7 +61,7 @@
6161
62MatchResult& MatchResult::operator=(MatchResult&& other)62MatchResult& MatchResult::operator=(MatchResult&& other)
63{63{
64 p = move(other.p);64 p = std::move(other.p);
65 return *this;65 return *this;
66}66}
6767
6868
=== modified file 'src/scope-harness/matcher/preview-column-matcher.cpp'
--- src/scope-harness/matcher/preview-column-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/preview-column-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -52,7 +52,7 @@
5252
53PreviewColumnMatcher::PreviewColumnMatcher(PreviewColumnMatcher&& other)53PreviewColumnMatcher::PreviewColumnMatcher(PreviewColumnMatcher&& other)
54{54{
55 *this = move(other);55 *this = std::move(other);
56}56}
5757
58PreviewColumnMatcher& PreviewColumnMatcher::operator=(const PreviewColumnMatcher& other)58PreviewColumnMatcher& PreviewColumnMatcher::operator=(const PreviewColumnMatcher& other)
@@ -63,7 +63,7 @@
6363
64PreviewColumnMatcher& PreviewColumnMatcher::operator=(PreviewColumnMatcher&& other)64PreviewColumnMatcher& PreviewColumnMatcher::operator=(PreviewColumnMatcher&& other)
65{65{
66 p = move(other.p);66 p = std::move(other.p);
67 return *this;67 return *this;
68}68}
6969
7070
=== modified file 'src/scope-harness/matcher/preview-matcher.cpp'
--- src/scope-harness/matcher/preview-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/preview-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -50,7 +50,7 @@
5050
51PreviewMatcher::PreviewMatcher(PreviewMatcher&& other)51PreviewMatcher::PreviewMatcher(PreviewMatcher&& other)
52{52{
53 *this = move(other);53 *this = std::move(other);
54}54}
5555
56PreviewMatcher& PreviewMatcher::operator=(const PreviewMatcher& other)56PreviewMatcher& PreviewMatcher::operator=(const PreviewMatcher& other)
@@ -61,7 +61,7 @@
6161
62PreviewMatcher& PreviewMatcher::operator=(PreviewMatcher&& other)62PreviewMatcher& PreviewMatcher::operator=(PreviewMatcher&& other)
63{63{
64 p = move(other.p);64 p = std::move(other.p);
65 return *this;65 return *this;
66}66}
6767
6868
=== modified file 'src/scope-harness/matcher/preview-widget-matcher.cpp'
--- src/scope-harness/matcher/preview-widget-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/preview-widget-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -92,7 +92,7 @@
9292
93PreviewWidgetMatcher::PreviewWidgetMatcher(PreviewWidgetMatcher&& other)93PreviewWidgetMatcher::PreviewWidgetMatcher(PreviewWidgetMatcher&& other)
94{94{
95 *this = move(other);95 *this = std::move(other);
96}96}
9797
98PreviewWidgetMatcher& PreviewWidgetMatcher::operator=(const PreviewWidgetMatcher& other)98PreviewWidgetMatcher& PreviewWidgetMatcher::operator=(const PreviewWidgetMatcher& other)
@@ -105,7 +105,7 @@
105105
106PreviewWidgetMatcher& PreviewWidgetMatcher::operator=(PreviewWidgetMatcher&& other)106PreviewWidgetMatcher& PreviewWidgetMatcher::operator=(PreviewWidgetMatcher&& other)
107{107{
108 p = move(other.p);108 p = std::move(other.p);
109 return *this;109 return *this;
110}110}
111111
@@ -127,7 +127,7 @@
127127
128PreviewWidgetMatcher& PreviewWidgetMatcher::data(sc::Variant&& data)128PreviewWidgetMatcher& PreviewWidgetMatcher::data(sc::Variant&& data)
129{129{
130 p->m_data = move(data);130 p->m_data = std::move(data);
131 return *this;131 return *this;
132}132}
133133
134134
=== modified file 'src/scope-harness/matcher/result-matcher.cpp'
--- src/scope-harness/matcher/result-matcher.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/result-matcher.cpp 2015-07-29 13:31:53 +0000
@@ -186,7 +186,7 @@
186186
187ResultMatcher& ResultMatcher::operator=(ResultMatcher&& other)187ResultMatcher& ResultMatcher::operator=(ResultMatcher&& other)
188{188{
189 p = move(other.p);189 p = std::move(other.p);
190 return *this;190 return *this;
191}191}
192192
193193
=== modified file 'src/scope-harness/matcher/scope-uri.cpp'
--- src/scope-harness/matcher/scope-uri.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/matcher/scope-uri.cpp 2015-07-29 13:31:53 +0000
@@ -81,7 +81,7 @@
8181
82ScopeUri::ScopeUri(ScopeUri&& other)82ScopeUri::ScopeUri(ScopeUri&& other)
83{83{
84 *this = move(other);84 *this = std::move(other);
85}85}
8686
87ScopeUri& ScopeUri::operator=(const ScopeUri& other)87ScopeUri& ScopeUri::operator=(const ScopeUri& other)
@@ -94,7 +94,7 @@
9494
95ScopeUri& ScopeUri::operator=(ScopeUri&& other)95ScopeUri& ScopeUri::operator=(ScopeUri&& other)
96{96{
97 p = move(other.p);97 p = std::move(other.p);
98 return *this;98 return *this;
99}99}
100100
101101
=== modified file 'src/scope-harness/preview/preview-widget-list.cpp'
--- src/scope-harness/preview/preview-widget-list.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/preview/preview-widget-list.cpp 2015-07-29 13:31:53 +0000
@@ -48,7 +48,7 @@
4848
49PreviewWidgetList::PreviewWidgetList(PreviewWidgetList&& other)49PreviewWidgetList::PreviewWidgetList(PreviewWidgetList&& other)
50{50{
51 *this = move(other);51 *this = std::move(other);
52}52}
5353
54PreviewWidgetList::~PreviewWidgetList()54PreviewWidgetList::~PreviewWidgetList()
@@ -63,7 +63,7 @@
6363
64PreviewWidgetList& PreviewWidgetList::operator=(PreviewWidgetList&& other)64PreviewWidgetList& PreviewWidgetList::operator=(PreviewWidgetList&& other)
65{65{
66 p = move(other.p);66 p = std::move(other.p);
67 return *this;67 return *this;
68}68}
6969
7070
=== modified file 'src/scope-harness/preview/preview-widget.cpp'
--- src/scope-harness/preview/preview-widget.cpp 2015-04-27 11:28:36 +0000
+++ src/scope-harness/preview/preview-widget.cpp 2015-07-29 13:31:53 +0000
@@ -81,7 +81,7 @@
8181
82PreviewWidget::PreviewWidget(PreviewWidget&& other)82PreviewWidget::PreviewWidget(PreviewWidget&& other)
83{83{
84 *this = move(other);84 *this = std::move(other);
85}85}
8686
87PreviewWidget& PreviewWidget::operator=(const PreviewWidget& other)87PreviewWidget& PreviewWidget::operator=(const PreviewWidget& other)
@@ -97,7 +97,7 @@
9797
98PreviewWidget& PreviewWidget::operator=(PreviewWidget&& other)98PreviewWidget& PreviewWidget::operator=(PreviewWidget&& other)
99{99{
100 p = move(other.p);100 p = std::move(other.p);
101 return *this;101 return *this;
102}102}
103103
104104
=== modified file 'src/scope-harness/registry/custom-registry.cpp'
--- src/scope-harness/registry/custom-registry.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/registry/custom-registry.cpp 2015-07-29 13:31:53 +0000
@@ -91,7 +91,7 @@
9191
92CustomRegistry::Parameters::Parameters(Parameters&& other)92CustomRegistry::Parameters::Parameters(Parameters&& other)
93{93{
94 p = move(other.p);94 p = std::move(other.p);
95}95}
9696
97CustomRegistry::Parameters& CustomRegistry::Parameters::operator=(const Parameters& other)97CustomRegistry::Parameters& CustomRegistry::Parameters::operator=(const Parameters& other)
@@ -106,7 +106,7 @@
106106
107CustomRegistry::Parameters& CustomRegistry::Parameters::operator=(Parameters&& other)107CustomRegistry::Parameters& CustomRegistry::Parameters::operator=(Parameters&& other)
108{108{
109 p = move(other.p);109 p = std::move(other.p);
110 return *this;110 return *this;
111}111}
112112
113113
=== modified file 'src/scope-harness/results/category.cpp'
--- src/scope-harness/results/category.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/results/category.cpp 2015-07-29 13:31:53 +0000
@@ -58,7 +58,7 @@
58Category::Category(Category&& other) :58Category::Category(Category&& other) :
59 p(new _Priv)59 p(new _Priv)
60{60{
61 *this = move(other);61 *this = std::move(other);
62}62}
6363
64Category::Category(const Category& other) :64Category::Category(const Category& other) :
@@ -77,7 +77,7 @@
7777
78Category& Category::operator=(Category&& other)78Category& Category::operator=(Category&& other)
79{79{
80 p = move(other.p);80 p = std::move(other.p);
81 return *this;81 return *this;
82}82}
8383
8484
=== modified file 'src/scope-harness/results/child-department.cpp'
--- src/scope-harness/results/child-department.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/results/child-department.cpp 2015-07-29 13:31:53 +0000
@@ -62,7 +62,7 @@
6262
63ChildDepartment::ChildDepartment(ChildDepartment&& other)63ChildDepartment::ChildDepartment(ChildDepartment&& other)
64{64{
65 *this = move(other);65 *this = std::move(other);
66}66}
6767
68ChildDepartment& ChildDepartment::operator=(const ChildDepartment& other)68ChildDepartment& ChildDepartment::operator=(const ChildDepartment& other)
@@ -74,7 +74,7 @@
7474
75ChildDepartment& ChildDepartment::operator=(ChildDepartment&& other)75ChildDepartment& ChildDepartment::operator=(ChildDepartment&& other)
76{76{
77 p = move(other.p);77 p = std::move(other.p);
78 return *this;78 return *this;
79}79}
8080
8181
=== modified file 'src/scope-harness/results/department.cpp'
--- src/scope-harness/results/department.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/results/department.cpp 2015-07-29 13:31:53 +0000
@@ -70,7 +70,7 @@
7070
71Department::Department(Department&& other)71Department::Department(Department&& other)
72{72{
73 *this = move(other);73 *this = std::move(other);
74}74}
7575
76Department& Department::operator=(const Department& other)76Department& Department::operator=(const Department& other)
@@ -82,7 +82,7 @@
8282
83Department& Department::operator=(Department&& other)83Department& Department::operator=(Department&& other)
84{84{
85 p = move(other.p);85 p = std::move(other.p);
86 return *this;86 return *this;
87}87}
8888
8989
=== modified file 'src/scope-harness/results/result.cpp'
--- src/scope-harness/results/result.cpp 2015-03-10 11:23:01 +0000
+++ src/scope-harness/results/result.cpp 2015-07-29 13:31:53 +0000
@@ -237,7 +237,7 @@
237237
238Result::Result(Result&& other)238Result::Result(Result&& other)
239{239{
240 *this = move(other);240 *this = std::move(other);
241}241}
242242
243Result::Result(const Result& other) :243Result::Result(const Result& other) :
@@ -264,7 +264,7 @@
264264
265Result& Result::operator=(Result&& other)265Result& Result::operator=(Result&& other)
266{266{
267 p = move(other.p);267 p = std::move(other.p);
268 return *this;268 return *this;
269}269}
270270

Subscribers

People subscribed via source and target branches

to all changes: