Closed
Bug 299750
Opened 19 years ago
Closed 19 years ago
compiler warnings about uninitialized variables (gcc4)
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
mozilla1.8beta3
People
(Reporter: wolfiR, Assigned: wolfiR)
References
Details
Attachments
(1 file)
|
2.34 KB,
patch
|
brendan
:
review+
brendan
:
approval1.8b3+
|
Details | Diff | Splinter Review |
There are some compiler warnings about used (not may used) uninitialized variables.
|
Assignee | |
Comment 1•19 years ago
|
||
Assignee: general → mozilla
Status: NEW → ASSIGNED
|
|
Assignee | |
Updated•19 years ago
|
Attachment #188339 -
Flags: review?(shaver)
|
||
Comment 2•19 years ago
|
||
Comment on attachment 188339 [details] [diff] [review] patch to make gcc happy [Checked in: Comment 3] Ah, fdlibm. Anyone who wants to help please apply your efforts more toward fixing bug 291003 and less toward patching this old dog of a library. Thanks, /be
Attachment #188339 -
Flags: review?(shaver)
Attachment #188339 -
Flags: review+
Attachment #188339 -
Flags: approval1.8b3+
checked in
Status: ASSIGNED → RESOLVED
Closed: 19 years ago
Resolution: --- → FIXED
|
||
Updated•19 years ago
|
Attachment #188339 -
Attachment description: patch to make gcc happy → patch to make gcc happy
[Checked in: Comment 3]
|
|
||
Comment 4•19 years ago
|
||
Comment on attachment 188339 [details] [diff] [review] patch to make gcc happy [Checked in: Comment 3] (I don't know about the other two,) but 'z' in <e_rem_pio2.c> was discussed in bug 288993, and it was indicated _not_ to fix it this way... Any comment ?
|
|
||
Comment 5•19 years ago
|
||
Oh, well, 'qx' in <k_cos.c> was discussed there too...
Target Milestone: --- → mozilla1.8beta3
|
|
Assignee | |
Comment 6•19 years ago
|
||
Should we reopen this and back it out? Some JS guru should decide it.
|
||
Updated•19 years ago
|
Flags: testcase-
You need to log in
before you can comment on or make changes to this bug.
Description
•