#195 "Litle optimization" in ASBeautifier.cpp

closed-fixed
Jim Pattee
Patches (3)
7
2015-01-03
2012-12-15
Ruzzz
No

"Litle optimization" in ASBeautifier.cpp based on tips from static code analyzer

Analyzer log:
V571 Recurring check. The 'line.length() > 0' condition was already verified in line 825. asbeautifier.cpp 831
V571 Recurring check. The '!inStatementIndentStack->empty()' condition was already verified in line 2751. asbeautifier.cpp 2753
V571 Recurring check. The 'isInStatement' condition was already verified in line 2751. asbeautifier.cpp 2753
V595 The 'activeBeautifierStack' pointer was utilized before it was verified against nullptr. Check lines: 851, 866. asbeautifier.cpp 851

Links:
V571 More http://www.viva64.com/en/d/0205/print/
V571 More http://www.viva64.com/en/d/0169/print/

Discussion

  • Ruzzz
    Ruzzz
    2012-12-15

    patch

     
    Attachments
  • Jim Pattee
    Jim Pattee
    2013-10-12

    • status: open --> open-fixed
    • assigned_to: Jim Pattee
    • Priority: 5 --> 7
     
  • Jim Pattee
    Jim Pattee
    2013-10-12

    Applied in SVN.

     
  • Jim Pattee
    Jim Pattee
    2013-11-05

    • status: open-fixed --> closed-fixed