search

betostafe / castlesand

Automatically exported from code.google.com/p/castlesand
0 stars 0 forks source link

HWND removal #44

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. There are 13 cases of HWND in code.
2. http://www.lazarus.freepascal.org/index.php/topic,12946.0.html suggest to 
remove it to become cross-platform.

We should use:
TPanel, TWinControl...

Please use labels and text to provide additional information.
If all agree I will do it later.

Original issue reported on code.google.com by vytautas...@gmail.com on 27 Apr 2011 at 4:26

GoogleCodeExporter commented 9 years ago 
I think you are right that we need to change this to make it Linux / Win64 
compatible. However, PLEASE use {$IFDEF FPC} around all of your work so you 
don't break the Delphi version by anything you do. HWND works fine in Delphi, 
so we probably won't change it.

Also, Krom might like you to do this in a branch so you can do whatever you 
want without creating any problems for the main trunk build.

Maybe you should mention this to him, I am too busy with assignments at the 
moment to talk.

Original comment by lewinjh@gmail.com on 28 Apr 2011 at 11:53

GoogleCodeExporter commented 9 years ago 
I agree with Lewin. 
Please create a new branch (I can explain how to do that) and continue working 
on Unix port there.

Original comment by kromster80@gmail.com on 28 Apr 2011 at 12:10

GoogleCodeExporter commented 9 years ago 
I got idea to change messagebox(hwnd(nil)... with application.messagebox(... (5 
cases), but have no idea how to deal with renderframe:hwnd and 
renderhandle:hwnd (2x2). At all I do not understand trender.create and create 
in KM-render.pas and can not fix them :(

Original comment by vytautas...@gmail.com on 4 May 2011 at 3:49

GoogleCodeExporter commented 9 years ago

Original comment by kromster80@gmail.com on 20 Nov 2011 at 4:52

GoogleCodeExporter commented 9 years ago 
I think we don't need it yet. And it is definitely not a "bug"

Original comment by kromster80@gmail.com on 17 Mar 2012 at 6:11