[Kujo]

'Kujo' wrote:

Hello,

I have some problems with TsNotebook (prior versions work fine):

• The “SkinSection” property doesn't work (the “CHECKBOX” value is always used). To demonstrate this issue is enough to put the component in a form, change the “SkinSection” property and run the program
• If the TsNotebook is placed in a DLL form, when the program is closed a memory leak error appears (if you replace the component with other components, all is ok). This is visible also if you deactivate the SkinManager component. To demonstrate this issue I have attached a demo program. Both files (EXE and DLL) were compiled with the “Link with runtime packages” activated and the only package explicitly specified is acntDelphiXE2_R.

I'm using RAD Studio XE2 on Windows 7 64 bit.

Thank you in advanced.

About the first problem:

maybe it is in the “acNotebook.TsNoteBook.WndProc(var Message: TMessage)” procedure because I think the “sd.SkinSection := s_CheckBox;” line should be changed with “sd.SkinSection := Self.FCommonData.SkinSection;”

About the second problem:

it is also visible in a EXE file. Simply put the TsNotebook in the form (is not necessary to add the SkinManager and/or SkinProvider components), add the “ReportMemoryLeaksOnShutdown := True;” line to the dpr file, run and close the program.

I think the problem is in the procedure “acNotebook.TsNoteBook.WndProc(var Message: TMessage)” where a TsCommonData (sd) is created but not destroyed

[Support]

Thank you for the demo. I can send you the patched file already, if you have sources. 🙂

Otherwise, you can wait the nearest release.

[Kujo]

'Support' wrote:

Thank you for the demo. I can send you the patched file already, if you have sources. 🙂

Otherwise, you can wait the nearest release.

Yes, I have the source code.

Thank you.

[Support]

Here this file is attached.

[Kujo]

'Support' wrote:

Here this file is attached.

This file solves the memory leak error but NOT the SkinSection error

[Support]

'Kujo' wrote:

This file solves the memory leak error but NOT the SkinSection error

It's solved now too, file is updated.

[Kujo]

'Support' wrote:

It's solved now too, file is updated.

It works perfectly. :a3:

Thank you very much.

[Author]

Posts