210297 - T * operator of nsAutoPtr doesn't work with MIPSpro 7.3(IRIX) and Cxx(Tru64)

Status: RESOLVED WONTFIX

(Core :: XPCOM, defect)

Description

[avi bercovich]

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.3.1) Gecko/20030425
Build Identifier: CVS pull 21-06-2003

After some conversation on irc.mozilla.org, the resident guru's
(timeles|bz|biesi) decided that what I was seeing is a bug in XPCOMM. Here's the
exact make trace at the point of breakage:

cc-1278 CC: ERROR File = nsUTF8ConverterService.cpp, Line = 80
  No instance of overloaded function "Substring" matches the argument list.

            The argument types are:  (nsAutoPtr<PRUnichar>, PRUnichar*).
      CopyUTF16toUTF8(Substring(ustr, ustr + dstLen), aResult);
                      ^

1 error detected in the compilation of "nsUTF8ConverterService.cpp".


Reproducible: Always

Steps to Reproduce:
gmake 





Using MIPSpro 7.3 with gmake 3.79.1, on r10k O2, and IRIX 6.5.19m

Comment 1

[Boris Zbarsky [:bzbarsky]]

So the question is why the T* operator is not working.... there are some
comments about Irix and ambiguity issues in nsAutoPtr.h/nsCOMPtr.h, but none in
the relevant part of the file...

Comment 2

[Matthias Versen [:Matti]]

*** Bug 210299 has been marked as a duplicate of this bug. ***

Comment 3

[Christian :Biesinger (don't email me, ping me on IRC)]

this was probably worked around by the checkin for bug 209328

Comment 4

[Jungshik Shin]

I guess Christian is right about my work-around.

 Cxx compiler on Tru64 has the same problem. Perhaps, the summary line has to be
changed to " T * operator os nsAutoPtr doesn't work with MIPSpro 7.3(Irix) and
Cxx(Tru64)'.  

Comment 5

[Jungshik Shin]

changing the summary line and adding [e-mail] to CC for Cxx
on Tru64.

Comment 6

[Jungshik Shin]

Attachment: 1-liner to use get() instead of casting to work around the problem

Per Boris' comment (bug 209328 comment #32). As soon as I get r/sr and the tree
is opened, I'll land this.

Comment 7

[Boris Zbarsky [:bzbarsky]]

I'd like to see dbaron's and sicking's (and scc's, if he's reading bugmail)
comments before we check anything in here, please.

Comment 8

[David Baron :dbaron: (⌚️UTC-4, no longer working on Mozilla)]

I probably prefer the .get() workaround.  I had to check in similar workarounds
back when we had the global GetStyleData function template.  It would be nice to
have an IRIX tinderbox so that we can see these things...

Comment 9

[Jonas Sicking (:sicking) No longer reading bugmail consistently]

yeah, the .get() looks better to me too. If this happens often for Substring we
should consider adding a version that takes an nsAutoPtr/nsAutoArrayPtr, but
lets cross that bridge if we get there.

Comment 10

[Victor Riley]

I understand from cls that the IRIX tinderbox system has a problem so is
offline till it can get fixed.  In the mean time, SGI will work on setting
up a new tinderbox.

	Victor Riley

Comment 11

[Matthias Versen [:Matti]]

IRIX build support got removed with bug 680792