Message ID | 20200701013908.42359-1-warthog618@gmail.com |
---|---|
State | New |
Headers | show |
Series | [libgpiod] bindings: cxx: fix sign-compare compile warnings in tests | expand |
On Wed, Jul 1, 2020 at 3:39 AM Kent Gibson <warthog618@gmail.com> wrote: > > Fix sign-compare compile warnings in cxx tests. > > Signed-off-by: Kent Gibson <warthog618@gmail.com> > --- > > Just noticed these warnings while rebuilding in a fresh build environment. > Not sure if they were triggered by an enviromental change, or if I just > wasn't paying enough attention before, but either way this patch makes > the compiler happier. > > bindings/cxx/tests/tests-iter.cpp | 2 +- > bindings/cxx/tests/tests-line.cpp | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/bindings/cxx/tests/tests-iter.cpp b/bindings/cxx/tests/tests-iter.cpp > index fdc2cb5..4c07613 100644 > --- a/bindings/cxx/tests/tests-iter.cpp > +++ b/bindings/cxx/tests/tests-iter.cpp > @@ -53,7 +53,7 @@ TEST_CASE("Line iterator works", "[iter][line]") > { > mockup::probe_guard mockup_chips({ 4 }); > ::gpiod::chip chip(mockup::instance().chip_name(0)); > - int count = 0; > + unsigned int count = 0; > > for (auto& it: ::gpiod::line_iter(chip)) > REQUIRE(it.offset() == count++); > diff --git a/bindings/cxx/tests/tests-line.cpp b/bindings/cxx/tests/tests-line.cpp > index 5353093..e2e4cbc 100644 > --- a/bindings/cxx/tests/tests-line.cpp > +++ b/bindings/cxx/tests/tests-line.cpp > @@ -210,7 +210,7 @@ TEST_CASE("Line bulk object works correctly", "[line][bulk]") > SECTION("bulk iterator works") > { > auto lines = chip.get_all_lines(); > - int count = 0; > + unsigned int count = 0; > > for (auto& it: lines) > REQUIRE(it.offset() == count++); > > base-commit: b56d6b6a452e47fee8c70514afb99ccd77ada677 > -- > 2.27.0 > Applied, thanks! Bartosz
diff --git a/bindings/cxx/tests/tests-iter.cpp b/bindings/cxx/tests/tests-iter.cpp index fdc2cb5..4c07613 100644 --- a/bindings/cxx/tests/tests-iter.cpp +++ b/bindings/cxx/tests/tests-iter.cpp @@ -53,7 +53,7 @@ TEST_CASE("Line iterator works", "[iter][line]") { mockup::probe_guard mockup_chips({ 4 }); ::gpiod::chip chip(mockup::instance().chip_name(0)); - int count = 0; + unsigned int count = 0; for (auto& it: ::gpiod::line_iter(chip)) REQUIRE(it.offset() == count++); diff --git a/bindings/cxx/tests/tests-line.cpp b/bindings/cxx/tests/tests-line.cpp index 5353093..e2e4cbc 100644 --- a/bindings/cxx/tests/tests-line.cpp +++ b/bindings/cxx/tests/tests-line.cpp @@ -210,7 +210,7 @@ TEST_CASE("Line bulk object works correctly", "[line][bulk]") SECTION("bulk iterator works") { auto lines = chip.get_all_lines(); - int count = 0; + unsigned int count = 0; for (auto& it: lines) REQUIRE(it.offset() == count++);
Fix sign-compare compile warnings in cxx tests. Signed-off-by: Kent Gibson <warthog618@gmail.com> --- Just noticed these warnings while rebuilding in a fresh build environment. Not sure if they were triggered by an enviromental change, or if I just wasn't paying enough attention before, but either way this patch makes the compiler happier. bindings/cxx/tests/tests-iter.cpp | 2 +- bindings/cxx/tests/tests-line.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) base-commit: b56d6b6a452e47fee8c70514afb99ccd77ada677