About use of sal_IntPtr for FontList::GetSizeAry, aStdSizeAry and mpSizeAry (svtools)

Caolán McNamara caolanm at redhat.com
Sat Sep 12 13:47:29 UTC 2020


On Thu, 2020-09-10 at 21:00 +0200, Julien Nabet wrote:
> Hello,
> 
> IMHO the use of sal_IntPtr in FontList class is wrong.
> 
> Let's take a look at the 2 vars:
> 
> 1) aStdSizeAry
> 
> 2) mpSizeAry

These are font heights, and IMO I feel a humble int is sufficient for
their maximum range (https://gerrit.libreoffice.org/c/core/+/102466) on
looking at what goes into them



More information about the LibreOffice mailing list