Discussion :

[guillaume07]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
bool Liste::add (boost::shared_ptr<Obj> ptf)
{  
    boost::lock_guard<boost::recursive_mutex> lock( mutex_ );
 
    boost::ptr_vector<Obj>::iterator res  = 
find_if( vecPtf_.begin() , vecPtf_.end() , FuncPTF(ptf->param().name()) );
 
    if(res == vecPtf_.end())  
     return false; 
 
      vecPtf_.push_back(ptf.get());   
        return  true;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
 
add(boost::shared_ptr<Obj>(new Obj))

vecPtf est de type ptr_vector est ce correct de faire ainsi ?

[Mat007]

Salut,

Non ce n'est pas correct.
Tu court-circuites le shared_ptr en utilisant get() pour passer la possession du pointeur au ptr_vector...

Remplace par un std::auto_ptr et retire le get() ?

MAT.

[poukill]

Mat007 a raison, ce n'est pas correct.

Mais pourquoi ne pas utiliser std::vector à la place de ptr_vector, puisque tu utilises déjà boost::shared_ptr pour tes éléments ?