search

pombreda / alembic

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

Clean source files of all gcc warnings (as pedantic as -wall) #165

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
When building I se several warnigns for unused locals and other non-critical 
things. This issue is to track the need to spent an hour and clean all of these 
up...
I'll paste my warning-level comp flags later on.

Original issue reported on code.google.com by cookingw...@gmail.com on 26 May 2011 at 6:07

GoogleCodeExporter commented 9 years ago

Original comment by ble...@gmail.com on 26 May 2011 at 6:08

GoogleCodeExporter commented 9 years ago 
Can we get these warning levels added to our CMake settings so we'll be able to 
see them?

Original comment by miller.lucas on 26 May 2011 at 6:10

GoogleCodeExporter commented 9 years ago

Original comment by ble...@gmail.com on 7 Jul 2011 at 10:47

GoogleCodeExporter commented 9 years ago

Original comment by ble...@gmail.com on 7 Jul 2011 at 10:52

GoogleCodeExporter commented 9 years ago 
Assigning owners for Monday - going with person who opened. Can reassign next 
week.

Original comment by scottmmo...@gmail.com on 8 Jul 2011 at 10:29

GoogleCodeExporter commented 9 years ago

Original comment by scottmmo...@gmail.com on 11 Jul 2011 at 11:08

GoogleCodeExporter commented 9 years ago 
We could break this into phases
phase 0: with current build settings do full build and fix warnings that come 
out currently 
phase 1: change build flags to warn on all and fix warnings

phase 0 would give the appearance of a clean build when people first download 
Alembic.

Original comment by ble...@gmail.com on 14 Jul 2011 at 4:24

GoogleCodeExporter commented 9 years ago 
Closing as a duplicate of Issue 145

Feel free to re-open or comment if you have concerns.

Thanks, Scott

Original comment by scottmmo...@gmail.com on 20 Jul 2011 at 10:03