Closed Bug 672951 Opened 10 years ago Closed 10 years ago

[OS/2] build break due to mismatch in function - follow-up to bug668758

Categories

(Core :: Layout: Text and Fonts, defect)

x86
OS/2
defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla8

People

(Reporter: wuno, Assigned: wuno)

References

Details

(Whiteboard: [inbound])

Attachments

(1 file)

E:/hg-src/hg/comm-central/mozilla/gfx/thebes/gfxOS2Fonts.cpp: In constructor 'gfxOS2FontGroup::gfxOS2FontGroup(const nsAString_internal&, const gfxFontStyle*, gfxUserFont Set*)':
E:/hg-src/hg/comm-central/mozilla/gfx/thebes/gfxOS2Fonts.cpp:543: error: no matching function for call to 'gfxOS2FontGroup::ForEachFont(PRBool (&)(const nsAString_internal&, const nsACString_internal&, void*), nsTArray<nsString, nsTArrayDefaultAllocator>*)'
E:/hg-src/hg/comm-central/mozilla/gfx/thebes/gfxFont.h:2349: note: candidates are: PRBool gfxFontGroup::ForEachFont(const nsAString_internal&, nsIAtom*, PRBool (*)(constnsAString_internal&, const nsACString_internal&, PRBool, void*), void*)
after checkin for bug668758
Blocks: 668758
Attached patch fixSplinter Review
Though not used, we need it to compile
Assignee: nobody → wuno
Status: NEW → ASSIGNED
Attachment #547253 - Flags: review?(dragtext)
Attachment #547253 - Flags: review?(dragtext) → review+
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/b4ef09ae5795
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.