Neuer FixInsight ist da!

[Uwe Raabe]

Zitat von Der schöne Günther:

die UserVoice-Seite kannte ich hingegen überhaupt nicht. Danke!

Kannte ich vorher auch nicht, wurde aber in der email für das Update erwähnt. Was mich ein bisschen stört: die Links in der email laufen alle über srclick.ru

[Roman Yankovsky]

Sorry guys, I don't speak German. I will reply in English, I hope this doesn't violate the website policy.

Zitat von Der schöne Günther:

PPPS: Huh, da muss sich aber noch was tun. Ich habe schon zwei verschiedene Wege gefunden ihn mit einem "Fatal Parse Error" aus der Bahn zu werfen.

Usually I fix parser issues pretty fast. Please let me know the details, maybe a small piece of code. The email is [e-mail]

It's a little bit not correct to say that FixInsight uses Castalia parser. It uses DelphiAST which is built on the top of CastaliaDelphiParser and was much improved. So it's not something like "parser is good, you have to fix your own code". Please let me know about the issues and I will fix them.

Zitat von Uwe Raabe:

Zitat von Der schöne Günther:

die UserVoice-Seite kannte ich hingegen überhaupt nicht. Danke!

Kannte ich vorher auch nicht, wurde aber in der email für das Update erwähnt. Was mich ein bisschen stört: die Links in der email laufen alle über srclick.ru

That's nothing more than a click counter for stats

[Sherlock]

Welcome Roman, it's OK to post in english. Stop by here more often

Sherlock

[Daniel]

Zitat von Roman Yankovsky:

Sorry guys, I don't speak German. I will reply in English, I hope this doesn't violate the website policy.

Au contraire, mon ami. ... oh wait, that was french. ehm - well, it's great having you here.

[freimatz]

Hallo,
habe heute mal die Demo installiert. Bei Aufruf mit einem meiner Projekte kommt ein EStringListError in Classes.TFileIterator.Next. (nach madExcept) Eine Ausgabe kommt dann bei continue trotzdem. Umständlich ist, dass mich die meisten units des Projektes gar nicht insteressieren.
Bei Aufruf von Kommandozeile geht gar nichts. D.h. FixInsightCL nacht eine kleine Pause und beendet dann. Zumindest bei --output und --xml hätte ich eine leere xml erwartet.

[Roman Yankovsky]

Zitat von freimatz:

Hallo,
habe heute mal die Demo installiert. Bei Aufruf mit einem meiner Projekte kommt ein EStringListError in Classes.TFileIterator.Next. (nach madExcept) Eine Ausgabe kommt dann bei continue trotzdem. Umständlich ist, dass mich die meisten units des Projektes gar nicht insteressieren.
Bei Aufruf von Kommandozeile geht gar nichts. D.h. FixInsightCL nacht eine kleine Pause und beendet dann. Zumindest bei --output und --xml hätte ich eine leere xml erwartet.

The EStringListError is fixed today.
Please download the latest build http://sourceoddity.com/download/Fix...upd1_setup.exe

Unfortunately, I cannot reproduce the issue with FixInsightCL. Could you do me a favor and help to debug it? If so, in your IDE click on Help -> About FixInsight and check "Debug Mode" option (see attached screenshot). Then run FixInsightCL and reproduce the issue. After that send me *.filog file (this log file will be created in your project folder). This will help me a lot to debug the issue.

[Der schöne Günther]

How can one surpress FixInsight warnings locally? I might, for example, leave an overridden virtual method empty on purpose. FixInsight is correctly raising a W519 here but the code is entirely valid and working as intended.


Oh, and while we're at it: There must be an option to exclude files. If you got some third party sources, gnugettext.pas for example, you're also going to get warnings on files you definitely don't want to modify.

I'm sure this is already on uservoice but I haven't bothered to check


One last thing: C104 ("Class name should start with 'T'") is true, but I suppose it doesn't apply to attributes.
Embarcaderos examples omit the "T".
Also, Marco Cantus new book states

Zitat:

So if you name your class SimpleAttribute you'll be able to use in the code an attribute called Simple or SimpleAttribute . For this is the reason the classic initial T for Object Pascal classes is generally not used in case of attributes.

[Roman Yankovsky]

Zitat von Der schöne Günther:

How can one surpress FixInsight warnings locally?

You may use _FIXINSIGHT_ compiler conditional. It works just like a real compiler directive.

For example,

Delphi-Quellcode:

{$IFNDEF _FIXINSIGHT_}
  ThisCodeWillBeIgnored(1,2,3);
{$ENDIF}

I have to add this to the documentation. I have never expected that so many people will need this functionality

Zitat von Der schöne Günther:

There must be an option to exclude files. If you got some third party sources, gnugettext.pas for example, you're also going to get warnings on files you definitely don't want to modify.

Yes, this would be great. It's on my todo list.

Zitat von Der schöne Günther:

One last thing: C104 ("Class name should start with 'T'") is true, but I suppose it doesn't apply to attributes.

Agree. I will fix this

[freimatz]

Zitat von Roman Yankovsky:

Zitat von freimatz:

Hallo,
habe heute mal die Demo installiert. Bei Aufruf mit einem meiner Projekte kommt ein EStringListError in Classes.TFileIterator.Next. (nach madExcept) Eine Ausgabe kommt dann bei continue trotzdem. Umständlich ist, dass mich die meisten units des Projektes gar nicht insteressieren.
Bei Aufruf von Kommandozeile geht gar nichts. D.h. FixInsightCL nacht eine kleine Pause und beendet dann. Zumindest bei --output und --xml hätte ich eine leere xml erwartet.

The EStringListError is fixed today.
Please download the latest build http://sourceoddity.com/download/Fix...upd1_setup.exe

Unfortunately, I cannot reproduce the issue ...

Nach Installation der neuen Version ist alles gut

[schöni]

Ab welcher Delphi Version kann man das einsetzen?

Ab welcher Windows Version funktioniert das Tool?

@freimatz: Ist das gefixte Tool im Link die Demo- oder die Vollversion?