Sign in Join

Live chat for Free Pascal, Lazarus, Delphi and every Pascal programmer.

Mailing lists / customdrawn / 2012-03

Customdrawn short term plans

2 messages · 1 person ·

Reply by email Subscribe to this list Your subscription

Read-only archive. Use j and k to move between messages.

Giuliano Colla

Felipe Monteiro de Carvalho ha scritto:

Thanks, applied in rev 36189 with modifications to still support the old paint model via a define.

I too had been tempted to keep the option, just in case.

Giuliano

Giuliano Colla Whenever people agree with me, I always feel I must be wrong (O. Wilde)
Giuliano Colla

Il 21/03/2012 12:18, Felipe Monteiro de Carvalho ha scritto:

Thanks, applied in rev 36189 with modifications to still support the old paint model via a define.

I updated and gave a quick check.

You omitted a correction I had made to an error of mine in customdrawnproc.pas, where TT_Close_Face(AFace) was closed twice (once too early, and the second time when I'm really done with AFace). It doesn't make a difference, it doesn't complain, but it's an error nonetheless.
I attach a diff, where in the local copy the offending line is just commented out, to locate it more quickly.

There's no hurry to commit, because apparently it doesn't make any harm. But one never knows, maybe with the next release of lazfreetypes it could give rise to obscure errors.

Giuliano

-------------- next part --------------
A non-text attachment was scrubbed...
Name: bug.diff
Type: text/x-patch
Size: 360 bytes
Desc: not available
URL: <http://lists.lazarus.freepascal.org/pipermail/customdrawn/attachments/20120321/6e8febd8/attachment.bin>