488979 - |mailnews/base/src/nsMsgBiffManager.cpp(261) : warning C4244: 'initializing' : conversion from 'double' to 'PRInt64', possible loss of data|

Status: RESOLVED FIXED

(MailNews Core :: Backend, defect)

Description

[Serge Gautherie (:sgautherie)]

[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.2a1pre) Gecko/20090417 SeaMonkey/2.0b1pre] (experimental/_m-c_, home, optim default) (W2Ksp4)
(http://hg.mozilla.org/mozilla-central/rev/9e06217fc471
 +http://hg.mozilla.org/comm-central/rev/...)

Code is
{
      PRInt64 jitter = (PRInt64)(chosenTimeInterval) * 0.05;
}

Comment 1

[Serge Gautherie (:sgautherie)]

Attachment: (Av1) Fix nits in SetNextBiffTime() [Checkin: Comment 4]

Comment 2

[Magnus Melin [:mkmelin]]

Comment on attachment 373472 [details] [diff] [review]
(Av1) Fix nits in SetNextBiffTime()
[Checkin: Comment 4]

Not that i'm a /mailnews reviewer... but my compiler doesn't complain about that.

Otherwise I suppose it looks ok.

Comment 3

[Serge Gautherie (:sgautherie)]

bienvenu, ping for review.

Comment 4

[Serge Gautherie (:sgautherie)]

Comment on attachment 373472 [details] [diff] [review]
(Av1) Fix nits in SetNextBiffTime()
[Checkin: Comment 4]


http://hg.mozilla.org/comm-central/rev/a0da9ca1c4b2

Comment 5

[WADA:World Anti-bad-Duping Agency]

Is Bug 509747 regression produced by this bug?