In <nsPresContext.h>, 4 "warning C4244: 'argument' : conversion from 'PRInt32' to 'float', possible loss of data"

VERIFIED FIXED in mozilla1.9.1b2

Status

()

--
minor
VERIFIED FIXED
10 years ago
10 years ago

People

(Reporter: sgautherie, Assigned: masayuki)

Tracking

Trunk
mozilla1.9.1b2
x86
Windows 2000
Points:
---
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

10 years ago
[Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9.1b1pre) Gecko/20080917032624 Minefield/3.1b1pre] (home, optim default) (W2Ksp4)

{
objdir\dist\include\layout\nsPresContext.h(528) : warning C4244: 'argument' : conversion from 'PRInt32' to 'float', possible loss of data
objdir\dist\include\layout\nsPresContext.h(532) : warning C4244: 'argument' : conversion from 'PRInt32' to 'float', possible loss of data
objdir\dist\include\layout\nsPresContext.h(536) : warning C4244: 'argument' : conversion from 'PRInt32' to 'float', possible loss of data
objdir\dist\include\layout\nsPresContext.h(547) : warning C4244: 'argument' : conversion from 'PRInt32' to 'float', possible loss of data
}
Created attachment 343184 [details] [diff] [review]
Patch v1.0

simple fix.
Assignee: nobody → masayuki
Status: NEW → ASSIGNED
Attachment #343184 - Flags: superreview?(roc)
Attachment #343184 - Flags: review?(roc)
Attachment #343184 - Flags: superreview?(roc)
Attachment #343184 - Flags: superreview+
Attachment #343184 - Flags: review?(roc)
Attachment #343184 - Flags: review+
(Reporter)

Comment 2

10 years ago
Ftr, isn't this more of a wallpaper than a fix ?
checked-in.

(In reply to comment #2)
> Ftr, isn't this more of a wallpaper than a fix ?

The patch should not have problems. Because the values are not such large value.
oops,

-> FIXED
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
(Reporter)

Comment 5

10 years ago
Good, that's just what I wanted to know.

***

V.Fixed.
Status: RESOLVED → VERIFIED
Flags: in-testsuite-
Target Milestone: --- → mozilla1.9.1b2
You need to log in before you can comment on or make changes to this bug.