Style Cleanups #614

Closed
blakewatters opened this Issue Mar 22, 2012 · 1 comment

Comments

Projects
None yet
2 participants
@blakewatters
Member

blakewatters commented Mar 22, 2012

There are few cosmetic changes to the coding style in the library I'd like to get in place:

  • Ensure the stars for all pointers belong to the variable rather than the type.
  • Let the compiler synthesize ivars rather than declaring them.
  • Move all private ivars into a class extension and synthesize them.
  • Eliminate any getter/setter overrides with KVO.
  • Get in line with the Apple convention of placing opening braces on a new line for methods.

@ghost ghost assigned jawwad May 18, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

Add missing space after comma. Refs #614
Used uncrustify's "sp_after_comma = add" option to detect issues.

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

Remove extra space before comma. Refs #614
Used uncrustify's "sp_before_comma = remove" option to detect issues.

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 4, 2012

Placing opening braces on a new line for methods to match Apple conve…
…ntion. Refs #614

Used uncrustify's "nl_fdef_brace = add" option to detect issues.

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 5, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 25, 2012

jawwad added a commit to jawwad/RestKit that referenced this issue Jun 25, 2012

jawwad added a commit that referenced this issue Jul 11, 2012

@blakewatters

This comment has been minimized.

Show comment
Hide comment
@blakewatters

blakewatters Aug 16, 2012

Member

Merged to development

Member

blakewatters commented Aug 16, 2012

Merged to development

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment