diff options
| author | Garrett D'Amore <garrett@damore.org> | 2018-02-14 14:50:04 -0800 |
|---|---|---|
| committer | Garrett D'Amore <garrett@damore.org> | 2018-02-14 18:28:36 -0800 |
| commit | 45bc175ef9278c175d2fc3a0678b49b18e74c449 (patch) | |
| tree | b1838778ee898112f28b35178364068c6f48c9b4 /tests/idhash.c | |
| parent | 8f93750ed2a6aaa1749eb689ddf119280f9aac7a (diff) | |
| download | nng-45bc175ef9278c175d2fc3a0678b49b18e74c449.tar.gz nng-45bc175ef9278c175d2fc3a0678b49b18e74c449.tar.bz2 nng-45bc175ef9278c175d2fc3a0678b49b18e74c449.zip | |
fixes #234 Investigate enabling more verbose compiler warnings
We enabled verbose compiler warnings, and found a lot of issues.
Some of these were even real bugs. As a bonus, we actually save
some initialization steps in the compat layer, and avoid passing
some variables we don't need.
Diffstat (limited to 'tests/idhash.c')
| -rw-r--r-- | tests/idhash.c | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/tests/idhash.c b/tests/idhash.c index e4183e98..32751334 100644 --- a/tests/idhash.c +++ b/tests/idhash.c @@ -1,6 +1,6 @@ // -// Copyright 2017 Garrett D'Amore <garrett@damore.org> -// Copyright 2017 Capitar IT Group BV <info@capitar.com> +// Copyright 2018 Staysail Systems, Inc. <info@staysail.tech> +// Copyright 2018 Capitar IT Group BV <info@capitar.com> // // This software is supplied under the terms of the MIT License, a // copy of which should be located in the distribution where this @@ -145,13 +145,12 @@ Main({ nni_idhash *h; int expect[5]; uint64_t id; - int i; So(nni_idhash_init(&h) == 0); Reset({ nni_idhash_fini(h); }); nni_idhash_set_limits(h, 10, 13, 10); So(1); Convey("We can fill the table", { - for (i = 0; i < 4; i++) { + for (uint64_t i = 0; i < 4; i++) { So(nni_idhash_alloc( h, &id, &expect[i]) == 0); So(id == (i + 10)); |