Closed Bug 459429 Opened 17 years ago Closed 17 years ago

OS/2 build brakes after rename of npupp.h to npfunctions.h

Categories

(Core Graveyard :: Plug-ins, defect)

x86
OS/2
defect
Not set
major

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: wuno, Assigned: mozilla)

References

Details

Attachments

(1 obsolete file)

User-Agent: Mozilla/5.0 (OS/2; U; Warp 4.5; en-US; rv:1.9.1b2pre) Gecko/20081010 Minefield/3.1b2pre Build Identifier: E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:65:19: error: npupp.h: No such file or directory E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp: In member function 'virtual nsresult UpOutputStream::WriteSegments(nsresult (*)(nsIOutputStream*, void*, char*, PRUint32, PRUint32, PRUint32*), void*, PRUint32, PRUint32*)': E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:4035: warning: overflow in implicit constant conversion E:/usr/src/hg/comm-central/mozilla/modules/plugin/os2wrapper/wrap_XPCOM.cpp:4039: warning: overflow in implicit constant conversion [snip] modules/plugin/os2wrapper/wrap_XPCOM.cpp doesn't find the renamed header. Reproducible: Always Steps to Reproduce: 1. 2. 3.
Attached patch correct the header designation (obsolete) — Splinter Review
Attachment #342638 - Flags: review?(mozilla)
Depends on: 455458
Version: unspecified → Trunk
Removing that include line was already enough for me, seems that we don't use the content of that file (any more).
Comment on attachment 342638 [details] [diff] [review] correct the header designation I checked in the line removal instead, see http://hg.mozilla.org/mozilla-central/rev/18ad520738e5
Attachment #342638 - Attachment is obsolete: true
Attachment #342638 - Flags: review?(mozilla)
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Assignee: nobody → mozilla
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: