Re: [Boost-users] [multi_index] Problem of scope with the replace in static data structure

Subject: Re: [Boost-users] [multi_index] Problem of scope with the replace in static data structure
From: joaquin_at_[hidden]
Date: 2009-02-11 04:44:57

Rodrigo Dias Ferreira escribió:
>> 2. If the answer to 1 is no, can you post the debug console
>> output of your program?
>>     
> The output of this part of the code (I will mark the part
> inside the function commitChanges();:
> ///////////begin commitChanges();/////////////////////////////
> DEBUG_at_commitChanges:before replacing:_git Name: COI:0351-123-4555
> Node version was not found!
> DEBUG_at_commitChanges:after replacing:_git Name: COI:0351-123-4555
> Found it: Name: Node:0351-123-4569 Timestamp: 1234340512
> Found it: Name: Node:0351-123-4569 Timestamp: 1234340521
> ///////////end commitChanges();/////////////////////////////
> Commit was successful!
> Retrieving the graph with commit:
> Node version was not found!
>   
[...]

Nothing illuminating here... BTW, the code you ran to produce this 
output cannot be the
same you posted before, as the traces are incompatible --the sentence 
"Retrieving the
graph with commit:" appears nowhere in the code.

>> 3. You're copying values of the type Graph. How's this type defined?
>> Has it proper copy semantics?
>>     
> Yes, I think so:
> class Graph {
> ...
> public:
>
>     // TODO: Really constructors
>     /// Default Constructor
>     Graph() {
>         _rangeBegin = 0;
>         _rangeEnd = 0;
>         }
>     /// Copy constructors
>     Graph(const Graph &g) : _rangeBegin(g._rangeBegin), 
> _rangeEnd(g._rangeEnd),
>         _nodeRep(g._nodeRep), _nodeNameRep(g._nodeNameRep),
>         _nodeTimeStampView(g._nodeTimeStampView), 
> _localPropRep(g._localPropRep),
>         _globalPropRep(g._globalPropRep){}
> ...
> }
>   

It's hard to know whether you got copy semantics right without knowing 
the types
of _rangeBegin, _rangeEnd and the rest of Graph data members, but this 
has prompted
me to suspect that maybe there's something wrong with the handler type 
Graph_h (which
I assume is some sort of ref-counted reference to Graph, right?). Can 
you please
test the following? In your //Test of the commit section, rather than 
reuse graph_h please
use a different variable to do the after-commitChanges check:

 //Test of the commit
 if (graph_h.commitChanges())
     cout << "Commit was successful!\n";
 else
     cout << "Commitretr was unsuccessful!\n";
 // Debug

 //Graph_h graph_h;
 //NodeIterator nodeIt;
 //Node_h node_h;

 graphIt it = DynamicGraphs::_graphRep.find("COI:0351-123-4555");
 Graph_h graph_h2 = Graph_h(it); // WE DON'T REUSE graph_h
 //propertystream_h propS;
 propS = graph_h2->lookupLocalProp("name");
 p_h = propS.next();
 cout << "DEBUG_at_main:right after the commit:_git " << p_h.value() <<endl;
 nodeIt = graph_h2->nodeIterator("Node:0351-123-4569");
     if (!nodeIt)
         cout << "Node version was not found!"<<endl;
     else //Works for both: one match and range lookup
         while (!nodeIt.end()){
             node_h = nodeIt.next();
             cout << "Found it: " << *node_h;
         }
 //Debug

Does this make any difference?

Joaquín M López Muñoz
Telefónica, Investigación y Desarrollo