Giulio Bernardi wrote:
I quickly looked at commit 23290. A couple of things: use makeintresource
instead of doing pchar(resid), since this might change in the future, and
pay attention to the endianess: all data structures are little endian.
Thanks. I will make few changes.
Now, I have some doubts: rebuilding an .ico stream seems an overkill to me.
I've not been using GUI stuff like lazarus or delphi since some times, so
I might be wrong in the following lines.
Isn't TIcon supposed to contain a single icon, and not a whole set of icons
as defined in a .ico file or in a RT_GROUP_ICON resource?
No. Lazarus TIcon has a set of images with different sizes and color depths.
This is because googling I found some example code that does things like this:

icon.handle:=LoadIcon(hInstance,'MYICON');
This is possible but we will only have one icon which is not we want.
I suppose that TIcon.LoadFromStream loads the first icon in the .ico stream.
Can you confirm it?
You are wrong.

Later we will use fcl-res probably but right now we can't depend on a non-released package.

Best regards,
Paul Ishenin.
_______________________________________________
fpc-devel maillist  -  [email protected]
http://lists.freepascal.org/mailman/listinfo/fpc-devel

Reply via email to