Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(43)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: base/third_party/dmg_fp/g_fmt.cc

Issue 590413006: Fix more MSVC warnings, this time mostly about possible value truncation (fixed (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« base/third_party/dmg_fp/dtoa.cc ('K') | « base/third_party/dmg_fp/dtoa.cc ('k') | base/third_party/dmg_fp/msvc_warnings.patch » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: base/third_party/dmg_fp/g_fmt.cc
diff --git a/base/third_party/dmg_fp/g_fmt.cc b/base/third_party/dmg_fp/g_fmt.cc
index e2fb7bd30dcb4f50f0b16f6789844762746d576a..bfa358d154c7d5cb2bead52329fc22af38687280 100644
--- a/base/third_party/dmg_fp/g_fmt.cc
+++ b/base/third_party/dmg_fp/g_fmt.cc
@@ -67,7 +67,7 @@ g_fmt(register char *b, double x)
for(j = 2, k = 10; 10*k <= decpt; j++, k *= 10) {}
for(;;) {
i = decpt / k;
- *b++ = i + '0';
+ *b++ = (char)i + '0';
if (--j <= 0)
break;
decpt -= i*k;
« base/third_party/dmg_fp/dtoa.cc ('K') | « base/third_party/dmg_fp/dtoa.cc ('k') | base/third_party/dmg_fp/msvc_warnings.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698