ECToo / cleancodequake2

Automatically exported from code.google.com/p/cleancodequake2
0 stars 0 forks source link

Fix warnings #13

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Because masochism is fun.

Original issue reported on code.google.com by Jayschwa on 3 Jun 2010 at 4:28

GoogleCodeExporter commented 9 years ago
Wanna show us the warnings perhaps? I compile CleanCode at W4 with Warnings as 
Errors
in Visual Studio.

Original comment by Jonno.5000 on 3 Jun 2010 at 11:52

GoogleCodeExporter commented 9 years ago
Here is a log when I built with GCC options -Wall and -Wextra:
http://cleancodequake2.googlecode.com/svn/trunk/GCC_warnings.txt

I may start hammering some of these out over the weekend if I'm bored. My guess 
is 
that most of them aren't a big deal.

Original comment by Jayschwa on 4 Jun 2010 at 3:09

GoogleCodeExporter commented 9 years ago
Okay, after examination, here are the warnings that can safely be ignored:

warning: unused parameter "xxx"

warning: "yyy" will be initialized after
warning:   "xxx"
warning:   when initialized here

warning: ignoring return value of "function xxx ()", declared with attribute
warn_unused_result

Can you remove just these warnings and compile?

Original comment by Jonno.5000 on 4 Jun 2010 at 4:24