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

 
 

bug #23666: Compilation errors with Sun Studio 12

Submitted by:  Petr Gotthard <gotthardp>
Submitted on:  Sat 21 Jun 2008 04:42:25 PM UTC  
 
Category: CERTISeverity: 3 - Normal
Item Group: Compilation errorStatus: Ready For Test
Privacy: PublicAssigned to: Eric NOULARD <erk>
Open/Closed: ClosedRelease: CVS
Reproducibility: Every TimeFixed Release: 3.3.0

Add a New Comment(Rich Markup)
   

You are not logged in

Please log in, so followups can be emailed to you.

 

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 AdministratorIn charge of this item.
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 AdministratorIn charge of this item.
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 AdministratorIn charge of this item.
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 File(s):
   
   
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
  • -unavailable- added by erk (Posted a comment)
  • -unavailable- added by gotthardp (Submitted the item)
  •  

    Please enter the title of George Orwell's famous dystopian book (it's a date):

     

     

    Follow 8 latest changes.

    Date Changed By Updated Field Previous Value => Replaced By
    Wed 02 Jul 2008 08:49:57 AM UTCerkOpen/ClosedOpen=>Closed
      Fixed ReleaseCVS=>3.3.0
    Wed 25 Jun 2008 08:17:55 AM UTCerkStatusIn Progress=>Ready For Test
      Fixed ReleaseUnknown=>CVS
    Mon 23 Jun 2008 01:27:15 PM UTCerkStatusNone=>In Progress
      Assigned toNone=>erk
    Sat 21 Jun 2008 06:41:07 PM UTCgotthardpAttached File-=>Added sunstudio.patch, #15905
    Sat 21 Jun 2008 04:42:25 PM UTCgotthardpAttached File-=>Added sunstudio.patch, #15904

    Back to the top


    Powered by Savane 3.1-cleanup1