bugCERTI - Bugs: bug #23666, Compilation errors with Sun Studio...

 
 

bug #23666: Compilation errors with Sun Studio 12

Submitter:  Petr Gotthard <gotthardp>
Submitted:  Sat 21 Jun 2008 04:42:25 PM UTC
 
Category:  CERTI Severity:  * 3 - Normal
Item Group:  Compilation error Status:  Ready For Test
Privacy:  Public Assigned to:  erk
Open/Closed:  Closed Release:  * CVS
Reproducibility:  Every Time Fixed Release:  * 3.3.0
* Mandatory Fields

Add a New Comment (Rich Markup)
   

Comment Type & Canned Response:
       No canned response available

 

Wed 25 Jun 2008 08:17:55 AM UTC, comment #4: 

CERTI on Solaris seems to run OK.
See ML thread on this subject:
http://lists.nongnu.org/archive/html/certi-devel/2008-06/msg00008.html

Eric NOULARD <erk>
Project Administrator
Mon 23 Jun 2008 01:27:15 PM UTC, comment #3: 

I've just checked-in your patch.

The current CERTI CVS has passed our testsuite on Linux x86.
Will do on Windows box soon.
Could you update and retry compilation with Sun Studio
Thank you.

Eric NOULARD <erk>
Project Administrator
Sun 22 Jun 2008 07:19:10 PM UTC, comment #2: 

Thank you very much Petr,

I will have look at this and possibly
merge tomorrow.

Eric NOULARD <erk>
Project Administrator
Sat 21 Jun 2008 06:41:07 PM UTC, comment #1: 

Sorry, the previous patch was incomplete. Here goes a second version.

(file #15905)

Petr Gotthard <gotthardp>
Project Member
Sat 21 Jun 2008 04:42:25 PM UTC, original submission:  

CERTI cannot be compiled with Sun Studio. Attached patch fixes the problems I've found. (I've used Sun Studio 12 at Linux. I had not yet enough time to install Solaris.)

Fixed problems:
 - function parameter hiding class members
 - copy of map<A,B> to vector<pair<A,B>> cannot be done with std::copy because the A in map is "const", but in vector it's not
 - list::remove_if at SunStudio doesn't accept as Predicate a class with overloaded operator(); I think it's because this remove_if is not a template
 - _func_ is not defined in SunStudio (it needs extra compiler flag)
 - and few missing "const"

Petr Gotthard <gotthardp>
Project Member

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

Attach Files:
   
   
Comment:
   

Attached Files
file #15905:  sunstudio.patch added by gotthardp (22KiB - text/x-patch - Updated version.)
file #15904:  sunstudio.patch added by gotthardp (11KiB - text/x-patch - Proposed fix.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by erk (Posted a comment)
  • -email is unavailable- added by gotthardp (Submitted the item)
  •  

     

     

     

    Follow 8 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2008-07-02 erk Open/ClosedOpen Closed
        Fixed ReleaseCVS 3.3.0
    2008-06-25 erk StatusIn Progress Ready For Test
        Fixed ReleaseUnknown CVS
    2008-06-23 erk StatusNone In Progress
        Assigned toNone erk
    2008-06-21 gotthardp Attached File- Added sunstudio.patch, #15905
    2008-06-21 gotthardp Attached File- Added sunstudio.patch, #15904

    Back to the top


    Powered by Savane 3.9