Kleiner Bug in Graphics32

[Schorschi5566]

Servus Gemeinde,

in GR32_image.pas ist meiner Meinung nach ein kleiner Bug enthalten, der das Bitmap-Alignment betrifft. Dadurch funktioniert baCenter nicht richtig. Schuld ist mal wieder ein With-Statement.

File: GR32_image.pas
Zeile: 1533 ff
Falscher Code:

Delphi-Quellcode:

    Size := GetBitmapSize;
    with Size do
    begin
      Result := Rect(0, 0, Cx, Cy);
      if BitmapAlign = baCenter then
        OffsetRect(Result, (Width - Cx) div 2, (Height - Cy) div 2)
      else if BitmapAlign = baCustom then
        OffsetRect(Result, Round(OffsetHorz), Round(OffsetVert));
    end;

Richtiger Code:

Delphi-Quellcode:

    Size := GetBitmapSize;
    Result := Rect(0, 0, Size.Cx, Size.Cy);
    if BitmapAlign = baCenter then
      OffsetRect(Result, (Width - Size.Cx) div 2, (Height - Size.Cy) div 2)
    else if BitmapAlign = baCustom then
      OffsetRect(Result, Round(OffsetHorz), Round(OffsetVert));

Der Fehler ist schon seit längerem enthalten und ich habe ihn jetzt mal im Tracker von GR32 gemeldet...


Grüße,
Uwe

[Union]

Gut. Aber erschreckend dass es immer noch Leute gibt, die ihren Source nicht checken bevor sie ihn ausliefern. Haben sie sicher bei Embobabu gelernt.

[Delphi-Laie]

Zitat von Union:

Gut. Aber erschreckend dass es immer noch Leute gibt, die ihren Source nicht checken bevor sie ihn ausliefern.

"Source checken"? Beim bloßen Betrachten des Quelltextes wird man die wenigsten Fehler entdecken. Die Fehler offenbaren sich beim Laufen der Programme bzw. Compilate, dann muß "rückwärts" in den Quelltexten die Ursache gesucht werden. Manchmal ist aber auch der Auslieferer der Programmierumgebung der eigentlich Schuldige.

Zitat von Union:

Haben sie sicher bei Embobabu gelernt.

Nun, ein Unternehmen, das immensen Aufwand bezüglich des Verhinderns der illegalen Nutzung seiner Programme betreibt, das muß damit logischerweise einen Teil seiner Aktionen, seiner Aufmerksamkeit dem eigentlichen, nämlich der Qualität seiner Programmierumgebung, opfern (das ist auch am neuen XE2 festzustellen, kann man wohl als gewisse Ernüchterung bezeichnen).

Das betrifft beileibe aber nicht nur "Embobabu", sondern dürfte sich inzwischen so ziemlich durch alle großen Programme (inkl. Betriebs-) bzw. Softwareunternehmen ziehen.

[Union]

Mit "Source checken" meinte ich natürlich einen Test. Und ich verstehe darunter nicht einen einmaligen Aufruf von Funktionen, sondern es sollte JEDE Programmzeile mindestens einmal angesprochen worden sein. Dann findet man auch sowas.

[Thom]

@Union:

Aha. Dann garantiert Deine Firma also eine absolut fehlerfreie Software!? Und die wird dann auch noch kostenlos abgegeben?
Das wäre mal was Neues und echt beispielhaft.

Zeig doch mal - nur so zur Information - eure AGB's...

[mkinzler]

Es handelt sich um ein OpenSource Projekt, welches Leute in ihrer Freizeit erstellen und kostenfrei der Allgemeinheit zur Verfügung stellen. Da finde ich deine Forderungen sehr überzogen!

[Union]

@thom, mkinzler: Ihr seid also dagegen, erstellte Software vor der Auslieferung gründlich zu testen? Gut zu wissen...

[mkinzler]

Nein, aber man darf das bei OS Software nicht zu Voraussetzung machen. Zudem ist das ja eine Tätigkeit, welche durchaus durch die Community erledigt werden kann.

[wicht]

Zitat:

@thom, mkinzler: Ihr seid also dagegen, erstellte Software vor der Auslieferung gründlich zu testen? Gut zu wissen...

Das hat so niemand gesagt. Freu dich doch, dass es kostenlos und Open-Source ist und du den Fehler selber finden konntest. Bei einem Produkt zum kaufen ohne Quelltexte würdest du jetzt vielleicht ein paar Wochen auf eine Antwort vom Support warten, die dich dann auf das nächste (kostenpflichtige?) Release vertrösten könnten...

[Union]

Meine Kritik war auch mehr auf eine bestimmte Firma bezogen. Die Herleitung war, dass wenn der Hersteller der IDE permanent schlampt, die Programmierer meinen könnten dies wäre der neue Standard. Natürlich gibt es fast keine fehlerfreie Software, aber durchaus Methoden die Anzahl der Fehler stark zu reduzieren. Diese Methoden basieren sind kein Geheimnis, lange bekannt und basieren eher auf Gründlichkeit und Verantwortung als auf komplizierter Technik.

Im konkreten Fall habe ich die GR32 auch schon eingesetzt, die ist ja ok. Was die IDE betrifft setze ich immer noch D7 ein Ein paar Versionen zwischendurch habe ich auch upgedatet, aber so gut wie nie produktiv genutzt. Aber durchaus getestet und den Schritt des produktiven Einsatzes nie gewagt. Sagen wir es mal ganz hart: Programmierer mit dem Qualitätsanspruch würden normalerweise die Probezeit nicht überstehen.