Eure besten Quellcode Kommentare...

[Codehunter]

Ist schon faszinierend... TiGü hat da eine richtige Tretmine ausgelegt

|__| <-- Achtung, Fettnapf! Bitte treten Sie nur nach sorgfältiger Planung, ausreichend Vorwarn- und Bedenkzeit sowie angemessen hochwertigem Schuhwerk hier hinein

[freimatz]

Hätte man den MMX benutzt wäre das wohl nicht passiert.

[Codehunter]

Zitat von freimatz:

Hätte man den MMX benutzt wäre das wohl nicht passiert.

So sehr ich den auch schätze, er ist nur so gut wie der Anwender ihn nutzt.

[p80286]

Es ist Sonntag und ich schlendere durch die DP.
Das fehlende "h" ist mir durchgegangen, und das "Get.." ist ungewöhnlich, wahrscheinlich wird der Wert aus irgendeinem obskuren Topf geholt?

Ist auf jeden Fall ein Hingucker.

Gruß
K-H

[TiGü]

JEDI JVCL: JvMemoryDataset.pas - Zeile 426 (frisch von Github):

  STATUSNAME = 'C67F70Z90'; (* Magic *)

WTF?

[Stevie]

Zitat von TiGü:

JEDI JVCL: JvMemoryDataset.pas - Zeile 426 (frisch von Github):

  STATUSNAME = 'C67F70Z90'; (* Magic *)

WTF?

Ok, ich versuch das mal aufzulösen: über git blame findet man den Original Commit.
Dort wird auf Mantis Issue #1447 verwiesen, wo die Codeänderungen ursprünglich gepostet wurden.
Hier wird ein internen Statusfeld benutzt, dem dieser String als Feldname verpasst wird. Woher kommt dieser Wert?

Die Initialen des Authors sind CFZ - die ASCII Codes für C, F und Z sind 67, 70 und 90.

:U+1F92F: (exploding head emoji)

[TiGü]

...🤯...

[Codehunter]

Delphi-Quellcode:

initialization

// UnregisterExpectedMemoryLeak  ;

end.

Jetzt kann man sich aussuchen, was man schlimmer findet:

• Dass es überhaupt ein erwartetes Speicherleck gibt
• Dass es Funktionen zum Umgang mit Speicherlecks gibt
• Dass diese Funktionen auch noch in der RTL selbst vorhanden sind
• Dass der Aufruf hier auskommentiert ist
• Dass der auskommentierte Aufruf keinen Parameter hat