On Sun, Sep 01, 2013 at 12:39:25AM -0700, julien2412 [via Document Foundation Mail Archive] wrote:
<br/>> Lionel Elie Mamane wrote
<br/><br/>>> On the gripping hand, I can't find where the "//Loader" key is used
<br/>>> afterwards either, so maybe my search-fu simply failed me. Or maybe
<br/>>> the whole function is pointless. As an added bonus remark, it seems
<br/>>> weird (but maybe innocuous) that it goes to extra pains to create
<br/>>> "//UNO/Loader" and "//Loader" as opposed to "/UNO/Loader" and
<br/>>> "/Loader".
<br/><br/>> I thought at first sight it would be "//Loader" and "//UNO/Loader" but then
<br/>> I noticed line 159
<br/>> aImpl += DBContentLoader::getImplementationName_Static();
<br/><br/>> So in fact, it would be:
<br/>> "/org.openoffice.comp.dbu.DBContentLoader/Loader" and
<br/>> "/org.openoffice.comp.dbu.DBContentLoader/UNO/Loader"
<br/><br/>Yes, right. Missed that :)
<br/><br/>>> And searching more, I see that this function is actually never
<br/>>> called...
<br/><br/>>> So actually, in this case, remove the whole function. And also the
<br/>>> function of the same name in dbaccess/source/filter/xml/dbloader2.cxx
<br/>>> ...
<br/><br/>> About removing, I made an Opengrok  search and found it was used there:
<br/>> dbaccess/source/ui/misc/uiservices.cxx
<br/>> (see
<br/>> <a href="http://opengrok.libreoffice.org/xref/core/dbaccess/source/ui/misc/uiservices.cxx#38" target="_top" rel="nofollow" link="external">http://opengrok.libreoffice.org/xref/core/dbaccess/source/ui/misc/uiservices.cxx#38</a>)
<br/>> I'm not sure but if we remove the function, it means we should remove it too
<br/>> in uiservices.cxx and so change the interface/api, doesn't it?
<br/><br/>My understanding is that this is not "used", that's just the
<br/>declaration, the stuff that is usually put in headers (but here was
<br/>"repeated" in every .c(xx) file where the function was used).
<br/><br/>The actual use was removed by
<br/><br/>commit 8e88ac109dc9eba88db92940d13933fc3a4393d8
<br/>Author: sb <<a href="/user/SendEmail.jtp?type=node&node=4072595&i=0" target="_top" rel="nofollow" link="external">[hidden email]</a>>
<br/>Date:   Fri Sep 10 13:10:07 2010 +0200
<br/><br/>    sb129: #i113189# change UNO components to use passive registration
<br/><br/>-- 
<br/>Lionel
<br/>

        
        
        
<br/><hr align="left" width="300" />
View this message in context: <a href="http://nabble.documentfoundation.org/About-writeDBLoaderInfo-function-in-dbloader-cxx-dbaccess-module-tp4072511p4072595.html">Re: About writeDBLoaderInfo function in dbloader.cxx (dbaccess module)</a><br/>
Sent from the <a href="http://nabble.documentfoundation.org/Dev-f1639786.html">Dev mailing list archive</a> at Nabble.com.<br/>