search

fastsupply / growl

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

Switch compiler to Clang #194

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What feature or enhancement do you propose?
Use Clang as our compiler, especially for Release builds.

What problem does it solve/What benefit does it provide?
More and better warnings and errors, so the compiler finds bugs for us.

Original issue reported on code.google.com by p...@growl.info on 28 Apr 2011 at 5:45

GoogleCodeExporter commented 9 years ago 
Switching to LLVM/Clang causes an impressive number of implicit cast warnings 
(primarily in NSLog calls), which are treated as errors due to Growl being 
compiled with -Werror enabled.  Disabling -Werror, it builds fine and passes 
all tests without issue when compiled with Clang; it just needs all the casting 
warnings fixed to be able to compile with -Werror on.  

The necessary fix should be just a few hours of explicitly dealing with the 
casting warnings.  I suspect this could easily be put in the 1.3 milestone 
rather than 2.0.

Original comment by cerulean...@gmail.com on 28 Apr 2011 at 6:32

GoogleCodeExporter commented 9 years ago

Original comment by p...@growl.info on 28 Apr 2011 at 7:12

GoogleCodeExporter commented 9 years ago

Original comment by cerulean...@gmail.com on 30 Apr 2011 at 4:20

GoogleCodeExporter commented 9 years ago 
Done in default, as of changeset cadc83113d74: 
http://growl.info/hg/growl-development/rev/cadc83113d74

Original comment by cerulean...@gmail.com on 5 May 2011 at 6:09

GoogleCodeExporter commented 9 years ago

Original comment by ch...@growl.info on 4 Nov 2011 at 2:55