AGB  ·  Datenschutz  ·  Impressum  







Anmelden
Nützliche Links
Registrieren

Neuer FixInsight ist da!

Ein Thema von Insider2004 · begonnen am 10. Feb 2015 · letzter Beitrag vom 24. Apr 2016
Antwort Antwort
Seite 4 von 7   « Erste     234 56     Letzte » 
Benutzerbild von jaenicke
jaenicke

Registriert seit: 10. Jun 2003
Ort: Berlin
7.207 Beiträge
 
Delphi 10.2 Tokyo Enterprise
 
#31

AW: Neuer FixInsight ist da!

  Alt 23. Mär 2015, 11:00
* 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.
Sebastian Jänicke
Alle eigenen Projekte sind eingestellt, ebenso meine Homepage, Downloadlinks usw. im Forum bleiben aktiv!
  Mit Zitat antworten Zitat
mkinzler
(Moderator)

Registriert seit: 9. Dez 2005
Ort: Heilbronn
39.338 Beiträge
 
Delphi 10.2 Tokyo Enterprise
 
#32

AW: Neuer FixInsight ist da!

  Alt 23. Mär 2015, 11:17
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.
Markus Kinzler
  Mit Zitat antworten Zitat
Benutzerbild von Stevie
Stevie

Registriert seit: 12. Aug 2003
Ort: Soest
3.531 Beiträge
 
Delphi 10.1 Berlin Enterprise
 
#33

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 07:12
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...)
Stefan
“Simplicity, carried to the extreme, becomes elegance.” Jon Franklin

Delphi Sorcery - DSharp - Spring4D - TestInsight
  Mit Zitat antworten Zitat
Roman Yankovsky

Registriert seit: 18. Feb 2015
 
#34

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 08:04
Sorry guys, I don't speak German. I will reply in English, I hope this doesn't violate the website policy.

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 support@sourceoddity.com

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.

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
  Mit Zitat antworten Zitat
Benutzerbild von Sherlock
Sherlock

Registriert seit: 10. Jan 2006
Ort: Offenbach
3.287 Beiträge
 
Delphi 10.3 Rio
 
#35

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 08:30
Welcome Roman, it's OK to post in english. Stop by here more often

Sherlock
Geändert von Sherlock (Morgen um 16:78 Uhr) Grund: Weil ich es kann
  Mit Zitat antworten Zitat
Daniel
(Administrator)

Registriert seit: 30. Mai 2002
Ort: Hamburg
14.617 Beiträge
 
Delphi 10.3 Rio
 
#36

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 08:44
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.
Daniel R. Wolf
Admin Delphi-PRAXiS
mit Grüßen aus Hamburg
  Mit Zitat antworten Zitat
freimatz
Online

Registriert seit: 20. Mai 2010
552 Beiträge
 
Delphi 10.2 Tokyo Professional
 
#37

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 10:36
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.
  Mit Zitat antworten Zitat
Roman Yankovsky

Registriert seit: 18. Feb 2015
 
#38

AW: Neuer FixInsight ist da!

  Alt 24. Mär 2015, 21:47
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.
Miniaturansicht angehängter Grafiken
2015-02-26-17-29-33-about-fixinsight.png  
  Mit Zitat antworten Zitat
Der schöne Günther

Registriert seit: 6. Mär 2013
4.916 Beiträge
 
Delphi 10 Seattle Enterprise
 
#39

AW: Neuer FixInsight ist da!

  Alt 25. Mär 2015, 09:21
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.

Geändert von Der schöne Günther (25. Mär 2015 um 09:45 Uhr)
  Mit Zitat antworten Zitat
Roman Yankovsky

Registriert seit: 18. Feb 2015
 
#40

AW: Neuer FixInsight ist da!

  Alt 26. Mär 2015, 09:52
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

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.

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
  Mit Zitat antworten Zitat
Themen-Optionen Thema durchsuchen
Thema durchsuchen:

Erweiterte Suche
Ansicht

Forumregeln

Es ist dir nicht erlaubt, neue Themen zu verfassen.
Es ist dir nicht erlaubt, auf Beiträge zu antworten.
Es ist dir nicht erlaubt, Anhänge hochzuladen.
Es ist dir nicht erlaubt, deine Beiträge zu bearbeiten.

BB-Code ist an.
Smileys sind an.
[IMG] Code ist an.
HTML-Code ist aus.
Trackbacks are an
Pingbacks are an
Refbacks are aus

Gehe zu:

Impressum · AGB · Datenschutz · Nach oben
Alle Zeitangaben in WEZ +1. Es ist jetzt 17:11 Uhr.
Powered by vBulletin® Copyright ©2000 - 2019, Jelsoft Enterprises Ltd.
LinkBacks Enabled by vBSEO © 2011, Crawlability, Inc.
Delphi-PRAXiS (c) 2002 - 2019 by Daniel R. Wolf