paweld/lazarus
1
0
Fork 0
mirror of https://gitlab.com/freepascal.org/lazarus/lazarus.git synced 2025-04-18 15:49:27 +02:00
Code Issues Packages Projects Releases Wiki Activity

TAChart: Make TCustomSortedChartSource.Sort safer and more versatile. Part of patch by Marcin Wiazowski.

Browse Source 
git-svn-id: trunk@60981 -
This commit is contained in:
wp 2019-04-15 08:32:20 +00:00
parent f308975433
commit 2d7a2cb821
1 changed files with 15 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
components/tachart/tasources.pas
View File

@ -29,9 +29,11 @@ type
procedure SetSorted(AValue: Boolean);
procedure SetOnCompare(AValue: TChartSortCompare);
protected
FCompareProc: TChartSortCompare;
FData: TFPList;
FSorted: Boolean;
function DefaultCompare(AItem1, AItem2: Pointer): Integer; virtual; abstract;
function DoCompare(AItem1, AItem2: Pointer): Integer; virtual;
procedure ExecSort(ACompare: TChartSortCompare); virtual;
procedure SetSortBy(AValue: TChartSortBy); override;
procedure SetSortDir(AValue: TChartSortDir); override;
@ -523,6 +525,11 @@ end;
{ TCustomSortedChartSource }
function TCustomSortedChartSource.DoCompare(AItem1, AItem2: Pointer): Integer;
begin
Result := FCompareProc(AItem1, AItem2);
end;
{ Built-in sorting algorithm of the ChartSource, a standard QuickSort.
Copied from the classes unit because the compare function must be a method. }
procedure TCustomSortedChartSource.ExecSort(ACompare: TChartSortCompare);
@ -614,9 +621,14 @@ procedure TCustomSortedChartSource.Sort;
begin
if csLoading in ComponentState then exit;
if (FSortBy = sbCustom) then begin
if Assigned(FOnCompare) then ExecSort(FOnCompare);
end else
ExecSort(@DefaultCompare);
if not Assigned(FOnCompare) then exit;
FCompareProc := FOnCompare;
end else begin
if (FSortBy = sbX) and (FSortIndex <> 0) and (FSortIndex >= FXCount) then exit;
if (FSortBy = sbY) and (FSortIndex <> 0) and (FSortIndex >= FYCount) then exit;
FCompareProc := @DefaultCompare;
end;
ExecSort(@DoCompare);
Notify;
end;