From 7d2e122f2fe71ef42b8677b9e8a048434809cb56 Mon Sep 17 00:00:00 2001 From: Kyle Manna Date: Mon, 17 Aug 2015 18:48:17 -0700 Subject: [PATCH] signedness: Fix obvious sign problems * These should be easily verified as correct. * Tested on gcc-5.2.0 and clang-3.6.2 --- capn-test.cpp | 6 +++--- compiler/schema-test.cpp | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/capn-test.cpp b/capn-test.cpp index b004c0f..0d94997 100644 --- a/capn-test.cpp +++ b/capn-test.cpp @@ -318,7 +318,7 @@ TEST(WireFormat, StructRoundTrip_OneSegmentPerAllocation) { struct capn ctx2; memset(&ctx2, 0, sizeof(ctx2)); - for (int i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { + for (size_t i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { capn_append_segment(&ctx2, segments[i]); } @@ -374,7 +374,7 @@ TEST(WireFormat, StructRoundTrip_OneSegmentPerAllocation_NoTag) { struct capn ctx2; memset(&ctx2, 0, sizeof(ctx2)); - for (int i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { + for (size_t i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { capn_append_segment(&ctx2, segments[i]); } @@ -424,7 +424,7 @@ TEST(WireFormat, StructRoundTrip_MultipleSegmentsWithMultipleAllocations) { struct capn ctx2; memset(&ctx2, 0, sizeof(ctx2)); - for (int i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { + for (size_t i = 0; i < sizeof(segments)/sizeof(segments[0]); i++) { capn_append_segment(&ctx2, segments[i]); } diff --git a/compiler/schema-test.cpp b/compiler/schema-test.cpp index b192016..802039b 100644 --- a/compiler/schema-test.cpp +++ b/compiler/schema-test.cpp @@ -189,7 +189,7 @@ TEST(Schema, ReadSimple) { struct CodeGeneratorRequest req; read_CodeGeneratorRequest(&req, root); - for (size_t i = 0; i < req.nodes.p.len; i++) { + for (int i = 0; i < req.nodes.p.len; i++) { } }