patchWarzone 2100 Project - Patches: patch #840, Remove all -Wno-XXXs from...

 
 
Show feedback again

You are not allowed to post comments on this tracker with your current authentification level.

patch #840: Remove all -Wno-XXXs from configure.ac

Submitted by:  Christian Ohm <cybersphinx>
Submitted on:  Fri 09 Nov 2007 01:51:38 PM UTC  
 
Category: FixPriority: 5 - Normal
Status: NonePrivacy: Public
Assigned to: Dennis Schridde <devurandom>Open/Closed: Open
Planned Release: None

(Jump to the original submission Jump to the original submission)

Fri 07 Mar 2008 11:08:55 AM UTC, comment #10:

tagfiles are not progressing too nicely, so this needs to be fixed till then.
There are also a lot of warnings about type punning in release (-O[123]) compiles, which want to be fixed. :)

Dennis Schridde <devurandom>
Project AdministratorIn charge of this item.
Fri 09 Nov 2007 10:46:19 PM UTC, comment #9:

Everything but src/game.c is now ptr<->int clean...
Maybe a few conversion mistakes happened, but they'll hopefully be found.
Hopefully Per will commit the tagfile stuff soon, so we can remove game.c uglyness...

Dennis Schridde <devurandom>
Project AdministratorIn charge of this item.
Fri 09 Nov 2007 10:17:51 PM UTC, comment #8:

I am already working on this... I "will do". ;)

Dennis Schridde <devurandom>
Project AdministratorIn charge of this item.
Fri 09 Nov 2007 09:42:49 PM UTC, comment #7:

Closing this one for now; as warning suppression of integer casts is still necessary (on 64bit systems) to be able to compile.

Giel van Schijndel <muggenhor>
Project Member
Fri 09 Nov 2007 09:38:32 PM UTC, comment #6:

Yes, the suppression of warnings is actually intended for 64bit systems.

Giel van Schijndel <muggenhor>
Project Member
Fri 09 Nov 2007 08:21:27 PM UTC, comment #5:

Oh, I guess you're on 64 bit, right? I still run a 32 bit system. Didn't think of that.

Christian Ohm <cybersphinx>
Project Member
Fri 09 Nov 2007 08:19:26 PM UTC, comment #4:

Doesn't happen here...

Christian Ohm <cybersphinx>
Project Member
Fri 09 Nov 2007 08:16:34 PM UTC, comment #3:

There are actually way more than those few...

Dennis Schridde <devurandom>
Project AdministratorIn charge of this item.
Fri 09 Nov 2007 07:34:37 PM UTC, comment #2:

frontend.c: In function ‘addTextButton’:
frontend.c:1196: warning: cast to pointer from integer of different size
frontend.c: In function ‘displayTextOption’:
frontend.c:1308: warning: cast from pointer to integer of different size

Dennis Schridde <devurandom>
Project AdministratorIn charge of this item.
Fri 09 Nov 2007 02:30:32 PM UTC, comment #1:

(for trunk)

Christian Ohm <cybersphinx>
Project Member
Fri 09 Nov 2007 01:51:38 PM UTC, original submission:

With the recent intptr_t patch, gcc (gcc (GCC) 4.2.3 20071014 (prerelease) (Debian 4.2.2-3)) now compiles the source with -Werror without the need to suppress any warnings.

Christian Ohm <cybersphinx>
Project Member

 

Attached Files

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -unavailable- added by muggenhor (Posted a comment)
  • -unavailable- added by devurandom (Posted a comment)
  • -unavailable- added by cybersphinx (Submitted the item)
  •  

    Do you think this task is very important?
    If so, you can click here to add your encouragement to it.
    This task has 0 encouragements so far.

    Only logged-in users can vote.

     

    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
    Fri 07 Mar 2008 11:08:55 AM UTCdevurandomStatusIn Progress=>None
    Fri 09 Nov 2007 10:17:51 PM UTCdevurandomCategoryNone=>Fix
      StatusWont Do=>In Progress
      Assigned toNone=>devurandom
      Open/ClosedClosed=>Open
    Fri 09 Nov 2007 09:42:49 PM UTCmuggenhorOpen/ClosedOpen=>Closed
    Fri 09 Nov 2007 09:42:48 PM UTCmuggenhorStatusNone=>Wont Do
    Fri 09 Nov 2007 01:51:38 PM UTCcybersphinxAttached File-=>Added dont_suppress_warnings.patch, #3164
    Show feedback again

    Back to the top


    Powered by Savane 3.1-cleanup