On Wed, 3 Apr 2013, Jed Brown wrote:
> We used 'uncrustify' the tool.
>
> Our scripts are in $PETSC_DIR/src/contrib/style/
And uncrustify config in bin/uncrustify.cfg?
Thanks to you both!
---
Roy
--
Minimize network d
Uncrustify has much more fine scale control than astyle. It is not perfect,
but can match "most" of the many documented PETSc formatting rules.
My dream is to required processing each source file through crustify
"automatically" before any source code is committed to the repository (so it
On Wed, Apr 3, 2013 at 7:27 PM, Roy Stogner wrote:
> On Wed, 3 Apr 2013, Jed Brown wrote:
>
> (PETSc just went through this "uncrustify" process.
>>
>
> Do you mean metaphorically or "literally, we used the uncrustify
> command"? uncrustify was my second or third choice for a beautifier
> after
On Wed, 3 Apr 2013, Jed Brown wrote:
> (PETSc just went through this "uncrustify" process.
Do you mean metaphorically or "literally, we used the uncrustify
command"? uncrustify was my second or third choice for a beautifier
after astyle and now I'm wondering if I ranked it too low.
---
Roy
---
Roy Stogner writes:
> Yelling "We just wanted to strip off trailing white space, what the
> hell!?!" at me would also be reasonable.
>
> There's no need to do any of this any time soon (and certainly not
> while Ben's got a major branch outstanding for the communicators
> stuff) but it couldn't h
I'll play later tomorrow and see how things look...
Sometimes I do like to use seemingly odd manual alignment for clarity - would
these break that?
-Ben
On Apr 3, 2013, at 5:02 PM, "Roy Stogner" wrote:
>
> A. "astyle -rn --style=gnu --lineend=linux" gets the indentation
> right, uses spac
A. "astyle -rn --style=gnu --lineend=linux" gets the indentation
right, uses spaces instead of tabs in indentation, converts any
dos/mac '\n' to Unix, and strips trailing white space, but doesn't
touch much else.
B. "astyle -rn --style=gnu --lineend=linux --break-blocks --pad-oper
--pad-header -
