diff options
| author | Rémi Verschelde <rverschelde@gmail.com> | 2019-09-23 10:00:31 +0200 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2019-09-23 10:00:31 +0200 |
| commit | 393a0152eaccaf21313b6186adeea1cf6a7f0623 (patch) | |
| tree | 28044bd643418b4293494391b3c1d32163126dd8 /platform/osx/export/export.cpp | |
| parent | 791da4cb83bda1b3ef8dc167e3db7cccf10e8762 (diff) | |
| parent | 50be65bf4314d9b3c5888f5f02d6ffae504aae7a (diff) | |
Merge pull request #32055 from qarmin/some_code_fixes
Changed some code found by Clang Tidy and Coverity
Diffstat (limited to 'platform/osx/export/export.cpp')
| -rw-r--r-- | platform/osx/export/export.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/platform/osx/export/export.cpp b/platform/osx/export/export.cpp index 56b0a44dbc..94090bcdc1 100644 --- a/platform/osx/export/export.cpp +++ b/platform/osx/export/export.cpp @@ -240,7 +240,7 @@ void EditorExportPlatformOSX::_make_icon(const Ref<Image> &p_icon, Vector<uint8_ { "is32", "s8mk", false, 16 } //16x16 24-bit RLE + 8-bit uncompressed mask }; - for (unsigned int i = 0; i < (sizeof(icon_infos) / sizeof(icon_infos[0])); ++i) { + for (uint64_t i = 0; i < (sizeof(icon_infos) / sizeof(icon_infos[0])); ++i) { Ref<Image> copy = p_icon; // does this make sense? doesn't this just increase the reference count instead of making a copy? Do we even need a copy? copy->convert(Image::FORMAT_RGBA8); copy->resize(icon_infos[i].size, icon_infos[i].size); |