[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Cppcheck reports
From: |
Marek Klein |
Subject: |
Re: Cppcheck reports |
Date: |
Fri, 18 May 2012 21:42:38 +0200 |
Hello
2012/5/17 Julien Nabet <address@hidden>
> >
> > I'm not top posting.
> Hello,
>
> I just git clone Lilypond project and launched cppcheck (git updated
> today).
> I thought it could interest you, here are some examples :
> [lily/tuplet-bracket.cc:594] -> [lily/tuplet-bracket.cc:594]: (style) Same
> expression on both sides of '-'
> 592 if (!follow_beam)
> 593 {
> 594 points.push_back (Offset (x0 - x0, staff[dir]));
> 595 points.push_back (Offset (x1 - x0, staff[dir]));
> 596 }
>
> [lily/tie-engraver.cc:240]: (performance) Prefer prefix ++/-- operators for
> non-primitive types
> 240 for (; it < heads_to_tie_.end (); it++)
> 241 report_unterminated_tie (*it);
> (+ it's safer to use it != heads_to_tie_.end ())
>
> [lily/paper-book.cc:346]: (performance) Possible inefficient checking for
> 'cols'
> emptiness
> 346 if (cols.size ())
> 347 {
> 348 Paper_column *col = dynamic_cast<Paper_column *>
> (cols.back ());
> 349 col->set_property (symbol, permission);
> 350 col->find_prebroken_piece (LEFT)->set_property (symbol,
> permission);
> 351 }
>
> If you're interested, I can send you the full report (since there's no
> possibility of attachment), just tell me where I can send it.
>
> Julien.
>
This need some discussion before tracking an issue, I think - therefore
cc-ing devel...
Marek Klein,
bug squad member
- Cppcheck reports, Julien Nabet, 2012/05/17
- Re: Cppcheck reports,
Marek Klein <=
- Re: Cppcheck reports, Carl Sorensen, 2012/05/18
- Re: Cppcheck reports, Julien Nabet, 2012/05/20
- Re: Cppcheck reports, Carl Sorensen, 2012/05/18
- Re: Cppcheck reports patch for 2546, Julien Nabet, 2012/05/20
- Re: Cppcheck reports patch for 2546, Graham Percival, 2012/05/20
- Re: Cppcheck reports patch for 2546, Carl Sorensen, 2012/05/21
- Re: Cppcheck reports, Mark Mathias, 2012/05/26
- Re: Cppcheck reports, Colin Hall, 2012/05/27