Date: prev next · Thread: first prev next last
2011 Archives by date, by thread · List index


Hi there,

On 27 June 2011 13:20, Michael Meeks <michael.meeks@novell.com> wrote:
But sometimes there also exist things like in:
http://opengrok.libreoffice.org/xref/components/extensions/source/preload/services.cxx#59
That doesn't seem interesting to me, but I don't know, maybe it
actually does something relevant.

       Ooh - you discovered some more junk code - nice. It would be wonderful
if you could hunt down and kill any uses of this 'preload' code - we
don't do OEM preloads, similarly we don't have product registration
either so killing both of these components and their related scp2/ would
be nice:

STD_LIB_FILE( gid_File_Lib_Preload, preload)
SPECIAL_COMPONENT_LIB_FILE( gid_File_Lib_Productregistration, productregistration.uno )

       would help to shrink the codebase, distributed binary size and compile
time.

Well, I had a look at preload.
It's used just in desktop/source/so_comp/socomp.component and that
component is used just in:
- desktop for http://opengrok.libreoffice.org/xref/libs-core/desktop/source/app/app.cxx#1675
    - maybe CheckInstallation should be now always true?
- desktop, sfx2, similar checkOEM methods
    - http://opengrok.libreoffice.org/xref/libs-core/desktop/source/app/app.cxx#CheckOEM
      - I'm not sure about this one. Whether that should be always
true or false or not removed.
    - the same with
http://opengrok.libreoffice.org/xref/libs-core/sfx2/source/appl/shutdowniconw32.cxx#445

So, if it's possible to get rid of checkOEM and CheckInstallation
methods we could remove preload.component and also socomp.component
and related source files, libraries, ...
Is it possible? I mean, both should be always true?

Thanks,
Matus

Context


Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.