Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

First try of the uncrustify config file #44

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

First try of the uncrustify config file #44

wants to merge 1 commit into from

Conversation

gregheo
Copy link
Contributor

@gregheo gregheo commented Jan 29, 2014

There are still hundreds of configuration options to wade through, but I thought I would start with what I had.

The obvious rules are covered:

  • tabs vs spaces and indent size
  • general spacing and newlines
  • brace placement
  • where the pointer *asterisk goes

Of course I included a test too. I hope it will grow with the configuration file so we can weed out any funny edge cases and get a sense of uncrustify's limits.

@StuartWebster
Copy link

I'm having an issue with sp_after_oc_msg_receiver = remove that turns
[[Foo alloc] init]
into
[[Foo alloc]init]

Changing the option to sp_after_oc_msg_receiver = force fixes it for me.

I also use sp_cond_colon = force and sp_cond_question = force for proper spacing on the ?: operator.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants