Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Игорь Пашев
Code formatting is completely out of scope of building system. -- Powered by www.kitware.com Please keep messages on-topic and check the CMake FAQ at: http://www.cmake.org/Wiki/CMake_FAQ Kitware offers various services to support the CMake community. For more information on each offering,

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Alan W. Irwin
On 2014-01-31 13:55+0400 Игорь Пашев wrote: Code formatting is completely out of scope of building system. I disagree with your attempt to shut off discussion. Some projects don't use code styling, others run a script aperiodically to clean up the code (PLplot uses uncrustify that way). Both

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Clifford Yapp
On Fri, Jan 31, 2014 at 2:01 AM, Rob McDonald rob.a.mcdon...@gmail.com wrote: All, I'm interested in adding a code beautifier to my project. I'm looking at AStyle, but am open to others. Implementing a FindAStyle.cmake is pretty trivial. Likewise, implementing custom targets is pretty

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Paul Smith
On Fri, 2014-01-31 at 02:26 -0800, Alan W. Irwin wrote: And to answer the OP's question, I can highly recommend uncrustify for code styling I agree with Alan. We did a huge reformatting effort last year to change a very large C++ codebase from a style based loosely on Whitesmith to a more

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Eric Noulard
2014-01-31 Paul Smith p...@mad-scientist.net: I had to do a bit of scripting around it since uncrustify didn't handle all the whitespace conversion we wanted, but it worked great! We didn't try to integrate it with the build system. We just checked in the configuration file and a script

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Marcel Loose
On 31/01/14 14:11, Paul Smith wrote: On Fri, 2014-01-31 at 02:26 -0800, Alan W. Irwin wrote: And to answer the OP's question, I can highly recommend uncrustify for code styling I agree with Alan. We did a huge reformatting effort last year to change a very large C++ codebase from a style

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Leif Walsh
What would be a good way to run a tool like this just before compiling? My naive solution is going to be to add a custom command to run it and touch a stamp, a target to run that custom command, and then make every single target in my project depend on that custom target. Is there a less

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Игорь Пашев
2014-01-31 Leif Walsh leif.wa...@gmail.com: What would be a good way to run a tool like this just before compiling? I'd prefer running such a tool *after* compiling. If you have a syntax error, your sources can be ruinned. -- Powered by www.kitware.com Please keep messages on-topic and check

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Miller Henry
approach. -Original Message- From: CMake [mailto:cmake-boun...@cmake.org] On Behalf Of Paul Smith Sent: Friday, January 31, 2014 7:12 AM To: Alan W. Irwin Cc: CMake ML Subject: Re: [CMake] AStyle or similar code beautifier On Fri, 2014-01-31 at 02:26 -0800, Alan W. Irwin wrote: And to answer

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Matthew Woehlke
On 2014-01-31 02:01, Rob McDonald wrote: I'm interested in adding a code beautifier to my project. I'm looking at AStyle, but am open to others. Implementing a FindAStyle.cmake is pretty trivial. Likewise, implementing custom targets is pretty straightforward in the simple case. However, I

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Gregoire Aujay
: vendredi 31 janvier 2014 16:54 To: Leif Walsh Cc: cmake@cmake.org Subject: Re: [CMake] AStyle or similar code beautifier 2014-01-31 Leif Walsh leif.wa...@gmail.com: What would be a good way to run a tool like this just before compiling? I'd prefer running such a tool *after* compiling. If you

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Rob McDonald
Thanks to everyone for all the helpful responses. It looks like most of the ideas were pretty similar to what I was thinking. Glad to know I wasn't missing anything huge. I had found UniversalIndentGUI and plan on using it. I had also found uncrustify and will take a closer look at it. On

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Alan W. Irwin
On 2014-01-31 14:24+0100 Eric Noulard wrote: Would you mind sharing the script you used with us? Although that question was directed to someone else, you might be interested in scripts/style_source.sh within the PLplot source code. There are options to show diffs (or not) and apply the

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Paul Smith
On Fri, 2014-01-31 at 14:24 +0100, Eric Noulard wrote: We didn't try to integrate it with the build system. We just checked in the configuration file and a script people could use if they wanted to re-beautify their code. Would you mind sharing the script you used with us? Sure, it's

Re: [CMake] AStyle or similar code beautifier

2014-01-31 Thread Eric Noulard
2014-01-31 Paul Smith p...@mad-scientist.net: On Fri, 2014-01-31 at 14:24 +0100, Eric Noulard wrote: We didn't try to integrate it with the build system. We just checked in the configuration file and a script people could use if they wanted to re-beautify their code. Would you mind

[CMake] AStyle or similar code beautifier

2014-01-30 Thread Rob McDonald
All, I'm interested in adding a code beautifier to my project. I'm looking at AStyle, but am open to others. Implementing a FindAStyle.cmake is pretty trivial. Likewise, implementing custom targets is pretty straightforward in the simple case. However, I thought there might be some