Make Ref's move-ctor/move-assignment noexcept

Summary: Without noexcept, STL may use the copy constructor instead (unless the copy constructor is not accessible, but a potentially throwing move constructor is, in which case the strong exception guarantee is waived).

Reviewed By: praihan

Differential Revision: D83781391

fbshipit-source-id: f09efa828fb8a1b986b1aebd83245d8555384e45

Author: yoney

cinderx/.clang-tidy

@@ -6,5 +6,7 @@ InheritParentConfig: true
 
 Checks: '
 -facebook-hte-NullableReturn,
+-facebook-hte-GlibcRegexIsAwful,
+-facebook-hte-StdRegexIsAwful,
 modernize-raw-string-literal,
 '

cinderx/Common/ref.h

@@ -174,7 +174,7 @@ class Ref : public RefBase<T> {
     ptr_ = nullptr;
   }
 
-  Ref(Ref&& other) {
+  Ref(Ref&& other) noexcept {
     ptr_ = other.ptr_;
     other.ptr_ = nullptr;
   }
@@ -185,7 +185,7 @@ class Ref : public RefBase<T> {
     ptr_ = reinterpret_cast<T*>(other.release());
   }
 
-  Ref& operator=(Ref&& other) {
+  Ref& operator=(Ref&& other) noexcept {
     if (this == &other) {
       return *this;
     }