commit 6f5c314f07724469071fd5f347ac5479d138a033
author Evan Shrubsole <eshr@webrtc.org> Wed Dec 21 11:20:58 2022
committer WebRTC LUCI CQ <webrtc-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Dec 21 12:07:06 2022
tree 4c27f3161b5810b3ec165744df2c1dd708dec41b
parent 70d9e4e02b7e9249505efdd3dd57e42bde5cbf84

Remove unused SeqNumUnwarpper::UnwrapForward/Backwards

Bug: webrtc:13982
Change-Id: I7cef4f9dd5f0c0dcb9680269005e4a9f04d3e490
Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/288741
Reviewed-by: Philip Eliasson <philipel@webrtc.org>
Reviewed-by: Harald Alvestrand <hta@webrtc.org>
Commit-Queue: Evan Shrubsole <eshr@webrtc.org>
Cr-Commit-Position: refs/heads/main@{#38939}

rtc_base/numerics/sequence_number_util.h

diff --git a/rtc_base/numerics/sequence_number_util.h b/rtc_base/numerics/sequence_number_util.h
index 5da9d45..d68c064 100644
--- a/rtc_base/numerics/sequence_number_util.h
+++ b/rtc_base/numerics/sequence_number_util.h
@@ -109,28 +109,6 @@
     return last_unwrapped_;
   }
 
-  int64_t UnwrapForward(T value) {
-    if (!last_value_) {
-      last_unwrapped_ = {value};
-    } else {
-      last_unwrapped_ += ForwardDiff<T, M>(*last_value_, value);
-    }
-
-    last_value_ = value;
-    return last_unwrapped_;
-  }
-
-  int64_t UnwrapBackwards(T value) {
-    if (!last_value_) {
-      last_unwrapped_ = {value};
-    } else {
-      last_unwrapped_ -= ReverseDiff<T, M>(*last_value_, value);
-    }
-
-    last_value_ = value;
-    return last_unwrapped_;
-  }
-
  private:
   int64_t last_unwrapped_ = 0;
   absl::optional<T> last_value_;

rtc_base/numerics/sequence_number_util_unittest.cc

diff --git a/rtc_base/numerics/sequence_number_util_unittest.cc b/rtc_base/numerics/sequence_number_util_unittest.cc
index c353c92..d7fa0b1 100644
--- a/rtc_base/numerics/sequence_number_util_unittest.cc
+++ b/rtc_base/numerics/sequence_number_util_unittest.cc
@@ -303,32 +303,4 @@
   }
 }
 
-TEST(SeqNumUnwrapper, UnwrapForward) {
-  SeqNumUnwrapper<uint8_t> unwrapper;
-  EXPECT_EQ(255, unwrapper.Unwrap(255));
-  EXPECT_EQ(256, unwrapper.UnwrapForward(0));
-  EXPECT_EQ(511, unwrapper.UnwrapForward(255));
-}
-
-TEST(SeqNumUnwrapper, UnwrapForwardWithDivisor) {
-  SeqNumUnwrapper<uint8_t, 33> unwrapper;
-  EXPECT_EQ(30, unwrapper.UnwrapForward(30));
-  EXPECT_EQ(36, unwrapper.UnwrapForward(3));
-  EXPECT_EQ(63, unwrapper.UnwrapForward(30));
-}
-
-TEST(SeqNumUnwrapper, UnwrapBackwards) {
-  SeqNumUnwrapper<uint8_t> unwrapper;
-  EXPECT_EQ(0, unwrapper.UnwrapBackwards(0));
-  EXPECT_EQ(-2, unwrapper.UnwrapBackwards(254));
-  EXPECT_EQ(-256, unwrapper.UnwrapBackwards(0));
-}
-
-TEST(SeqNumUnwrapper, UnwrapBackwardsWithDivisor) {
-  SeqNumUnwrapper<uint8_t, 33> unwrapper;
-  EXPECT_EQ(0, unwrapper.Unwrap(0));
-  EXPECT_EQ(-2, unwrapper.UnwrapBackwards(31));
-  EXPECT_EQ(-33, unwrapper.UnwrapBackwards(0));
-}
-
 }  // namespace webrtc