commit 1dc4a1636069ff3e1be38320be4d5716ec4c7ea4
author lpino <lpino@google.com> Tue Mar 27 03:21:47 2018 -0700
committer Copybara-Service <copybara-piper@google.com> Tue Mar 27 03:23:10 2018 -0700
tree 0858ba07d524389b9cf0cd30c3d20d82e91b42eb
parent 0b8d0341c6c7f4a115b2f12a16cdd8687f24cbca

Automated rollback of commit 8c864137d46cf6c3714fb47799a34e4859aa2f13.

*** Reason for rollback ***

Breaks Bazel on Windows: https://buildkite.com/bazel/bazel-bazel/builds/1041

*** Original change description ***

Fix a warning about comparing signed and unsigned values

PiperOrigin-RevId: 190593548

third_party/ijar/mapped_file.h

diff --git a/third_party/ijar/mapped_file.h b/third_party/ijar/mapped_file.h
index d58fc18..7653638 100644
--- a/third_party/ijar/mapped_file.h
+++ b/third_party/ijar/mapped_file.h
@@ -76,7 +76,7 @@
 
   // The mapped contents of the file.
   u1* Buffer() const { return buffer_; }
-  int Close(u8 size);
+  int Close(int size);
 };
 
 }  // namespace devtools_ijar

third_party/ijar/mapped_file_unix.cc

diff --git a/third_party/ijar/mapped_file_unix.cc b/third_party/ijar/mapped_file_unix.cc
index 5271f00..ccb8787 100644
--- a/third_party/ijar/mapped_file_unix.cc
+++ b/third_party/ijar/mapped_file_unix.cc
@@ -131,9 +131,9 @@
   delete impl_;
 }
 
-int MappedOutputFile::Close(u8 size) {
+int MappedOutputFile::Close(int size) {
   if (size > estimated_size_) {
-    snprintf(errmsg, MAX_ERROR, "size %lld > estimated size %lld", size,
+    snprintf(errmsg, MAX_ERROR, "size %d > estimated size %lld", size,
              estimated_size_);
     errmsg_ = errmsg;
     return -1;