What is wrong with this method?

Hi, The method below is: 1. Quite slow 2. Sometimes gives an uncaught exception error. I have no additional debugging information unfortunately.

I'm not a C++ expert. Can you please tell me if it can be sped up and if it can be made more robust to avoid the uncaught exception? Thank you!

CellComplex::Ptr CellComplex::ByCells(const std::list<Cell::Ptr>& rkCells, const bool kCopyAttributes)
    {
        // ByOcctSolids does the actual construction. This method extracts the OCCT structures from the input
        // and wrap the output in a Topologic class.
        TopTools_ListOfShape occtShapes;
        for (const Cell::Ptr& kpCell : rkCells)
        {
            occtShapes.Append(kpCell->GetOcctShape());
        }

        TopoDS_CompSolid occtCompSolid = ByOcctSolids(occtShapes);
        CellComplex::Ptr pCellComplex = std::make_shared<CellComplex>(occtCompSolid);
        CellComplex::Ptr pCopyCellComplex = std::dynamic_pointer_cast<CellComplex>(pCellComplex->DeepCopy());
        if (kCopyAttributes)
        {
            std::list<Topology::Ptr> cellsAsTopologies;
            for (const Cell::Ptr& kpCell : rkCells)
            {
                cellsAsTopologies.push_back(kpCell);
                AttributeManager::GetInstance().DeepCopyAttributes(kpCell->GetOcctSolid(), pCopyCellComplex->GetOcctCompSolid());
            }
            CellComplex::Ptr pCopyCellComplex = TopologicalQuery::Downcast<CellComplex>(pCellComplex->DeepCopyAttributesFrom(cellsAsTopologies));
        }
        //GlobalCluster::GetInstance().AddTopology(pCopyCellComplex->GetOcctCompSolid());

        return pCopyCellComplex;
    }