mingw build error - error: no matching function for call to `nsAutoString::nsAutoString(jschar*)'

RESOLVED FIXED

Status

()

Core
XPConnect
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Martijn Wargers (dead), Assigned: Martijn Wargers (dead))

Tracking

Trunk
x86
Windows XP
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

(Assignee)

Description

11 years ago
Created attachment 274358 [details] [diff] [review]
patch

I get this build error in my mingw debug build.
The attached patch fixes it for me.

G_READ -DPNG_NO_MMX_CODE -DMOZ_PNG_WRITE   -I../../../../dist/sdk/include
-fno-rtti -fno-exceptions -Wall -Wconversion -Wpointer-arith -Wcast-align -Wover
loaded-virtual -Wsynth -Wno-ctor-dtor-privacy -Wno-non-virtual-dtor -Wno-long-lo
ng -pedantic -mms-bitfields -pipe  -DDEBUG -D_DEBUG -DDEBUG_mw -DTRACING -g   -D
MOZILLA_CLIENT -include ../../../../mozilla-config.h /cygdrive/c/mozilla/mozilla
/js/src/xpconnect/src/XPCCrossOriginWrapper.cpp
c:/mozilla/mozilla/js/src/xpconnect/src/XPCCrossOriginWrapper.cpp: In function `
JSBool IsValFrame(JSContext*, JSObject*, jsval, XPCWrappedNative*)':
c:/mozilla/mozilla/js/src/xpconnect/src/XPCCrossOriginWrapper.cpp:196: error: no
 matching function for call to `nsAutoString::nsAutoString(jschar*)'
../../../../dist/include/string/nsTString.h:534: note: candidates are: nsAutoStr
ing::nsAutoString(const nsSubstringTuple&)
../../../../dist/include/string/nsTString.h:528: note:                 nsAutoStr
ing::nsAutoString(const nsAString_internal&)
../../../../dist/include/string/nsTString.h:521: note:                 nsAutoStr
ing::nsAutoString(const nsAutoString&)
../../../../dist/include/string/nsTString.h:515: note:                 nsAutoStr
ing::nsAutoString(const PRUnichar*, PRUint32) <near match>
../../../../dist/include/string/nsTString.h:508: note:                 nsAutoStr
ing::nsAutoString(PRUnichar) <near match>
../../../../dist/include/string/nsTString.h:503: note:                 nsAutoStr
ing::nsAutoString()
make[5]: *** [XPCCrossOriginWrapper.o] Error 1
make[5]: Leaving directory `/cygdrive/c/mozilla/mozilla/js/src/xpconnect/src'
make[4]: *** [libs] Error 2
make[4]: Leaving directory `/cygdrive/c/mozilla/mozilla/js/src/xpconnect'
make[3]: *** [libs_tier_gecko] Error 2
make[3]: Leaving directory `/cygdrive/c/mozilla/mozilla'
make[2]: *** [tier_gecko] Error 2
make[2]: Leaving directory `/cygdrive/c/mozilla/mozilla'
make[1]: *** [alldep] Error 2
make[1]: Leaving directory `/cygdrive/c/mozilla/mozilla'
make: *** [alldep] Error 2
Attachment #274358 - Flags: superreview?(mrbkap)
Attachment #274358 - Flags: review?(mrbkap)
Comment on attachment 274358 [details] [diff] [review]
patch

nit: remove the space after the > and move the parenthesized expression to the next line (under the <) to preserve the 80th column.
Attachment #274358 - Flags: superreview?(mrbkap)
Attachment #274358 - Flags: superreview?(jst)
Attachment #274358 - Flags: review?(mrbkap)
Attachment #274358 - Flags: review+
(Assignee)

Comment 2

11 years ago
Created attachment 274402 [details] [diff] [review]
patchv2
Attachment #274358 - Attachment is obsolete: true
Attachment #274402 - Flags: superreview?(jst)
Attachment #274358 - Flags: superreview?(jst)

Updated

11 years ago
Attachment #274402 - Flags: superreview?(jst) → superreview+
(Assignee)

Updated

11 years ago
Attachment #274402 - Flags: approval1.9?
(Assignee)

Updated

11 years ago
Assignee: nobody → martijn.martijn

Updated

11 years ago
Attachment #274402 - Flags: approval1.9? → approval1.9+
(Assignee)

Comment 3

11 years ago
Checking in XPCCrossOriginWrapper.cpp;
/cvsroot/mozilla/js/src/xpconnect/src/XPCCrossOriginWrapper.cpp,v  <--  XPCCross
OriginWrapper.cpp
new revision: 1.7; previous revision: 1.6
done

Checked into trunk.
Status: NEW → RESOLVED
Last Resolved: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.