search

ryanmaxwell / UncrustifyX

Uncrustify utility and documentation browser for Mac OS X
Other
586 stars 44 forks source link

Add newer options #18

Closed mgrebenets closed 8 years ago

mgrebenets commented 8 years ago

Add 46 new uncrustify configuration options from version 0.61

mgrebenets commented 8 years ago

bump

@ryanmaxwell are there concerns with this PR? It's been sitting there for quite a while now.

ryanmaxwell commented 8 years ago

Sorry been wanting to pull this in, but I'm away overseas with no computer until the new year, I'll aim to get to it in Jan!

mgrebenets commented 8 years ago

Cool! No rush :+1: I just wasn't sure if it was going anywhere :)

dzenbot commented 8 years ago

This would be great!

dzenbot commented 8 years ago

🙌 thanks!

ryanmaxwell commented 8 years ago

@dzenbot No prob. How did you find the diffs between 0.6 and 0.61? Just manually? I used to have a script that did it for me but i've misplaced it :/

dzenbot commented 8 years ago

Looking at different config files from people, to be honest. But UncristifyX didn't recognize them; good thing it just ignore them, instead of overriding/deleting them 👍

dzenbot commented 8 years ago

A script would be very useful! Or a feature in the app, highlithing the missing configurations from newer version of Uncrustify.

mgrebenets commented 8 years ago

Same here. I couldn't even find the 0.61 config I refereed to as "latest" while making the changes... can't even google it up now... It would be nice to have such an option in oclint itself, to spit out config file with all the possible rules in it.

ryanmaxwell commented 8 years ago

I guess watching https://github.com/bengardner/uncrustify/commits/master/src/options.h is also a good way to go about diffs!

ryanmaxwell commented 8 years ago

I've found 2 new options and added them in. I'll tag release a new version tonight.