diff options
author | Brian Norris <briannorris@chromium.org> | 2016-07-22 10:17:56 -0700 |
---|---|---|
committer | chrome-bot <chrome-bot@chromium.org> | 2016-07-25 21:01:46 -0700 |
commit | c61df529d073ee2440d3b80c0a58a85a92d98e8f (patch) | |
tree | c0b4dea397c4a8072646f6ff403c9937ebad30b2 /utility | |
parent | 1d4c49b94fb426e429312582188b4858f6b241e8 (diff) | |
download | vboot-c61df529d073ee2440d3b80c0a58a85a92d98e8f.tar.gz |
bmpblk_utility: fix build warnings (errors) with USE="-minimal"
Warning like this:
utility/bmpblk_utility.cc: In member function 'void vboot_reference::BmpBlockUtil::load_yaml_config(const char*)':
utility/bmpblk_utility.cc:104:61: error: format '%ld' expects argument of type 'long int', but argument 2 has type 'std::vector<std::basic_string<char> >::size_type {aka unsigned int}' [-Werror=format=]
printf("%ld image_names\n", config_.image_names.size());
^
utility/bmpblk_utility.cc:108:59: error: format '%ld' expects argument of type 'long int', but argument 2 has type 'std::map<std::basic_string<char>, vboot_reference::ImageConfig>::size_type {aka unsigned int}' [-Werror=format=]
printf("%ld images_map\n", config_.images_map.size());
^
utility/bmpblk_utility.cc:119:61: error: format '%ld' expects argument of type 'long int', but argument 2 has type 'std::map<std::basic_string<char>, vboot_reference::ScreenConfig>::size_type {aka unsigned int}' [-Werror=format=]
printf("%ld screens_map\n", config_.screens_map.size());
^
utility/bmpblk_utility.cc: In member function 'void vboot_reference::BmpBlockUtil::pack_bmpblock()':
utility/bmpblk_utility.cc:597:70: error: format '%ld' expects argument of type 'long int', but argument 3 has type 'std::basic_string<char>::size_type {aka unsigned int}' [-Werror=format=]
current_offset, it->second.compressed_content.length());
^
utility/bmpblk_utility.cc:609:59: error: format '%ld' expects argument of type 'long int', but argument 3 has type 'std::basic_string<char>::size_type {aka unsigned int}' [-Werror=format=]
current_offset, config_.locale_names.size());
^
Technically, C++ container::size_type is slightly different than size_t, but I
think %zd should work fine.
BRANCH=none
BUG=none
TEST=`USE="-minimal" emerge-kevin vboot_reference`
Change-Id: I9d0b22818714cad2ad7e95557df7837c91c2b0f7
Signed-off-by: Brian Norris <briannorris@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/362621
Reviewed-by: Randall Spangler <rspangler@chromium.org>
Diffstat (limited to 'utility')
-rw-r--r-- | utility/bmpblk_utility.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/utility/bmpblk_utility.cc b/utility/bmpblk_utility.cc index 5069e4a8..51a35eed 100644 --- a/utility/bmpblk_utility.cc +++ b/utility/bmpblk_utility.cc @@ -101,11 +101,11 @@ namespace vboot_reference { // The number of localizations should match the number of locale_index if (debug_) { - printf("%ld image_names\n", config_.image_names.size()); + printf("%zd image_names\n", config_.image_names.size()); for (unsigned int i = 0; i < config_.image_names.size(); ++i) { printf(" %d: \"%s\"\n", i, config_.image_names[i].c_str()); } - printf("%ld images_map\n", config_.images_map.size()); + printf("%zd images_map\n", config_.images_map.size()); for (StrImageConfigMap::iterator it = config_.images_map.begin(); it != config_.images_map.end(); ++it) { @@ -116,7 +116,7 @@ namespace vboot_reference { it->second.data.tag, it->second.data.format); } - printf("%ld screens_map\n", config_.screens_map.size()); + printf("%zd screens_map\n", config_.screens_map.size()); for (StrScreenConfigMap::iterator it = config_.screens_map.begin(); it != config_.screens_map.end(); ++it) { @@ -593,7 +593,7 @@ namespace vboot_reference { current_filled += sizeof(it->second.data); current_offset += sizeof(it->second.data); if (debug_) - printf("I1: current offset is 0x%08x (len %ld)\n", + printf("I1: current offset is 0x%08x (len %zd)\n", current_offset, it->second.compressed_content.length()); std::copy(it->second.compressed_content.begin(), it->second.compressed_content.end(), @@ -605,7 +605,7 @@ namespace vboot_reference { current_offset = config_.header.locale_string_offset; current_filled = bmpblock_.begin() + current_offset; if (debug_) - printf("locale_names: offset 0x%08x (len %ld)\n", + printf("locale_names: offset 0x%08x (len %zd)\n", current_offset, config_.locale_names.size()); std::copy(config_.locale_names.begin(), config_.locale_names.end(), |