[Libreoffice] comment unnecessary?

[Xisco Faulí]

yeah, it's right, the code looks as follow:

        OSL_ENSURE( aInfo.mpOldBasicPassword,
"ImplRepository::impl_createManagerForModel: wrong BasicLibraries
implementation!" );
        pBasicManager->SetLibraryContainerInfo( aInfo );
        //pBasicCont->setBasicManager( pBasicManager );
            // that's not needed anymore today. The containers will retrieve
their associated
            // BasicManager from the BasicManagerRepository, when needed.

        // initialize the containers
        impl_initDocLibraryContainers_nothrow( xBasicLibs, xDialogLibs );

So it can be deleted because it refers to //pBasicCont->setBasicManager(
pBasicManager );

2011/3/20 Christina Roßmanith <ChrRossmanith at web.de>

> Hi,
>
> I'm reviewing and applying patches removing commented code. What about the
> following lines from source/basmgr/basicmanagerrepository.cxx?
>
>       //pBasicCont->setBasicManager( pBasicManager );
>             // that's not needed anymore today. The containers will
> retrieve their associated
>             // BasicManager from the BasicManagerRepository, when needed.
>
>
> I have the feeling, that the text-comment belongs to the commented line of
> code above. Is that true? In that case it can be removed as well (I'll do
> that if I get an ok for that).
>
> Christina
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/libreoffice/attachments/20110320/a99eb604/attachment.html>