307041 - nsGREGlue.cpp change breaks Solaris Studio 10 build

Status: RESOLVED FIXED

(Core :: XPCOM, defect)

Description

[km]

User-Agent:       Mozilla/5.0 (X11; U; SunOS sun4u; en-US; rv:1.9a1) Gecko/20050831 Firefox/1.6a1
Build Identifier: 

The changes in 1.12 broke the Solaris Studio 10 build over the definition of
INIClosure. The errors are:

"mozilla/xpcom/glue/nsGREGlue.cpp", line 448: Error: A class with a reference
member must have a user-defined constructor.
"mozilla/xpcom/glue/nsGREGlue.cpp", line 495: Error: Expected an expression.
"mozilla/xpcom/glue/nsGREGlue.cpp", line 501: Error: Use ";" to terminate
statements.
3 Error(s) detected.


Any suggestions? 

Reproducible: Always

Comment 1

[Frank Wein [:mcsmurf]]

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

Comment 2

[Mitch]

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

Comment 3

[Stephen Donner [:stephend] Not actively reading bugmail]

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

Comment 4

[Doug Turner (:dougt)]

Attachment: patch v.1

Comment 5

[Darin Fisher]

Comment on attachment 197445 [details] [diff] [review]
patch v.1

> GRE_GetPathFromConfigFile(const char* filename,
...
> {
>   nsINIParser parser;
>+  nsresult rv = parser->Init(filename);

Oops.  Wrong patch?

Comment 6

[Doug Turner (:dougt)]

Comment on attachment 197445 [details] [diff] [review]
patch v.1

wow.

Comment 7

[Doug Turner (:dougt)]

Attachment: patch v.2

Comment 8

[Darin Fisher]

Shouldn't there be some code to actually initialize INIClosure::parser?  It
looks like you're missing changes to GRE_GetPathFromConfigFile.

Comment 9

[Peter Müller]

(In reply to comment #7)
> Created an attachment (id=197458) [edit]
> patch v.2 

after appliing patch v.2:
...
"nsGREGlue.cpp", line 524: Error: Cannot use nsINIParser to initialize nsINIParser*.
...

Comment 10

[Peter Müller]

(In reply to comment #9)

> after applying patch v.2:

and

% diff p2/nsGREGlue.cpp  p3/nsGREGlue.cpp
518,519c518,519
<   nsINIParser parser;
<   nsresult rv = parser.Init(filename);
---
>   nsINIParser *parser;
>   nsresult rv = parser->Init(filename);
531c531
<   parser.GetSections(CheckINIHeader, &c);
---
>   parser->GetSections(CheckINIHeader, &c);

the compiler (CC: Sun C++ 5.7 Patch 117830-03 2005/07/21) 
seems to be satisfied 8-)

Comment 11

[Wyllys Ingersoll]

(In reply to comment #10)
> (In reply to comment #9)
> 
> > after applying patch v.2:
> 
> and
> 
> % diff p2/nsGREGlue.cpp  p3/nsGREGlue.cpp
> 518,519c518,519
> <   nsINIParser parser;
> <   nsresult rv = parser.Init(filename);
> ---
> >   nsINIParser *parser;
> >   nsresult rv = parser->Init(filename);
> 531c531
> <   parser.GetSections(CheckINIHeader, &c);
> ---
> >   parser->GetSections(CheckINIHeader, &c);
> 
> the compiler (CC: Sun C++ 5.7 Patch 117830-03 2005/07/21) 
> seems to be satisfied 8-)


I ran into this same problem today and the above fixes seem to work
for both Studio 8 and Studio 10 compilers.

Comment 12

[Leon Sha]

(In reply to comment #10)
> >   nsINIParser *parser;
> >   nsresult rv = parser->Init(filename);
It is not correct.
"parser" here is just a pointer without initialise.

Comment 13

[Leon Sha]

Attachment: Patch v3

Comment 14

[Peter Müller]

(In reply to comment #13)
> Created an attachment (id=199044) [edit]
> Patch v3

Patch v3 is OK for the compiler mentioned above :-)

Comment 15

[Benjamin Smedberg]

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

Comment 16

[Iain MacDonnell]

(In reply to comment #15)
> *** Bug 311924 has been marked as a duplicate of this bug. ***

Argh! One of these days I'll learn how to query :/

Anyway, patch v3 works fine for me. Thanks!

Comment 17

[Mats Larsson]

(In reply to comment #16)

> Argh! One of these days I'll learn how to query :/

But that's hard work to learn ;).

> Anyway, patch v3 works fine for me. Thanks!

Works fine for me too.

Comment 18

[Doug Turner (:dougt)]

Checking in nsGREGlue.cpp;
/cvsroot/mozilla/xpcom/glue/nsGREGlue.cpp,v  <--  nsGREGlue.cpp
new revision: 1.15; previous revision: 1.14
done

Comment 19

[Benjamin Smedberg]

Should get this on 1.8branch as well.

Comment 20

[Darin Fisher]

fixed1.8

Comment 21

[Scott MacGregor]

belated flag cleanup

Comment 22

[Benjamin Smedberg]

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