Re: static clang source code analysis (Code Review Requested)

2012-12-10 Thread John Smith
Hi, So now that people know how to generate these reports, lets hope they are going to be put to good use ! ;) - John Smith. On Mon, Dec 10, 2012 at 9:19 AM, Miklos Vajna wrote: > Hi John, > > On Fri, Dec 07, 2012 at 04:41:48PM +0100, John Smith > wrote: >> The clang analyzer how-to is here

Re: static clang source code analysis (Code Review Requested)

2012-12-10 Thread Miklos Vajna
Hi John, On Fri, Dec 07, 2012 at 04:41:48PM +0100, John Smith wrote: > The clang analyzer how-to is here : > http://wiki.documentfoundation.org/Development/Clang Thanks a lot for both! Miklos signature.asc Description: Digital signature ___ LibreOf

Re: static clang source code analysis (Code Review Requested)

2012-12-07 Thread John Smith
Hi, The clang analyzer how-to is here : http://wiki.documentfoundation.org/Development/Clang Regards, John Smith. On Fri, Dec 7, 2012 at 11:08 AM, John Smith wrote: > Hi, > > > For the lcov reports, I put the following in the wiki: > http://wiki.documentfoundation.org/Development/Lcov > Feel

Re: static clang source code analysis (Code Review Requested)

2012-12-07 Thread John Smith
Hi, For the lcov reports, I put the following in the wiki: http://wiki.documentfoundation.org/Development/Lcov Feel free to try it out and edit it as you see fit. Regards, John Smith. On Wed, Dec 5, 2012 at 5:33 PM, Miklos Vajna wrote: > On Wed, Dec 05, 2012 at 04:15:45PM +0100, John Smith

Re: static clang source code analysis (Code Review Requested)

2012-12-05 Thread Miklos Vajna
On Wed, Dec 05, 2012 at 04:15:45PM +0100, John Smith wrote: > Sure, I can put short tutorials for generating both reports on the > wiki. How do I get a wiki account, and where on the wiki should I put > them ? Creating a wiki account: https://wiki.documentfoundation.org/index.php?title=Special:

Re: static clang source code analysis (Code Review Requested)

2012-12-05 Thread John Smith
> Miklos Vajna vmiklos at suse.cz > > Hi John, > > On Mon, Nov 05, 2012 at 02:45:22AM +0100, John Smith > wrote: > > Here's a clang static source code analysis of the latest git sources : > > > > http://dev-builds.libreoffice.org/clang_reports/master~2012-11-04_17.27.13/ > > I know this is a clan

Re: static clang source code analysis (Code Review Requested)

2012-11-05 Thread Norbert Thiebaud
On Mon, Nov 5, 2012 at 11:11 AM, Miklos Vajna wrote: > Hi Norbert, > > On Mon, Nov 05, 2012 at 08:22:53AM -0600, Norbert Thiebaud > wrote: >> I've been playing with gcov/lcov >> >> to get the coverage for a given module: >> >> 1/ build the product >> 2/ make module.clean >> 3/ gb_GCOV=YES make m

Re: static clang source code analysis (Code Review Requested)

2012-11-05 Thread Miklos Vajna
Hi Norbert, On Mon, Nov 05, 2012 at 08:22:53AM -0600, Norbert Thiebaud wrote: > I've been playing with gcov/lcov > > to get the coverage for a given module: > > 1/ build the product > 2/ make module.clean > 3/ gb_GCOV=YES make module Fine till this. I assume at this stage the unit tests of i

Re: static clang source code analysis (Code Review Requested)

2012-11-05 Thread bfo
John Smith wrote > Hi, > Here's a clang static source code analysis of the latest git sources : > http://dev-builds.libreoffice.org/clang_reports/master~2012-11-04_17.27.13/ Hi! Is it comparable with the previous one? Hope you will update lcov one too. Any chances to have those two updated on a re

Re: static clang source code analysis (Code Review Requested)

2012-11-05 Thread Norbert Thiebaud
On Mon, Nov 5, 2012 at 6:36 AM, Miklos Vajna wrote: > I know this is a clang report and not the lcov one, but -- can you > please put up the commands you use to generate these reports to the > wiki, or (maybe even better) add two scripts like > bin/generate-lcov-report and bin/generate-clang-repor

Re: static clang source code analysis (Code Review Requested)

2012-11-05 Thread Miklos Vajna
Hi John, On Mon, Nov 05, 2012 at 02:45:22AM +0100, John Smith wrote: > Here's a clang static source code analysis of the latest git sources : > > http://dev-builds.libreoffice.org/clang_reports/master~2012-11-04_17.27.13/ I know this is a clang report and not the lcov one, but -- can you pleas