PR_CALLBACK missing in layout/html/base/nsPresShell.cpp

VERIFIED FIXED in mozilla0.8.1

Status

()

VERIFIED FIXED
18 years ago
18 years ago

People

(Reporter: achimha, Assigned: rods)

Tracking

Trunk
mozilla0.8.1
Other
Other
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

18 years ago
Hashtable callback functions without the proper callback flag. This is
a build blocker for OS/2 and must go in before branch. Patch follows.
(Reporter)

Comment 1

18 years ago
Created attachment 27686 [details] [diff] [review]
proposed patch

Comment 2

18 years ago
Shouldn't that be PR_STATIC_CALLBACK(PRIntn) then?
yes, it should be PR_STATIC_CALLBACK if it is needed.
Created attachment 27731 [details] [diff] [review]
Use PR_STATIC_CALLBACK rather than PR_CALLBACK

Comment 5

18 years ago
Reassigning to Rods.
Assignee: karnaze → rods
Target Milestone: --- → mozilla0.9
We're not sure why this isn't breaking the tinderbox. It's definitely breaking 
our builds.

The change does not impact other platforms because PR_STATIC_CALLBACK is only 
set for OS/2.

Comment 7

18 years ago
r=jag
(Assignee)

Comment 8

18 years ago
changing target milestone to 0.8.1
Status: NEW → ASSIGNED
Target Milestone: mozilla0.9 → mozilla0.8.1
I got approval from pchen on IRC to get this in.

Checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 18 years ago
Resolution: --- → FIXED

Comment 10

18 years ago
Marking verified per last comments.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.