Menu assignment has lot of unneeded LCL code

[AlexTP]

Form1.Menu:= MainMenu1;
has lot of code: some destroying of menus, some validating handles... Imho at least on Win32 it should be simple code:

Windows.SetMenu(Form1.Handle, Menu1.Handle); //or 0 if to hide

Agree?

[ASerge]

Form1.Menu:= MainMenu1;
has lot of code: some destroying of menus, some validating handles... Imho at least on Win32 it should be simple code:

TMenuItem is a separate wrapper objects around the system menu items. Therefore, they should be removed if menu items are not used.

[AlexTP]

Sorry for noise, I see that all LCL code is needed.