Re: [Lazarus] LCL Grids: not scalable line width

2019-10-06 Thread Werner Pamler via lazarus

Am 06.10.2019 um 12:14 schrieb AlexeyT via lazarus:


IMO I have found place where width of canvas line is not scaled to 
current DPI.


procedure TCustomGrid.ChangeCursor(ACursor: TCursor;

...
Canvas.Pen.Width:=3;

...
Canvas.Pen.Width:=1;

AFAIK, pen widths so far are not scaled at all. If they were, 
pen.Width=1 should be scaled, too, and this can have a detrimental 
effect on line styles and drawing speed, at least on Windows.
-- 
___
lazarus mailing list
lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


[Lazarus] LCL Grids: not scalable line width

2019-10-06 Thread AlexeyT via lazarus
IMO I have found place where width of canvas line is not scaled to 
current DPI.


procedure TCustomGrid.ChangeCursor(ACursor: TCursor;

...
Canvas.Pen.Width:=3;

...
Canvas.Pen.Width:=1;

--
Regards,
Alexey

-- 
___
lazarus mailing list
lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] LCL Controls.pp needs 'inline'

2019-10-06 Thread Florian Klämpfl via lazarus

Am 06.10.19 um 10:45 schrieb Juha Manninen via lazarus:

I inlined the non-deprecated one liners in r62000.
BTW, ChangeBiDiModeAlignment is not a one liner. You must be careful
with "inline". The effect may become negative if the function has more
code.


For the record: FPC has also -Ooautoinline. IMO it is much better to use 
(and tune) this one instead of adding manually "inline".

--
___
lazarus mailing list
lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus


Re: [Lazarus] LCL Controls.pp needs 'inline'

2019-10-06 Thread Juha Manninen via lazarus
I inlined the non-deprecated one liners in r62000.
BTW, ChangeBiDiModeAlignment is not a one liner. You must be careful
with "inline". The effect may become negative if the function has more
code.

Juha
-- 
___
lazarus mailing list
lazarus@lists.lazarus-ide.org
https://lists.lazarus-ide.org/listinfo/lazarus