https://bugs.documentfoundation.org/show_bug.cgi?id=91527

            Bug ID: 91527
           Summary: Convert misleading use of sal_uIntPtr to size_t
           Product: LibreOffice
           Version: Inherited From OOo
          Hardware: Other
                OS: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: medium
         Component: LibreOffice
          Assignee: [email protected]
          Reporter: [email protected]

Grep for sal_uIntPtr in the code-base (preferably module-by-module, so any
potential patches are of manageable size).

Notice that tons of variables of that type are for things like 'nAnz',
'nCount', 'nFlags' etc, so clearly no pointer-y things at all.

To further reduce the WTF-factor of the code base, replacing those obvious
cases with size_t instead would be great.

To the best of my knowledge, both types are defined equivalently on all current
platforms, still, strive to ideally rename the entire usage around one
occurence - i.e. change the function signature, all call sites, and any
depending variables. Make sure to build the code with werror enabled.

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to