A review / introduction to FixInsight, Roman Yankovsky's new static code analysis plugin for Delphi.

-
A review / introduction to FixInsight, Roman Yankovsky's new static code analysis plugin for Delphi.

http://parnassus.co/review-fixinsight-sourceoddity/
http://parnassus.co/review-fixinsight-sourceoddity

Comments

  1. Lübbe OnkenFebruary 17, 2015 at 8:38 AM

    I've whipped up a tool that compares the FixInsight output as well as the delphi compiler output and integrated it into our build system. Now the build fails if a warning count has increased anywhere.
    You don't get a bonus for removing warnings ;)
    The output of the last good build is saved as the input for the current build.

    ReplyDelete
  2. Roman YankovskyFebruary 17, 2015 at 11:51 AM

    > As it happens, the form is in fact correctly freed.

    To be honest, I don't think it is a false positive. TForm has Release method to correctly free a form in a form’s method.

    ReplyDelete
  3. David MillingtonFebruary 18, 2015 at 5:39 AM

    Roman Yankovsky I think you're right. I should be using Release. Chalk up another +1 for FixInsight :)

    ReplyDelete

Post a Comment