Neuer FixInsight ist da!

[jaenicke]

Zitat von Stevie:

* Introduced rule C108 ("Nested WITH statement")

Fehlt nur noch "WITH statement" an sich, denn wenn ich das nested richtig verstehe, meldet es ja nur with innerhalb eines with, oder?

Neuer Code mit with kann bei uns schlicht nicht mehr eingecheckt werden, aber als Qualitäts- und Stabilitätsprüfung des Codes wäre auch die Prüfung auf die Verwendung von with an sich als Regel sinnvoll.

[mkinzler]

Zitat von Daniel:

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.

Selbst Schuld. Zumindest laut Nick Hodges. FixInsight verwendet ein Derivat des Castalia-Parsers und dazu schreibt Nick in seinem aktuellen Blog-Eintrag:

Zitat:

If Castalia has trouble parsing your code, then the first place you should look to solve the problem is in your code.

Also räum gefälligst Deinen Code auf, bevor Du Bug-Einträge erzeugst.

...

§1: A macht niemals Fehler.
§2: Sollte A, widererwartens einen Fehler machen gilt automatisch §1.

[Stevie]

Zitat von Daniel:

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.

Selbst Schuld. Zumindest laut Nick Hodges. FixInsight verwendet ein Derivat des Castalia-Parsers und dazu schreibt Nick in seinem aktuellen Blog-Eintrag:

Zitat:

If Castalia has trouble parsing your code, then the first place you should look to solve the problem is in your code.

Was für ein Schwachsinn, den Nick da schreibt.
Übrigens benutzt FixInsight DelphiAST und das ist weiter entwickelt als Castalia.
Immerhin spuckt FixInsight aber die Zeile des Fehler aus und gibt nicht sang und klanglos den Geist auf (oder lässt die IDE krachen...)

[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