kodecocodes / objective-c-style-guide

A style guide that outlines the coding conventions for Kodeco
http://www.raywenderlich.com/62570/objective-c-style-guide
3.1k stars 628 forks source link

Warning flags #20

Closed hollance closed 10 years ago

hollance commented 10 years ago

What are good warning flags to enable by default?

I prefer -Wall -Wextra -Wno-unused-parameter. It might be good to have these as suggested settings.

rwenderlich commented 10 years ago

Good idea.

But how should we handle this in tutorials? Should we direct the reader to add this flag during project setup, or should we just do it while developing the project behind the scenes?

gregheo commented 10 years ago

We can mention it as the recommended setting as a tip to readers, and ask authors to add it to sample/final projects attached to tutorials. I wouldn't want to mention it every time if a tutorial is a "start a project from scratch" type.

ghost commented 10 years ago

I don't believe warning flags belong in a style guide, as they don't pertain to style and have no direct impact on formatting. IMO these belong more in a 'Best Practices' type guide.

ndubbs commented 10 years ago

I agree with @micpringle regarding 'Best Practices'.

+1 for not including this in the style guide.

rwenderlich commented 10 years ago

Good point, yeah it does make sense to put this guideline in a best practices guide instead of a style guide. Maybe we can just put it on the tutorial team guide somewhere.