AIX Compilation error for 3.6a1pre source xpcom/io/nsWildCard.cpp", line 215.28: 1540-0274 (S) The name lookup for "_shexp_match" did not find a declaration

RESOLVED FIXED in mozilla1.9.2a1

Status

()

Core
XPCOM
--
major
RESOLVED FIXED
8 years ago
8 years ago

People

(Reporter: Shailen, Assigned: Shailen)

Tracking

Trunk
mozilla1.9.2a1
Other
AIX
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

4.21 KB, patch
Benjamin Smedberg
: review+
Details | Diff | Splinter Review
(Assignee)

Description

8 years ago
User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.10) Gecko/2009042316 Firefox/3.0.10
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9.0.10) Gecko/2009042316 Firefox/3.0.10

"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 215.28: 1540-0274 (S) The name lookup for "_shexp_match" did not find a declaration.
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 215.28: 1540-1292 (I) Static declarations are not considered for a function call if the function is not qualified.
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 193.1: 1540-0700 (I) The previous message was produced while processing "_shexp_match<unsigned short>(const unsigned short *, const unsigned short *, PRBool)".
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 310.8: 1540-0700 (I) The previous message was produced while processing "NS_WildCardMatch_<unsigned short>(const unsigned short *, const unsigned short *, PRBool)".
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 331.12: 1540-0700 (I) The previous message was produced while processing "NS_WildCardMatch(const PRUnichar *, const PRUnichar *, PRBool)".
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 215.28: 1540-0274 (S) The name lookup for "_shexp_match" did not find a declaration.
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 215.28: 1540-1292 (I) Static declarations are not considered for a function call if the function is not qualified.
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 193.1: 1540-0700 (I) The previous message was produced while processing "_shexp_match<char>(const char *, const char *, PRBool)".
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 310.8: 1540-0700 (I) The previous message was produced while processing "NS_WildCardMatch_<char>(const char *, const char *, PRBool)".
"/home/guest/sb/mercurial/src/xpcom/io/nsWildCard.cpp", line 324.12: 1540-0700 (I) The previous message was produced while processing "NS_WildCardMatch(const char *, const char *, PRBool)".


Reproducible: Always

Steps to Reproduce:
1. Build Mozilla Firefox 3.6a1pre build on AIX with Gnome RPMS of 64 bit
version.
2.
3.
Actual Results:  
Build fails with the above error.


Expected Results:  
No build failure
(Assignee)

Updated

8 years ago
OS: Other → AIX
Version: unspecified → Trunk
(Assignee)

Comment 1

8 years ago
Created attachment 382939 [details] [diff] [review]
Patch V 1

I checked with compiler team for this issue and below is the explanation I received for the compiler behaviour.

' In the C++ Standard (ISO/IEC 14882), section 14.6.4.2 "Candidate functions" describes that: for an unqualified function call that depends on a template parameter, only function declarations with external linkage from the template definition context should be found. 

 In this scenario the "static" function results in internal linkage and therefore cannot be found. 

One solution is to make the function non-static by removing "static".

Alternatively we may make the function-call qualified. ' 

The patch submitted has taken alternate approach of making the function-call qualified.
Attachment #382939 - Flags: review?(benjamin)

Updated

8 years ago
Attachment #382939 - Flags: review?(benjamin) → review+

Comment 2

8 years ago
Comment on attachment 382939 [details] [diff] [review]
Patch V 1

That's a really amazing rule I hadn't heard about before!
Assignee: nobody → shailen.n.jain
Component: General → XPCOM
Product: Firefox → Core
QA Contact: general → xpcom

Updated

8 years ago
Status: UNCONFIRMED → NEW
Ever confirmed: true

Updated

8 years ago
Keywords: checkin-needed
http://hg.mozilla.org/mozilla-central/rev/c2e179151f9c
Status: NEW → RESOLVED
Last Resolved: 8 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.2a1
Blocks: 524271
You need to log in before you can comment on or make changes to this bug.