Fix a signed vs unsigned comparison warning in the ijar zipper.
This is more than just a cosmetic issue. Since groovy and appengine
rules depend directly on this target, the zipper gets built in the
user's workspace whenever a groovy/appengine target is built. If the
workspace is set to upgrade warnings to errors, this can cause builds
to fail.
--
Change-Id: I6c8b347df14098945c788411fc3e38f9c128596f
Reviewed-on: https://bazel-review.googlesource.com/1951
MOS_MIGRATED_REVID=102223767
diff --git a/third_party/ijar/zip_main.cc b/third_party/ijar/zip_main.cc
index 6c2a974..43b035a 100644
--- a/third_party/ijar/zip_main.cc
+++ b/third_party/ijar/zip_main.cc
@@ -72,8 +72,8 @@
void concat_path(char* out, const size_t size,
const char *path1, const char *path2) {
int len1 = strlen(path1);
- int l = len1;
- strncpy(out, path1, size-1);
+ size_t l = len1;
+ strncpy(out, path1, size - 1);
out[size-1] = 0;
if (l < size - 1 && path1[len1] != '/' && path2[0] != '/') {
out[l] = '/';
