Re: CRefArray doesn't respect C++ copy semantics

[Alok Gandhi]

The docs say that: CRefArray& operator= ( const CRefArray &  in_refArray ) Assigns a CRefArray object to this one. Parameters: in_refArray A constant CRefArray object. Returns: A new reference object. So what I think is happening is that the copy constructor is doing exactly what it is supposed to do and returns the new CRefArray object which still points to a1, 'assigns' is the operative word here. To keep them separate I would rather do:     CRefArray a1;     a1.Add(CRef());     a1.Add(CRef());     CRefArray a2;     for(int i=0; i<a1.GetCount(); i++)     {         a2.Add(a1[i]);     }     a2.Add(CRef());     //a2.Add(CRef());     LONG n1 = a1.GetCount();  // expected n1 == 2     LONG n2 = a2.GetCount();  // expected n2 == 3 which gives me correctly: # VERBOSE : cRefArrayTest_Execute called # VERBOSE : Count a1: 2 # VERBOSE : Count a2: 3 On 4/30/2012 7:13 PM, Nicolas Burtnyk wrote: Yeah, exactly as I unfortunately discovered :( On Mon, Apr 30, 2012 at 3:49 PM, Alok Gandhi <alok.gan...@modusfx.com> wrote: A quick test gives me following result: # VERBOSE : cRefArrayTest_Execute called # VERBOSE : Count a1: 3 # VERBOSE : Count a2: 3 On 4/30/2012 6:24 PM, Nicolas Burtnyk wrote: I ran into this today while trying to figure out why my code was broken. Thought I'd pass this along and hopefully save someone some wasted time in the future... CRefArray a1; a1.Add(CRef()); a1.Add(CRef()); CRefArray a2(a1); a2.Add(CRef()); LONG n1 = a1.GetCount();  // expected n1 == 2 LONG n2 = a2.GetCount();  // expected n2 == 3 I expected a2 to be a copy of a1 before the last add and so I assumed a1 would have 2 elements. Instead, I was surprised to find that n1 == n2 == 3! No virus found in this message. Checked by AVG - www.avg.com Version: 2012.0.1831 / Virus Database: 2090/4557 - Release Date: 10/17/11 Internal Virus Database is out of date. No virus found in this message. Checked by AVG - www.avg.com Version: 2012.0.1831 / Virus Database: 2090/4557 - Release Date: 10/17/11 Internal Virus Database is out of date.