diff options
Diffstat (limited to 'meta-oe/recipes-extended/mozjs/mozjs-115/1894423.patch')
-rw-r--r-- | meta-oe/recipes-extended/mozjs/mozjs-115/1894423.patch | 30 |
1 files changed, 0 insertions, 30 deletions
diff --git a/meta-oe/recipes-extended/mozjs/mozjs-115/1894423.patch b/meta-oe/recipes-extended/mozjs/mozjs-115/1894423.patch deleted file mode 100644 index 93e0f0f4f9..0000000000 --- a/meta-oe/recipes-extended/mozjs/mozjs-115/1894423.patch +++ /dev/null | |||
@@ -1,30 +0,0 @@ | |||
1 | Bug 1894423 - Remove unused ExclusiveData move constructor. r=spidermonkey-reviewers,jonco | ||
2 | |||
3 | Because the constructor is actually not used, the compiler used to not | ||
4 | complain about it being broken. Recent changes on clang trunk made it | ||
5 | catch this problem without the constructor being used. | ||
6 | |||
7 | As Mutex doesn't have a move constructor, it's also not only a matter of | ||
8 | adding the missing underscore to lock. | ||
9 | |||
10 | As the constructor is never used, just remove it. | ||
11 | |||
12 | Differential Revision: https://phabricator.services.mozilla.com/D209108 | ||
13 | |||
14 | Upstream-Status: Backport [https://hg.mozilla.org/mozilla-central/rev/223087fdc29f] | ||
15 | Signed-off-by: Khem Raj <raj.khem@gmail.com> | ||
16 | |||
17 | --- a/js/src/threading/ExclusiveData.h | ||
18 | +++ b/js/src/threading/ExclusiveData.h | ||
19 | @@ -109,11 +109,6 @@ class ExclusiveData { | ||
20 | explicit ExclusiveData(const MutexId& id, Args&&... args) | ||
21 | : lock_(id), value_(std::forward<Args>(args)...) {} | ||
22 | |||
23 | - ExclusiveData(ExclusiveData&& rhs) | ||
24 | - : lock_(std::move(rhs.lock)), value_(std::move(rhs.value_)) { | ||
25 | - MOZ_ASSERT(&rhs != this, "self-move disallowed!"); | ||
26 | - } | ||
27 | - | ||
28 | ExclusiveData& operator=(ExclusiveData&& rhs) { | ||
29 | this->~ExclusiveData(); | ||
30 | new (mozilla::KnownNotNull, this) ExclusiveData(std::move(rhs)); | ||