IntrepidPursuits / objective-c-style-guide

MIT License
4 stars 0 forks source link

Feedback CB: Requesting dropping of "methods" from pragma marks #10

Closed cprime closed 9 years ago

cprime commented 9 years ago

“pragma mark -“: This is a pet peeve of mine, but "#pragma mark - UITableViewDataSource methods” is unnecessarily verbose. Just say "#pragma mark - UITableViewDataSource”. You wouldn’t call a method “doSomethingCoolMethod” would you? :)

brightredchilli commented 9 years ago

I think the bigger issue here is not defining the pragma mark at all. But a bit of verbosity does not hurt esp if it's in 'comments'. @colinbrash how big of an issue is this for you?

colinbrash commented 9 years ago

Not a big issue.  Obviously the example gets the point of the style convention across and my pet peeve is tangential.  If you guys change the example I would support that, though. :)

On Thu, Dec 11, 2014 at 9:01 AM, brightredchilli notifications@github.com wrote:

I think the bigger issue here is not defining the pragma mark at all. But a bit of verbosity does not hurt esp if it's in 'comments'. @colinbrash how big of an issue is this for you?

Reply to this email directly or view it on GitHub: https://github.com/IntrepidPursuits/objective-c-style-guide/issues/10#issuecomment-66622344

cprime commented 9 years ago

I can remove it from the pragma mark example, but won't make it as requirement to have it or not.

colinbrash commented 9 years ago

Yeah sorry that was unclear.  I’m not trying to force people not to use the word “methods”!

On Thu, Dec 11, 2014 at 10:16 AM, cprime notifications@github.com wrote:

I can remove it from the pragma mark example, but won't make it as requirement to have it or not.

Reply to this email directly or view it on GitHub: https://github.com/IntrepidPursuits/objective-c-style-guide/issues/10#issuecomment-66632993