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

Gtk3: safer way to free allocated data.

Browse Source
This commit is contained in:
zeljan1 2025-01-09 21:27:21 +01:00
parent 324cb06157
commit 51a8bb1b80
1 changed files with 8 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
lcl/interfaces/gtk3/gtk3widgets.pas
View File

@ -542,7 +542,6 @@ type
procedure SetListBoxStyle(AValue: TListBoxStyle);
procedure SetMultiSelect(AValue: Boolean);
protected
procedure DestroyWidget; override;
function CreateWidget(const {%H-}Params: TCreateParams):PGtkWidget; override;
function EatArrowKeys(const {%H-}AKey: Word): Boolean; override;
public
@ -5408,6 +5407,12 @@ begin
TGtk3Widget(AData).DeliverMessage(Msg, False);
end;
procedure FreeStoreStringList(aData: gpointer); cdecl;
begin
TGtkListStoreStringList(aData).Free;
end;
function TGtk3ListBox.CreateWidget(const Params: TCreateParams): PGtkWidget;
var
AListBox: TCustomListBox;
@ -5433,7 +5438,8 @@ begin
g_object_unref (liststore);
ItemList := TGtkListStoreStringList.Create(PGtkListStore(PGtkTreeView(FCentralWidget)^.get_model), 0, LCLObject);
g_object_set_data(PGObject(FCentralWidget),GtkListItemLCLListTag, ItemList);
g_object_set_data_full(PGObject(FCentralWidget),GtkListItemLCLListTag, ItemList, @FreeStoreStringList);
// g_object_set_data(PGObject(FCentralWidget),GtkListItemLCLListTag, ItemList);
Renderer := LCLIntfCellRenderer_New();
@ -5598,20 +5604,6 @@ begin
end;
end;
procedure TGtk3ListBox.DestroyWidget;
var
aData: TGtkListStoreStringList;
begin
if FCentralWidget <> nil then
begin
aData := TGtkListStoreStringList(g_object_get_data(PGObject(FCentralWidget),GtkListItemLCLListTag));
if aData <> nil then
aData.Free;
g_object_set_data(PGObject(FCentralWidget),GtkListItemLCLListTag, nil);
end;
inherited DestroyWidget;
end;
function TGtk3ListBox.GetSelCount: Integer;
var
Selection: PGtkTreeSelection;