shithub: libvpx

Download patch

ref: e50ea014c3d41652df34ed1325511803268e2ca7
parent: 943f9ee25ced68146dfe5be677b69129a75d4d90
author: Johann <[email protected]>
date: Thu Jun 8 06:20:21 EDT 2017 

Revert "buffer.h: use size_t"

This reverts commit f08581c1d010ea95b8cfae686b5c0a64b32519f9.

type conversion warnings abound.

Change-Id: I41d4c0e7a388e1008bdbc55fefda4bbca3f89f00


--- a/test/buffer.h
+++ b/test/buffer.h
@@ -25,14 +25,14 @@
 template <typename T>
 class Buffer {
  public:
-  Buffer(size_t width, size_t height, size_t top_padding, size_t left_padding,
-         size_t right_padding, size_t bottom_padding)
+  Buffer(int width, int height, int top_padding, int left_padding,
+         int right_padding, int bottom_padding)
       : width_(width), height_(height), top_padding_(top_padding),
         left_padding_(left_padding), right_padding_(right_padding),
         bottom_padding_(bottom_padding), padding_value_(0), stride_(0),
         raw_size_(0), num_elements_(0), raw_buffer_(NULL) {}
 
-  Buffer(size_t width, size_t height, size_t padding)
+  Buffer(int width, int height, int padding)
       : width_(width), height_(height), top_padding_(padding),
         left_padding_(padding), right_padding_(padding),
         bottom_padding_(padding), padding_value_(0), stride_(0), raw_size_(0),
@@ -42,7 +42,7 @@
 
   T *TopLeftPixel() const;
 
-  size_t stride() const { return stride_; }
+  int stride() const { return stride_; }
 
   // Set the buffer (excluding padding) to 'value'.
   void Set(const T value);
@@ -74,8 +74,12 @@
   bool CheckValues(const Buffer<T> &a) const;
 
   bool Init() {
-    EXPECT_GT(width_, 0u);
-    EXPECT_GT(height_, 0u);
+    EXPECT_GT(width_, 0);
+    EXPECT_GT(height_, 0);
+    EXPECT_GE(top_padding_, 0);
+    EXPECT_GE(left_padding_, 0);
+    EXPECT_GE(right_padding_, 0);
+    EXPECT_GE(bottom_padding_, 0);
     stride_ = left_padding_ + width_ + right_padding_;
     num_elements_ = stride_ * (top_padding_ + height_ + bottom_padding_);
     raw_buffer_ = new (std::nothrow) T[num_elements_];
@@ -88,16 +92,16 @@
  private:
   bool BufferSizesMatch(const Buffer<T> &a) const;
 
-  const size_t width_;
-  const size_t height_;
-  const size_t top_padding_;
-  const size_t left_padding_;
-  const size_t right_padding_;
-  const size_t bottom_padding_;
+  const int width_;
+  const int height_;
+  const int top_padding_;
+  const int left_padding_;
+  const int right_padding_;
+  const int bottom_padding_;
   T padding_value_;
-  size_t stride_;
-  size_t raw_size_;
-  size_t num_elements_;
+  int stride_;
+  int raw_size_;
+  int num_elements_;
   T *raw_buffer_;
 };
 
@@ -111,8 +115,8 @@
 void Buffer<T>::Set(const T value) {
   if (!raw_buffer_) return;
   T *src = TopLeftPixel();
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       src[width] = value;
     }
     src += stride();
@@ -123,8 +127,8 @@
 void Buffer<T>::Set(ACMRandom *rand_class, T (ACMRandom::*rand_func)()) {
   if (!raw_buffer_) return;
   T *src = TopLeftPixel();
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       src[width] = (*rand_class.*rand_func)();
     }
     src += stride();
@@ -138,8 +142,8 @@
 
   T *a_src = a.TopLeftPixel();
   T *b_src = this->TopLeftPixel();
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       b_src[width] = a_src[width];
     }
     a_src += a.stride();
@@ -150,9 +154,9 @@
 template <typename T>
 void Buffer<T>::DumpBuffer() const {
   if (!raw_buffer_) return;
-  for (size_t height = 0; height < height_ + top_padding_ + bottom_padding_;
+  for (int height = 0; height < height_ + top_padding_ + bottom_padding_;
        ++height) {
-    for (size_t width = 0; width < stride(); ++width) {
+    for (int width = 0; width < stride(); ++width) {
       printf("%4d", raw_buffer_[height + width * stride()]);
     }
     printf("\n");
@@ -174,8 +178,8 @@
   T *b_src = TopLeftPixel();
 
   printf("This buffer:\n");
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       if (a_src[width] != b_src[width]) {
         printf("*%3d", b_src[width]);
       } else {
@@ -191,8 +195,8 @@
   b_src = TopLeftPixel();
 
   printf("Reference buffer:\n");
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       if (a_src[width] != b_src[width]) {
         printf("*%3d", a_src[width]);
       } else {
@@ -211,7 +215,7 @@
   padding_value_ = padding_value;
 
   T *src = raw_buffer_;
-  for (size_t i = 0; i < num_elements_; ++i) {
+  for (int i = 0; i < num_elements_; ++i) {
     src[i] = padding_value;
   }
 }
@@ -220,8 +224,8 @@
 bool Buffer<T>::CheckValues(const T value) const {
   if (!raw_buffer_) return false;
   T *src = TopLeftPixel();
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       if (value != src[width]) {
         return false;
       }
@@ -238,7 +242,7 @@
 
   // Top padding.
   T const *top = raw_buffer_;
-  for (size_t i = 0; i < stride() * top_padding_; ++i) {
+  for (int i = 0; i < stride() * top_padding_; ++i) {
     if (padding_value_ != top[i]) {
       return false;
     }
@@ -246,8 +250,8 @@
 
   // Left padding.
   T const *left = TopLeftPixel() - left_padding_;
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < left_padding_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < left_padding_; ++width) {
       if (padding_value_ != left[width]) {
         return false;
       }
@@ -257,8 +261,8 @@
 
   // Right padding.
   T const *right = TopLeftPixel() + width_;
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < right_padding_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < right_padding_; ++width) {
       if (padding_value_ != right[width]) {
         return false;
       }
@@ -268,7 +272,7 @@
 
   // Bottom padding
   T const *bottom = raw_buffer_ + (top_padding_ + height_) * stride();
-  for (size_t i = 0; i < stride() * bottom_padding_; ++i) {
+  for (int i = 0; i < stride() * bottom_padding_; ++i) {
     if (padding_value_ != bottom[i]) {
       return false;
     }
@@ -284,8 +288,8 @@
 
   T *a_src = a.TopLeftPixel();
   T *b_src = this->TopLeftPixel();
-  for (size_t height = 0; height < height_; ++height) {
-    for (size_t width = 0; width < width_; ++width) {
+  for (int height = 0; height < height_; ++height) {
+    for (int width = 0; width < width_; ++width) {
       if (a_src[width] != b_src[width]) {
         return false;
       }
@@ -301,8 +305,8 @@
   if (!raw_buffer_) return false;
   if (a.width_ != this->width_ || a.height_ != this->height_) {
     printf(
-        "Reference buffer of size %zux%zu does not match this buffer which is "
-        "size %zux%zu\n",
+        "Reference buffer of size %dx%d does not match this buffer which is "
+        "size %dx%d\n",
         a.width_, a.height_, this->width_, this->height_);
     return false;
   }