Neuer FixInsight ist da!

[himitsu]

Zitat von Daniel:

Es handelt sich dabei um einen Parser, der den Projekt-Quellcode analysiert und ...

Ich bin aber immernoch der Meinung, daß Emba endlich mal seinen Parser "öffentlich" zugänglich machen sollte
und die sollten vorallem in ihren eigenen Projekten endlich mal überall den selben Parser benutzen,
denn Error-Insight und Compiler meinen ja oft was Anderes.

Also nicht unbedingt kompltt öffentlich, aber wenigstens so, wie man es z.B. von PHP kennt, wo es eine Funktion gibt, welche PHP-Code parsen kann.
http://php.net/manual/de/function.token-get-all.php

So gäbe es endlich mal eine Möglichkeit, daß Tools immer jeden Delphi-Code für die aktuelle Delphi-IDE verstehen und nicht so ein dreck passiert, wo z.B. das Documentation-Insight bei gewissen Strukturen (vorallem bei "aktuellen" Sachen bei den Conditional-Expressions, Generics oder Class-Helpern) einfach so verreckt.
Und als Bonus könnte es ausversehn passieren, daß danach das Error-Insight auch seine letzten Probleme los wird.

[Bernhard Geyer]

Zitat von himitsu:

Ich bin aber immernoch der Meinung, daß Emba endlich mal seinen Parser "öffentlich" zugänglich machen sollte
und die sollten vorallem in ihren eigenen Projekten endlich mal überall den selben Parser benutzen,
denn Error-Insight und Compiler meinen ja oft was Anderes.

Ich denke mit dem LLVM-Ansatz beim 64-Bit/ARM-Compiler ist man schon auf dem Weg. 32-Bit Compiler ist uralt - Da wird man nicht zu viel "öffentlich" mehr machen können und das beim Error-Insight ein ander Parser drin ist, ist dem (für Delphi) Irrweg .net geschultet. Ist also auch noch ein Altlast.

[himitsu]

Jupp, gerade für LLVM brauchen die doch sowieso einen "Parser", um das Zeug in Ruhe übersetzen zu können?

[Der schöne Günther]

Großer Gott, ich saß jetzt bestimmt 30-45 Minuten an einem Problem das schlichtweg darin begründet lag dass ein Schlaubold das inherited im Destruktor weggelassen hat. Nichts hat mich weitergebracht bis ich es eher durch Zufall gefunden hatte.

Ich glaube langsam auch dass wir das Tool brauchen

[Stevie]

Gibt ne neue Version:

Zitat:

What's new?
* Introduced rule C108 ("Nested WITH statement")
* Introduced rule W521 ("Return value of function might be undefined")
* Improved compiler directives handling in IDE expert
* Improved rule W519 ("Method is empty")
* Improved parser
* Minor fixes

[Der schöne Günther]

Zitat von Stevie:

* Introduced rule W521 ("Return value of function might be undefined")

Großer Gott, ja!



PS: Jetzt wollte ich es endlich einmal ausprobieren, scheitere schon an der Installation.

1. Setup ausführen
2. Bpl hinzufügen (wozu dann das Setup?)
3. Und jetzt?

PPS: Ooh, es versteckt sich im Menü "Projekt". Aber warum konnte es dann die magische F6-Taste mit "Fix" nicht finden?

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.

[Uwe Raabe]

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.

Hast du das schon gemeldet ([e-mail])? Andernfalls könnte es länger dauern, bis das behoben wird.

Oder hier: http://fixinsight.userecho.com/

[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.

...

[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.