diff options
| author | Freddie Chopin <freddie.chopin@gmail.com> | 2018-07-11 13:32:21 +0300 |
|---|---|---|
| committer | Freddie Chopin <freddie.chopin@gmail.com> | 2018-07-11 13:32:21 +0300 |
| commit | 7e67f9324e1fb0a910244baac60121e945566639 (patch) | |
| tree | a769f8a2f4bb0a6abf116e2d96a324cdf54be242 /emubd | |
| parent | 5a17fa42e40b7f4943a36e117fcf01d280c30f63 (diff) | |
Use PRIu32 and PRIx32 format specifiers to fix warnings
When using "%d" or "%x" with uint32_t types, arm-none-eabi-gcc reports
warnings like below:
-- >8 -- >8 -- >8 -- >8 -- >8 -- >8 --
In file included from lfs.c:8:
lfs_util.h:45:12: warning: format '%d' expects argument of type 'int', but argument 4 has type 'lfs_block_t' {aka 'long unsigned int'} [-Wformat=]
printf("lfs debug:%d: " fmt "\n", __LINE__, __VA_ARGS__)
^~~~~~~~~~~~~~~~
lfs.c:2512:21: note: in expansion of macro 'LFS_DEBUG'
LFS_DEBUG("Found partial move %d %d",
^~~~~~~~~
lfs.c:2512:55: note: format string is defined here
LFS_DEBUG("Found partial move %d %d",
~^
%ld
-- >8 -- >8 -- >8 -- >8 -- >8 -- >8 --
Fix this by replacing "%d" and "%x" with `"%" PRIu32` and `"%" PRIx32`.
Diffstat (limited to 'emubd')
| -rw-r--r-- | emubd/lfs_emubd.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/emubd/lfs_emubd.c b/emubd/lfs_emubd.c index ed3414a..682ad92 100644 --- a/emubd/lfs_emubd.c +++ b/emubd/lfs_emubd.c @@ -16,6 +16,7 @@ #include <unistd.h> #include <assert.h> #include <stdbool.h> +#include <inttypes.h> // Block device emulated on existing filesystem @@ -85,7 +86,7 @@ int lfs_emubd_read(const struct lfs_config *cfg, lfs_block_t block, memset(data, 0, size); // Read data - snprintf(emu->child, LFS_NAME_MAX, "%x", block); + snprintf(emu->child, LFS_NAME_MAX, "%" PRIx32, block); FILE *f = fopen(emu->path, "rb"); if (!f && errno != ENOENT) { @@ -124,7 +125,7 @@ int lfs_emubd_prog(const struct lfs_config *cfg, lfs_block_t block, assert(block < cfg->block_count); // Program data - snprintf(emu->child, LFS_NAME_MAX, "%x", block); + snprintf(emu->child, LFS_NAME_MAX, "%" PRIx32, block); FILE *f = fopen(emu->path, "r+b"); if (!f) { @@ -171,7 +172,7 @@ int lfs_emubd_erase(const struct lfs_config *cfg, lfs_block_t block) { assert(block < cfg->block_count); // Erase the block - snprintf(emu->child, LFS_NAME_MAX, "%x", block); + snprintf(emu->child, LFS_NAME_MAX, "%" PRIx32, block); struct stat st; int err = stat(emu->path, &st); if (err && errno != ENOENT) { @@ -239,4 +240,3 @@ int lfs_emubd_sync(const struct lfs_config *cfg) { return 0; } - |