Closed Bug 351472 Opened 18 years ago Closed 18 years ago

Building trunk with GCC 3.2.2 on OS/2 fails in java_xpcom.c

Categories

(Core Graveyard :: Java to XPCOM Bridge, defect)

x86
OS/2
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mozilla, Assigned: mozilla)

References

Details

(Keywords: fixed1.8.1)

Attachments

(1 file)

Following the checkin from bug 333618 java_xpcom.c does not compile any more on OS/2 with GCC 3.2.2:

I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c: In function 
`interface_declaration':
I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: `PATH_MAX' undeclared 
(first use in this function)
I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: (Each undeclared 
identifier is reported only once
I:/mozilla/xpcom/typelib/xpidl/xpidl_java.c:253: for each function it 
appears in.)

It seems that GCC 3.2.2 does not define MAX_PATH by default.
Attached patch the fixSplinter Review
This should fix it.

But I wonder if the same patch is needed for extensions/java/xpcom/tools/xpidl/xpidl_java.c.
Assignee: jhpedemonte → mozilla
Status: NEW → ASSIGNED
Attachment #236865 - Flags: review?(jhpedemonte)
Blocks: 333618
Comment on attachment 236865 [details] [diff] [review]
the fix

Looks good.  None of the code in extensions/java/xpcom/tools is used any more after bug 333618, so don't worry about it.
Attachment #236865 - Flags: review?(jhpedemonte) → review+
Checked into trunk.
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
Comment on attachment 236865 [details] [diff] [review]
the fix

Not that the respective patch for bug 333618 went into MOZILLA_1_8_BRANCH we should get this in, too.
Attachment #236865 - Flags: approval1.8.1?
Comment on attachment 236865 [details] [diff] [review]
the fix

a=schrep for os2 build fix.
Attachment #236865 - Flags: approval1.8.1? → approval1.8.1+
Keywords: fixed1.8.1
Product: Core → Core Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: