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 <chrrossman...@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@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
>
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
