FormatTimeInMillisAsSeconds mixes floating-point and integer values

[GoogleCodeExporter]

In gtest-1.6.0, FormatTimeInMillisAsSeconds() divides an integer parameter 
("ms") by a floating-point constant (1000.0). This generates a warning under 
gcc-4.6.2.

* What steps will reproduce the problem?
1. Compile gtest-all.cc under gcc-4.6.2

* What is the expected output? What do you see instead?
I do not expect to see a warning. I do see a warning:

gtest-all.cc:4502:12: warning: conversion to 'double' from 
'testing::internal::TimeInMillis {aka long long int}' may alter its value 
[-Wconversion]

* What version of Google Test are you using? On what operating system?
gcc-4.6.2 under Ubuntu 10.04.

Original issue reported on code.google.com by mark.thu...@linguamatics.com on 6 Mar 2012 at 11:08

[GoogleCodeExporter]

This patch should fix the warning.

Original comment by dirk....@gmail.com on 23 Apr 2012 at 9:36

Attachments:

• diff